Improvements in line page icons#1436
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 5 files and all commit messages, and made 1 comment.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on suvikankare).
ui/src/components/routes-and-lines/line-details/LineDetailsByIdPage.tsx line 55 at r1 (raw file):
); default: return null;
Meillä on kuitenkin ne muutkin ikonit olemassa ja värit tiedossa, niin ne vois saman tien nakata tänne sisään, niin ei tarvi sitten uudelleen joskus tulevassa tänne palata tai ottaa sitä vaaraa että tulee joku bugi tiketti ku sieltä kuitenkin saa valita sen metron tai laitan jo nyt, mutta sitten ei ikoni näykkään.
suvikankare
left a comment
There was a problem hiding this comment.
@suvikankare made 1 comment.
Reviewable status: 4 of 20 files reviewed, 1 unresolved discussion (waiting on Huulivoide).
ui/src/components/routes-and-lines/line-details/LineDetailsByIdPage.tsx line 55 at r1 (raw file):
Previously, Huulivoide (Jesse Jaara) wrote…
Meillä on kuitenkin ne muutkin ikonit olemassa ja värit tiedossa, niin ne vois saman tien nakata tänne sisään, niin ei tarvi sitten uudelleen joskus tulevassa tänne palata tai ottaa sitä vaaraa että tulee joku bugi tiketti ku sieltä kuitenkin saa valita sen metron tai laitan jo nyt, mutta sitten ei ikoni näykkään.
Done.
Huulivoide
left a comment
There was a problem hiding this comment.
@Huulivoide reviewed 16 files and all commit messages, and resolved 1 discussion.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on suvikankare).
This change is