Terminals: latest change history

[suvikankare]

This change is 

[github-actions]

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

[Huulivoide]

@Huulivoide reviewed 16 files and all commit messages, and made 2 comments.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on suvikankare).

---

ui/src/components/common/ChangeHistory/SortByButton.tsx line 48 at r1 (raw file):

    <button
      className={twMerge(
        'group flex w-full items-center justify-end gap-2',

Osa sarakkeista on tasattu oikeelle, osa vasemmalle, tässä vois määritellä nää nappien tasaukset tuolla Table komponentin tasolla per nappi

---

ui/src/components/common/ChangeHistory/ChangeHistoryTable.tsx line 25 at r1 (raw file):

// nameCol one eighth to avoid line breaks with names
const dataColWidth = 'w-[calc((max(96rem,100vw)-(10*var(--spacing)))/5)]';
const nameColWidth = 'w-[calc((max(96rem,100vw)-(10*var(--spacing)))/8)]';

Ite onlin näitä leveyksii aiemmin tunkanneena aatellut että nimi on näistä se matalimman prioriteetin kenttä. Että pistetään kaikki muu lähtökohtasesti mahtuu yhelle riville nätisti ja pätkötään sit nimet. Pysäkin esteettämystiedoissa tais olla jotkain megapitkiä yhdyussana käännöksiä, ne vois tarkastaa että ne ei hajoile millään ruudun koolla ja mahdollisesti tunkee niihin soft-hyphen merkin (U+00AD '­' ← tuossa noin hipsujen välissä näkymättämänä, ei tavallinen - väliviiva merkki) väliin sanarajalle, niin selain osaa pätkästä sen oikeesta kohtaa

[suvikankare]

@suvikankare made 2 comments.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on Huulivoide).

---

ui/src/components/common/ChangeHistory/ChangeHistoryTable.tsx line 25 at r1 (raw file):

Previously, Huulivoide (Jesse Jaara) wrote…

Ite onlin näitä leveyksii aiemmin tunkanneena aatellut että nimi on näistä se matalimman prioriteetin kenttä. Että pistetään kaikki muu lähtökohtasesti mahtuu yhelle riville nätisti ja pätkötään sit nimet. Pysäkin esteettämystiedoissa tais olla jotkain megapitkiä yhdyussana käännöksiä, ne vois tarkastaa että ne ei hajoile millään ruudun koolla ja mahdollisesti tunkee niihin soft-hyphen merkin (U+00AD '­' ← tuossa noin hipsujen välissä näkymättämänä, ei tavallinen - väliviiva merkki) väliin sanarajalle, niin selain osaa pätkästä sen oikeesta kohtaa

Designistä tuli palaute siitä et nimi vetää jopa kolmelle riville ja toi otsikkorivi olis hyvä olla mahdollisimman matala, itse muutosdata sai tarvittaessa rivittyä. Sillä perusteella tehty tämä. Koitin valita kaikista pisimmät arvot muutetuiksi, sekä fi että en ei näytä katkovan sanoja keskeltä!

---

ui/src/components/common/ChangeHistory/SortByButton.tsx line 48 at r1 (raw file):

Previously, Huulivoide (Jesse Jaara) wrote…

Osa sarakkeista on tasattu oikeelle, osa vasemmalle, tässä vois määritellä nää nappien tasaukset tuolla Table komponentin tasolla per nappi

Kaikki sortattavat sarakkeet mitä näin oli tasattu oikeelle 🤔

[Huulivoide]

@Huulivoide resolved 2 discussions.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on suvikankare).