Change logo in header - Githubissues

code4romania / rezultate-vot-client

https://rezultatevot.ro

Mozilla Public License 2.0

5 stars 12 forks source link

Change logo in header #26

Closed overeha closed 3 years ago

overeha commented 3 years ago

Change the header to look like this:

Here are the two logos: logo-_rezultate vot

logo code4 transparency

dapetcu21 commented 3 years ago

@overeha Te rog, dacă ai logo-ul în format vector îl prefer așa. Also, pe mobil e foarte tight spațiul, că nu avem hamburger menu (mai bine, tbh) și aș comprima logo-ul să fie minimal (hide la logo-ul code4ro, poate hide și la textul de "REZULTATE VOT").

overeha commented 3 years ago

logo-_rezultate vot.zip

Nu, pe mobil facem resize, nu dam hide la nimic.

overeha commented 3 years ago

Code4logo_transp.zip

dapetcu21 commented 3 years ago

Nu, pe mobil facem resize, nu dam hide la nimic.

Unde facem resize? Chiar nu e loc. O să fie foarte mici. Și nu am când să fac un hamburger menu.

dapetcu21 commented 3 years ago

Eu scot butonul de donează de pe mobil dacă vreți ca să fie loc de ambele logo-uri, dar cred că ăla e prioritar.

overeha commented 3 years ago

Hai sa le punem pe doua randuri

row 1 - logos row2 - the menu

dapetcu21 commented 3 years ago

Nu pot să schimb height-ul top bar-ului să se schimbe dinamic pe mobil fără un refactor la CSS de care nu e timp. Deci ori mărim headerul și pe mobil și pe desktop, ori nu.

Dar problema mare e că ar arăta așa: E 40% din spațiul vertical (și poate deveni și mai mare acel header secundar în functie de cât de lung e titlul alegerii) folosit pentru headere și asta e even without Safari's window chrome. Rămâne 60% din spațiu pentru conținut. E foarte foarte rău pentru UX. Haide să nu compromitem UX-ul pentru niște logo-uri.