-
Notifications
You must be signed in to change notification settings - Fork 179
W3 validation errors #73
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@EkaterinaVu Oki, za ovaj select pogleda @spinn malo. Da, nije to trebao biti paragraf i tako složeno sve zapravo. Isprva je to bilo drugačije i predviđeno za inline elemente unutar njega, ali ako si sredila sad super.
@EkaterinaVu tu imamo konflikt jedan pa je najbolje da ti središ jer znaš. Znači trebala bi master povuću u svoj branch pa resolve i onda novi pull request napraviš. Kad si već tu onda. Daj samo common.css 344 linija, ne znam zašto je tu, ako treba biti barem stavi da je 20px margin-bottom, iako to već i overidaš s tim? Zašto si div.klasu stavila ako nigdje u .css-u nije tako? > stavljaš isto samo ako je potrebno baš, ako je potrebno nema problema. Jer što ako se html promijeni recimo. Pogotovo nemoj recimo > > ili više, to usporava jako.
@EkaterinaVu Ima i na na 350 liniji nešto prazno pa samo maknu, sorry ako zezam sa sitnicama
div.fh2 stoji jer je ponegdje ostao i p sa fh2 klasom , tamo gdje nije bilo div elemenata unutra. Taj p ima 1em marginu, jer nemamo css reset za to, pa sam stavila samo da uskladim. U principu bi možda bilo bolje da i te p-ove preuredim u div pa neće biti potrebe za div.fh2.
> stoji jer div iza q-inner nekad ima klasu a nekad nema, ovisno o tipu pitanja. Nisam se sjetila kako da ga ciljam drugačije.
Dobro, sredit ću to.
@EkaterinaVu može pogledaj pa sredi, ma svejedno samo stavi da je 20px ili ekvivalent blizu u em-ovima jer bolje tako izgleda. A ovo sam mislio gađaj klasu a ne div.klasu jer jer sve ostalo tako nigdje se ne gađa preko diva pa tek toliko da i ostane
#71
Ovo je moj prijedlog za rješavanje W3 validation errora. Znači, def ne smiju ići div u span, div u p itd. Div je nekako najsigurniji za vanjske elemente, prilagodila sam i CSS.
E, sad, za error koji javlja za option - tu sam dušu ispustila. Ne mogu staviti size na select jer ne funkcionira dobro ovako kako je kod složeno, niti ništa drugo, jedino taj prvi option element sa empty value funkcionira nekako. Idealno bi bilo da se može sakriti, ili smanjiti , ali ne mogu se nikako riješiti visine na tom elementu