[Rating]: New TEDI-Ready component

DoD

The component is developed using best practices, conventions, and modern frontend trends
The component follows BEM naming conventions (if applicable)
The component implementation matches the Figma design, including all defined states and interactions
The component supports responsiveness and scales across breakpoints (if applicable)
The component does not introduce runtime errors or warnings
Linter passes
Component has export if needed
The component has unit tests with at least 80% code coverage
CodeCov passes minimum coverage threshold
The component has passed code review
The component has passed Chromatic visual review
Storybook stories are created:
- Cover all Figma variations
- Include edge cases, empty/error states
Component’s API (props, slots, events) is documented
ZeroHeight is updated with design/development guidelines
External dependencies used in the component are documented in Confluence with rationale.
Semantic commit format is used (for semantic-release)
External libraries comply with validated open source licenses
Component is versioned correctly and ready for release if required
Update statuses page in Zeroheight

https://storybook.tedi.ee/react/feat/531-rating-new-tedi-ready-component/?path=/docs/tedi-ready-components-form-rating--docs

@airikej

@airikej

tärnide all olev tekst peaks olema extra small e 12px, praegu paistab small ja 14px.
Tekst peaks Figma järgi olema joondatud tärnide suhtes keskele.
Kui ratingut ei ole veel ja see tähendab ka, et selgitavat teksti all ei ole, siis kursoriga tärnide peale minnes ei tohiks seda hüppamist olla, mis tuleb teksti lisandumisega. Kui see on mingi hinnangu vorm, mille üks osa on need tärnid ja sellele järgneb veel vormivälju või elemente, siis on see hüppamine häiriv. Ma vaatan, et figmas on ilma rateinguta variandi default kõrgus sama, mis tekstiga, ehk ratinguga.

Numbritega skaalal peaks olema mõlemas otsas tekstiline väärtus, praegu on ainult vasakul, ehk "negatiivses" otsas. Peaks olema skaala halvast heani, et vasakul ääres "väga halb" ja paremal ääres "Suurepärane"
Numbritega skaala peaks mobiilis/kitsastel ekraanidel minema kahele reale, FIgmas allpool on näide, kus on esimesed 5 numbrit ühel real ja viimased viis teisel real. Samuti on numbrid 44px, hetkel paistab mobiilis 40px. Tekst mobiilis peaks endiselt jääma extra small e 12px
Ma eeldan, et neid labeleid e tekste saab muuta seal all ja see kaotab ära selle probleemi näiteks, et ikoonidega variandi puhul on hetkel tekstid liiga pikad ja lohisevad ja lükkavad selle rea veidraks (sest kasutavad samu tekste, mis tärnidega variant). Iga projekt saab siis ise panna sinna mis tahes lühikese teksti ja sellega on element kontrollitav?
Kas disabled state on vajalik, jäin ise praegu mõtlema, et mis puhul seda kuvada? Read only on jah üks olukord, mida oleks vaja disainis juurde lisada aga mis puhul oleks disabled vajalik?
Stated peab disainis nagunii üle vaatama ja mõningad lisama, focus oleks kindlasti vajalik.

fix(rating): fix test coverage, fix accessibility bug #531

feat(rating): cr fixes #531

@Liberiina

@Liberiina parandused/täiendused sisse viidud

@mart-sessman

@mart-sessman

Üks hover state veel muutus, et oleks ühtasemad stiilid. Tähe hover state sai helesinise taustavärvi nagu teistelgi.

States sektsioon võiks ühtlasem olla, see keskmine plokk numbritega sõidb veidi.

Nendel tähekestel on heveritega midagi sassis, hover ei peaks enam seda tähte seest ära täitma tumesiniselt. Stated nagu on õiged, kui state storyt vaadata aga miskipärast hoverdades kuvab ikkagi vana olekut.
Samas state storys on ka kuidagi nii praegu, et kui ma seal peal hoverdan, siis peale seda ta enam seda helesinist tausta ei näita hoveri olekuks, mis tal olema peaks. Ta nagu muutub defaultiks peale hoverdamist. Kui lehte refreshin siis ilmub jälle tagasi.

Focus states vist renderdab mingit äärt seal taustal? Sinise ümbritseva ringi sees tekib nagu mingi säbru? Siin puldil ta võimendab seda aga kui vaadata 1:1 storybookis, siis virvendab see äär.

Kui ma read only propsiga mängin default storys, siis seal kuvab neid välja valge taustaga (mõtlen tähekest ja ikooni), tegelikult peaks read only variandis olema see täidetud. Kui on täis skaala, siis on osad täidetud ja osad mitte.

fix(rating): applied design review changes #531

@Liberiina

@Liberiina viisin muudatused sisse

fix(rating): increased star style specificity #531

docs(rating): readOnly description improvement #531

Navigation Menu

Repository navigation

[Rating]: New TEDI-Ready component #531

DoD

23 remaining items

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

[Rating]: New TEDI-Ready component #531

Description

DoD

Activity

airikej commented on Jun 26, 2026

Liberiina commented on Jun 30, 2026

23 remaining items

airikej commented on Oct 4, 2026

Liberiina commented on Oct 8, 2026

mart-sessman commented on Oct 9, 2026

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions