-
Notifications
You must be signed in to change notification settings - Fork 5
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
ui-speedspacechart : add speed limit tags layer #79
Conversation
2eeccf8
to
1417f9d
Compare
9a512a5
to
42e3e48
Compare
dd1c3ee
to
ed2d8d6
Compare
b5b6ae6
to
731e3bc
Compare
ui-speedspacechart/src/components/helpers/drawElements/speedLimitTags.ts
Outdated
Show resolved
Hide resolved
ui-speedspacechart/src/components/helpers/drawElements/speedLimitTags.ts
Outdated
Show resolved
Hide resolved
bd95633
to
b0a8c89
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.
Thank you for this PR and great work, especially for all the svg/icons utils, it must have been painfull !
Left comments and suggestions.
ui-speedspacechart/src/components/helpers/drawElements/speedLimitTags.ts
Outdated
Show resolved
Hide resolved
ui-speedspacechart/src/components/helpers/drawElements/speedLimitTags.ts
Outdated
Show resolved
Hide resolved
ui-speedspacechart/src/components/helpers/drawElements/speedLimitTags.ts
Outdated
Show resolved
Hide resolved
ui-speedspacechart/src/components/helpers/drawElements/speedLimitTags.ts
Outdated
Show resolved
Hide resolved
ui-speedspacechart/src/components/helpers/drawElements/speedLimitTags.ts
Outdated
Show resolved
Hide resolved
ui-speedspacechart/src/components/helpers/drawElements/speedLimitTags.ts
Outdated
Show resolved
Hide resolved
ui-speedspacechart/src/components/helpers/drawElements/speedLimitTags.ts
Outdated
Show resolved
Hide resolved
ui-speedspacechart/src/components/helpers/drawElements/speedLimitTags.ts
Outdated
Show resolved
Hide resolved
ui-speedspacechart/src/components/helpers/drawElements/speedLimitTags.ts
Outdated
Show resolved
Hide resolved
7591fee
to
7972dbe
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.
Lgtm and tested, great job !
- adding fake data for speed limit tags visualisation - add drawSvgImageWithColor, LoadSvgImage and drawRoundRect in utils.ts - adding raw.d.ts to allow raw svg import - adding createSvgBlobUrl to convert svg to string - changed the logic for calculating the linear width - repositioned the linear within the canvas - adding the tooltip above incompatible speed limit tag - modify drawSeparatorLinearLayer to fix the missing separator problem - adding colorDictionary - adding new const after refactorisation
7972dbe
to
f484e9d
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.
lgtm good job !
close OpenRailAssociation/osrd#7732