Skip to content

Conversation

@ArtemHoruzhenko
Copy link
Collaborator

What

reworked:

  • job screen core layout
  • header for job config container
  • dry run panel

known issues which will be addressed in a separate tasks

  • border/background color for monaco editors
  • no layout for job output (I assume we will change text formatting a bit)
  • tooltip for "SQL and JMESPath Editor".

Testing

Screen.Recording.2025-11-06.at.17.11.11.mov

@github-actions
Copy link
Contributor

github-actions bot commented Nov 6, 2025

Code Coverage - Frontend unit tests

St.
Category Percentage Covered / Total
🟢 Statements 82.53% 20488/24824
🟡 Branches 67.75% 8777/12955
🟡 Functions 77.08% 5492/7125
🟢 Lines 82.94% 20060/24185

Test suite run success

5255 tests passing in 687 suites.

Report generated by 🧪jest coverage report action from df12ae0

KrumTy
KrumTy previously approved these changes Nov 7, 2025
Copy link
Collaborator

@KrumTy KrumTy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good 👍!

@valkirilov valkirilov changed the title Fe/feature/ri 7687 adjust rdi job config page RI-7687 Adjust RDI job config page Nov 7, 2025
valkirilov
valkirilov previously approved these changes Nov 7, 2025
import { FlexItem } from 'uiSrc/components/base/layout/flex'

export const StyledRdiJobConfigContainer = styled(FlexItem)`
padding: 2.4rem;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: You can also use the theme, they have an equivalent variable for this spacing.(example)

Suggested change
padding: 2.4rem;
padding: ${({ theme }: { theme: Theme }) => theme.core.space.space300};

@ArtemHoruzhenko ArtemHoruzhenko merged commit 50ad022 into main Nov 7, 2025
13 checks passed
@ArtemHoruzhenko ArtemHoruzhenko deleted the fe/feature/RI-7687-adjust-RDI-job-config-page branch November 7, 2025 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants