search

fmfi-svt / anketa

Študentská anketa FMFI a iných fakúlt UK
https://anketa.uniba.sk/
Other
0 stars 1 forks source link

frontend: Pridane nove hviezdicky #225

Closed mrshu closed 7 years ago

mrshu commented 7 years ago

Signed-off-by: mr.Shu mr@shu.io

This change is Reviewable

Sameth commented 7 years ago

Nechce mat ta mouseover hviezdicka rovnaky okraj ked je modra, ako ked nie je modra (cize rovnako, ked je "on", ako ked je "off")? A non-mouseover hviezdicka ma byt realne taka priesvitna? Nieze by som sa do designu nejako moc vyznal, ale nebude to budit dojem, ze "este som vlastne toto hviezdickove hodnotenie neriesil"?

Comments from Reviewable

mrshu commented 7 years ago

@sameth vsetko dobre komentare, v zasade s nimi suhlasim.

Osobne sa do deisgnu nevyznam vobec, len som zmenil sprite, co mi bol poslany.

@Foxiy mozes s tymto nieco robit?

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Sameth commented 7 years ago

LGTM Vyzera to lepsie, ale nie je to uplne to, co som mal na mysli. Kazdopadne suhlasim s tym, ze to chceme spustit a mat aspon nejake nove hviezdicky, mozme si pozriet, ako to vyzera v praxi (co je lepsie ako testovanie na produkcnom servri?).

Co som mal na mysli ja su nasledovne 2 pointy:

  1. Border na hviezdickach, ked na ne prideme mysou, je hruby a velmi vyrazny, pokial ide o "neoznacenu" hviezdicku (cize to, co sa nemeni), ale po oznaceni je taky... Meh. Mozno to je fajn (zvyrazni to rozdiel medzi oznacenymi a neoznacenymi hviezdickami).
  2. Nie su oznacene (nove modre) hviezdicky prilis nevyrazne, pokial nad nimi nevisime mysou? Opat, stoji za pozretie, ako to vyzera v praxi a mozeme si aj pockat na feedback od pouzivatelov (aspon uvidime, aky je hodnotny :D )

Comments from Reviewable

Sameth commented 7 years ago

Reviewed 1 of 1 files at r1. Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable