-
Notifications
You must be signed in to change notification settings - Fork 46
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
front: harmonize icons lock, pencil, trash #5894
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## dev #5894 +/- ##
============================================
- Coverage 19.70% 19.70% -0.01%
Complexity 2345 2345
============================================
Files 910 910
Lines 108075 108167 +92
Branches 2634 2634
============================================
+ Hits 21299 21310 +11
- Misses 85216 85297 +81
Partials 1560 1560
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
62acc34
to
681f7cb
Compare
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.
Tested and LGMT.
For the sake of visual harmonization, for example in scenario.tsx, I've proposed other icon changes, but there're not mandatory to approve this PR.
front/src/modules/trainschedule/components/Timetable/TimetableTrainCard.tsx
Show resolved
Hide resolved
Thanks for these comments @Akctarus, we will change the other icons later :) |
closes #5827
Infrastructure edition modal

Rolling stock edition buttons
