Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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

Merged
phre1 merged 2 commits into master from w3
Aug 17, 2020
Merged

W3 validation errors #73

phre1 merged 2 commits into master from w3
Aug 17, 2020

Conversation

@EkaterinaVu
Copy link
Collaborator

@EkaterinaVu EkaterinaVu commented Aug 17, 2020

#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

Copy link
Contributor

phre1 commented Aug 17, 2020

@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.

Copy link
Contributor

phre1 commented Aug 17, 2020

@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.

Copy link
Contributor

phre1 commented Aug 17, 2020

@EkaterinaVu Ima i na na 350 liniji nešto prazno pa samo maknu, sorry ako zezam sa sitnicama

Copy link
Collaborator Author

EkaterinaVu commented Aug 17, 2020
edited
Loading

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.

Copy link
Contributor

phre1 commented Aug 17, 2020

@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

@phre1 phre1 merged commit 1c6d81c into master Aug 17, 2020
@spinn spinn deleted the w3 branch August 18, 2020 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /