Jak wyszło? + kilka pytań.

pytanie zadane 13 lipca 2017 w Nasze projekty przez Hiskiel Pasjonat (22,830 p.)

Cześć! Napisałem ostatnio z pomocą poradników pana Mirosława stronę (w kilku wersjach). Chciałbym, żebyście ocenili jak mi wyszło i może podali lepsze rozwiązanie kodu źródłowego. Chciałbym też prosić o pomoc:

http://imgur.com/a/nFwW0 - tutaj macie screena jak wygląda takie "menu"

I mam pyt. Co zrobić, żeby te napisy np. 19+ były dosłownie na środku tego diva? bo wygląda to taq:

div links a w nim 5 divow ads (nwm czmu taq, taq sb) i chodzi o, to, żeby ten napis był dosłownie w środku.

Proszę o szybką pomoc. Dziękuję

~0zi

A tutaj link do pobrania paczki zip z tymi stronami (nie zwracajcie uwagi na nazwy itd. bo to mi se taq przyszło. Ja np. nie lubie tych seriali xD) :

http://www.mediafire.com/file/5xdcrttb7rpq9o3/Ta_strona_xD.rar

3 odpowiedzi

odpowiedź 13 lipca 2017 przez Comandeer Guru (601,110 p.)

https://github.com/CodersCommunity/CodeReview-HTML-02

komentarz 13 lipca 2017 przez Hiskiel Pasjonat (22,830 p.)

Co mi po tym?

Prosiłem o pomoc i ocenę...

komentarz 13 lipca 2017 przez Comandeer Guru (601,110 p.)

No to jest ocena kodu, który posłałeś.

Co do problemu: text-align: center dla .links powinno starczyć.

komentarz 13 lipca 2017 przez Comandeer Guru (601,110 p.)

Heh, źle popatrzyłem. Co do problemu, to możesz określić np. line-height linków na tę samą wysokość, co wysokość linku.

komentarz 13 lipca 2017 przez Hiskiel Pasjonat (22,830 p.)

1. Nie widzę w twoim poprzednim poście żadnej oceny, widzę tylko link do zobaczenia materiałów z kursu pana Mirosława :V

2. To nie wystarczy. nie chodzi o wycentrowanie tekstu, tylko o to żeby tekst był na samym środku nie tak (cała zawartość będzie w obrazku)

http://imgur.com/a/rrvSQ

~0zi

komentarz 13 lipca 2017 przez Hiskiel Pasjonat (22,830 p.)

Nie rozumiem? Mam ustawić line-height linków na tę samą wysokość co wysokość linków?

komentarz 13 lipca 2017 przez MałyAleWariat Bywalec (2,830 p.)

text-align: center
To jest druga opcja z linka: http://imgur.com/a/rrvSQ
A trzecią osiągniesz ustalając line-height: 1 lub 1.5 lub 2 itd.
ps. Comandeer to Pan Mirosław ;d

komentarz 13 lipca 2017 przez Hiskiel Pasjonat (22,830 p.)

Dobrze, przepraszam, udało się! Line-height mi pomógł! Tutaj bardzo Ci dziękuję! Jednakże dalej zależy mi na pisemnej opinii na temat tego co napisałem. Z góry dziękuję!

~0zi

komentarz 13 lipca 2017 przez Hiskiel Pasjonat (22,830 p.)

Thx, wgl COOOOOOOOOO????????????? Sam Pan Mirosław Zelent udzielał mi odpowiedzi? Jak?!?!?!?!?!?!? Ja nie wieże!!! Jakim cudem? Co? Jestem zaszczycony!

~0zi

komentarz 13 lipca 2017 przez Comandeer Guru (601,110 p.)

ps. Comandeer to Pan Mirosław ;d

WTF? Nie jestem panem Zelentem… Skąd ten skrajnie idiotyczny pomysł?

Nie widzę w twoim poprzednim poście żadnej oceny, widzę tylko link do zobaczenia materiałów z kursu pana Mirosława :V

No napisano tam, że masz przełączyć na inny branch, np. https://github.com/CodersCommunity/CodeReview-HTML-02/tree/refactor czy https://github.com/CodersCommunity/CodeReview-HTML-02/tree/html5

komentarz 13 lipca 2017 przez MałyAleWariat Bywalec (2,830 p.)

Sugerowałem się rankingiem :D Ale jak tak to wybacz ;d

komentarz 13 lipca 2017 przez MałyAleWariat Bywalec (2,830 p.)

Ale śmiać się z siebie do tej pory przestać nie mogę :D

odpowiedź 13 lipca 2017 przez xandros Nałogowiec (29,450 p.)

Czyli tylko ja bym użył:

.ads {
	justify-content: center;
	align-items: center;
	display: flex;
}

:D

Łap moją poprawke: https://jsfiddle.net/nf3s40ue/3/

edif: a raczej nowy kod...

komentarz 13 lipca 2017 przez Comandeer Guru (601,110 p.)

Ale menu to mogłeś poprawić :P

komentarz 13 lipca 2017 przez xandros Nałogowiec (29,450 p.)

My bad :P

https://jsfiddle.net/nf3s40ue/5/

(zmienione na liste :U)

odpowiedź 13 lipca 2017 przez EdeX Mądrala (5,110 p.)

Widze że się dopiero uczysz html i css to radze ci się pouczyć to daje: box-sizing:border-box;

zastosować to i dodać tylko: padding: 10px;

to:

padding: 10px;
box-sizing:border-box;

dodaj do a.cl i się naprawi

komentarz 13 lipca 2017 przez EdeX Mądrala (5,110 p.)

to jest to drugiego projektu

komentarz 14 lipca 2017 przez Hiskiel Pasjonat (22,830 p.)

Okay! Bardzo dziękuję! Każda nowa indtrukcja, nowy tag wzbogaca moją wiedzę :D. Jest mi niezmiernie miło, że ktoś bardziej doświadczony postanawia mnie czegoś uczyć.

~0zi