Tweaked stop symbols and lines according to design - #1528
PasiVuohijoki wants to merge 1 commit into
Conversation
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
Huulivoide
left a comment
There was a problem hiding this comment.
@Huulivoide reviewed 2 files and all commit messages, and made 2 comments.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on PasiVuohijoki).
ui/src/components/Map/Stops/ExistingStops/Stop.tsx line 82 at r1 (raw file):
// own color when selected. if (isSelected) { const primaryMode = activeTransportModes.at(0);
Primary mode noudetaan nyt kahdessa eri kohtaa tän function sisällä. Vois nostaa kummatkin haut oleen yhteen kertaan tuolla funktion nokassa
ui/src/components/Map/Stops/ExistingStops/StopMarker.tsx line 173 at r1 (raw file):
{inSelection && ( <path transform="translate(7 7) scale(0.65) translate(-7 -7)"
Äkkiseltään kun atteelen niin eiks tästä vois pudottaa pois noi translatete, koska nehän kumoaa toisensa 🤔 eikä ton välissä olevan scalen pitäs mitenkään vaikuttaa siihen että missä koordinaatissa elementin origin on niin ei pitäs olla eroo 🤔
ef4b024 to
78db85e
Compare
PasiVuohijoki
left a comment
There was a problem hiding this comment.
@PasiVuohijoki made 1 comment.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on Huulivoide).
ui/src/components/Map/Stops/ExistingStops/StopMarker.tsx line 173 at r1 (raw file):
Previously, Huulivoide (Jesse Jaara) wrote…
Äkkiseltään kun atteelen niin eiks tästä vois pudottaa pois noi translatete, koska nehän kumoaa toisensa 🤔 eikä ton välissä olevan scalen pitäs mitenkään vaikuttaa siihen että missä koordinaatissa elementin origin on niin ei pitäs olla eroo 🤔
Testasin. Checkkimerkki meni ihan vitun vinoon. 😆
PasiVuohijoki
left a comment
There was a problem hiding this comment.
@PasiVuohijoki made 1 comment.
Reviewable status: 1 of 2 files reviewed, 2 unresolved discussions (waiting on Huulivoide).
ui/src/components/Map/Stops/ExistingStops/Stop.tsx line 82 at r1 (raw file):
Previously, Huulivoide (Jesse Jaara) wrote…
Primary mode noudetaan nyt kahdessa eri kohtaa tän function sisällä. Vois nostaa kummatkin haut oleen yhteen kertaan tuolla funktion nokassa
Done.
Huulivoide
left a comment
There was a problem hiding this comment.
@Huulivoide reviewed 1 file and all commit messages, and resolved 2 discussions.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on PasiVuohijoki).
This change is