-
Notifications
You must be signed in to change notification settings - Fork 65
ui: fix table style and use it in application #14651
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
124aab6 to
dba6163
Compare
71b09c6 to
16e501a
Compare
e44ffff to
e7a25f3
Compare
e5d12ce to
685a062
Compare
c581707 to
ffa4d7d
Compare
19d6dab to
6b15203
Compare
SharglutDev
approved these changes
Jan 28, 2026
Contributor
SharglutDev
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm and tested ✅ Thank you for the good work
SarahBellaha
approved these changes
Jan 28, 2026
Contributor
SarahBellaha
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm !
2c4142e to
18b1768
Compare
936b05c to
2bd9e6c
Compare
Signed-off-by: Theo Macron <[email protected]>
7067527 to
330460d
Compare
330460d to
fedb793
Compare
Signed-off-by: Theo Macron <[email protected]>
Signed-off-by: Theo Macron <[email protected]>
fedb793 to
0482b1f
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I adapted the style so that we don't need to create a new style file in osrd.
Adding the
narrowandcomfortablehorizontal padding AC.Adding fixed width for all columns except OP one.
Adapt application to use this new style in it, which involves adding style classes to the component, renaming it, and creating a custom style for our use (railway-specific).
Certain columns, such as the name, secondary code or computed departure/arrival , must have a “computed” filter applied :
