Code review pierwszej responsywnej strony

Code review pierwszej responsywnej strony
Mateuszto
  • Rejestracja:prawie 7 lat
  • Ostatnio:4 miesiące
  • Lokalizacja:Poznań
  • Postów:163
0

https://github.com/Mateuszto/Pierwsza-strona-responsywna-2018
Jeżeli ktoś miałby czas na zrobienie code review mojej pierwszej responsywnej strony byłbym bardzo wdzięczny.

TU
  • Rejestracja:prawie 8 lat
  • Ostatnio:9 miesięcy
  • Lokalizacja:Wrocław
  • Postów:56
2

Nazewnictwo klas i ID do poprawy. Najlepiej klasy pisać po angielsku, zgodnie z ktorakolwiek konwencją nazewniczą, np. Block Element Modifier. Żeby to miało jakiś ład i płaską strukturę.
Unikalbym też nadmiernego stylowania po selektorach i ID.
Reszta wydaje się ok.

edytowany 2x, ostatnio: tubaris
Mateuszto
Dzięki :) No, rzeczywiście z nazwami jest trochę ciężko. Nawet z ich wymyślaniem hah. Mam nadzieję, że się w tym poprawie, bo jakby projekt był większy to trudno się w tym póżniej połapać :D. Jeszcze raz dzięki.
TU
Też miałem z tym problem na początku, myślałem kilka minut jaką wymyślić nazwę dla klasy. Ale z czasem zobaczysz, że będzie Ci to przychodzić coraz łatwiej. Ja czytałem różne tutoriale na ten temat. Podrzucam 2 arty które mi najlepiej pomogły. http://javascript-html5-tutorial.pl/styl-konwencje-i-metodologie-css-czesc-2-sass-bem-rscss-smacss-wtf.html https://seesparkbox.com/foundry/bem_by_example
cerrato
Moderator Kariera
  • Rejestracja:około 7 lat
  • Ostatnio:około 10 godzin
  • Lokalizacja:Poznań
  • Postów:8769
1

To RWD nie do końca działa tak, jak powinno - pomniejszając okno, trzecie kółko całkowicie znika, a dopiero po chwili następuje przełączenie na widok pionowy. Powinno się przełączyć już w chwili, gdy skrajne kółko mocno zbliży się do krawędzi okna. Obecne zachowanie jest niedopuszczalne.

Ewentualnie mógłbyś dodać jeszcze z jakiś 1 czy 2 breakpointy, które podczas skalowania okna będą zbliżać kółka do siebie, albo w ogóle je od siebie odsunąć nie na sztywno, ale w jakiś płynny/procentowy sposób. Ale poza tym nie widzę większych problemów, strona raczej OK, zwłaszcza, jak na pierwszą - po kilku kolejnych nabędziesz biegłości :)

.

screenshot-20180709093544.png


edytowany 1x, ostatnio: cerrato
czysteskarpety
czysteskarpety
niekoniecznie, przecież użytkownik nie zmniejsza i powiększa okna dla zabawy tylko ogląda w danej rozdzielczości, więc jeśli te bazowe są ok, to nie widze większego problemu
cerrato
Na komórce - OK. Ale na desktopie może mieć dowolny rozmiar, także "nietypowy". I nawet nie bawiąc się w zmienianie - może mieć przeglądarkę odpaloną w rozmiarze z mojego screena lub podobnym, w związku z czym straci część treści. Pomijając kwestię estetyki, może w ten sposób coś przeoczyć i nawet nie mieć świadomości tego.
czysteskarpety
czysteskarpety
  • Rejestracja:prawie 10 lat
  • Ostatnio:ponad 4 lata
  • Lokalizacja:Piwnica
  • Postów:7697
0

prócz powyższego, mała podpowiedź, odtwórz źródło strony w firefox i tam też często pokazuje co do poprawki
ponadto możesz też sprawdzić walidatorem: https://validator.w3.org/nu/?doc=https%3A%2F%2Fmateuszto.github.io%2FPierwsza-strona-responsywna-2018%2F

screenshot-20180709110238.png


adams0
  • Rejestracja:ponad 7 lat
  • Ostatnio:5 dni
  • Postów:308
1

Możesz dodać swój telefon w postaci linku a nie surowego tekstu. Będzie łatwiej dzwonić skajpajem :-)

Kliknij, aby dodać treść...

Pomoc 1.18.8

Typografia

Edytor obsługuje składnie Markdown, w której pojedynczy akcent *kursywa* oraz _kursywa_ to pochylenie. Z kolei podwójny akcent **pogrubienie** oraz __pogrubienie__ to pogrubienie. Dodanie znaczników ~~strike~~ to przekreślenie.

Możesz dodać formatowanie komendami , , oraz .

Ponieważ dekoracja podkreślenia jest przeznaczona na linki, markdown nie zawiera specjalnej składni dla podkreślenia. Dlatego by dodać podkreślenie, użyj <u>underline</u>.

Komendy formatujące reagują na skróty klawiszowe: Ctrl+B, Ctrl+I, Ctrl+U oraz Ctrl+S.

Linki

By dodać link w edytorze użyj komendy lub użyj składni [title](link). URL umieszczony w linku lub nawet URL umieszczony bezpośrednio w tekście będzie aktywny i klikalny.

Jeżeli chcesz, możesz samodzielnie dodać link: <a href="link">title</a>.

Wewnętrzne odnośniki

Możesz umieścić odnośnik do wewnętrznej podstrony, używając następującej składni: [[Delphi/Kompendium]] lub [[Delphi/Kompendium|kliknij, aby przejść do kompendium]]. Odnośniki mogą prowadzić do Forum 4programmers.net lub np. do Kompendium.

Wspomnienia użytkowników

By wspomnieć użytkownika forum, wpisz w formularzu znak @. Zobaczysz okienko samouzupełniające nazwy użytkowników. Samouzupełnienie dobierze odpowiedni format wspomnienia, zależnie od tego czy w nazwie użytkownika znajduje się spacja.

Znaczniki HTML

Dozwolone jest używanie niektórych znaczników HTML: <a>, <b>, <i>, <kbd>, <del>, <strong>, <dfn>, <pre>, <blockquote>, <hr/>, <sub>, <sup> oraz <img/>.

Skróty klawiszowe

Dodaj kombinację klawiszy komendą notacji klawiszy lub skrótem klawiszowym Alt+K.

Reprezentuj kombinacje klawiszowe używając taga <kbd>. Oddziel od siebie klawisze znakiem plus, np <kbd>Alt+Tab</kbd>.

Indeks górny oraz dolny

Przykład: wpisując H<sub>2</sub>O i m<sup>2</sup> otrzymasz: H2O i m2.

Składnia Tex

By precyzyjnie wyrazić działanie matematyczne, użyj składni Tex.

<tex>arcctg(x) = argtan(\frac{1}{x}) = arcsin(\frac{1}{\sqrt{1+x^2}})</tex>

Kod źródłowy

Krótkie fragmenty kodu

Wszelkie jednolinijkowe instrukcje języka programowania powinny być zawarte pomiędzy obróconymi apostrofami: `kod instrukcji` lub ``console.log(`string`);``.

Kod wielolinijkowy

Dodaj fragment kodu komendą . Fragmenty kodu zajmujące całą lub więcej linijek powinny być umieszczone w wielolinijkowym fragmencie kodu. Znaczniki ``` lub ~~~ umożliwiają kolorowanie różnych języków programowania. Możemy nadać nazwę języka programowania używając auto-uzupełnienia, kod został pokolorowany używając konkretnych ustawień kolorowania składni:

```javascript
document.write('Hello World');
```

Możesz zaznaczyć również już wklejony kod w edytorze, i użyć komendy  by zamienić go w kod. Użyj kombinacji Ctrl+`, by dodać fragment kodu bez oznaczników języka.

Tabelki

Dodaj przykładową tabelkę używając komendy . Przykładowa tabelka składa się z dwóch kolumn, nagłówka i jednego wiersza.

Wygeneruj tabelkę na podstawie szablonu. Oddziel komórki separatorem ; lub |, a następnie zaznacz szablonu.

nazwisko;dziedzina;odkrycie
Pitagoras;mathematics;Pythagorean Theorem
Albert Einstein;physics;General Relativity
Marie Curie, Pierre Curie;chemistry;Radium, Polonium

Użyj komendy by zamienić zaznaczony szablon na tabelkę Markdown.

Lista uporządkowana i nieuporządkowana

Możliwe jest tworzenie listy numerowanych oraz wypunktowanych. Wystarczy, że pierwszym znakiem linii będzie * lub - dla listy nieuporządkowanej oraz 1. dla listy uporządkowanej.

Użyj komendy by dodać listę uporządkowaną.

1. Lista numerowana
2. Lista numerowana

Użyj komendy by dodać listę nieuporządkowaną.

* Lista wypunktowana
* Lista wypunktowana
** Lista wypunktowana (drugi poziom)

Składnia Markdown

Edytor obsługuje składnię Markdown, która składa się ze znaków specjalnych. Dostępne komendy, jak formatowanie , dodanie tabelki lub fragmentu kodu są w pewnym sensie świadome otaczającej jej składni, i postarają się unikać uszkodzenia jej.

Dla przykładu, używając tylko dostępnych komend, nie możemy dodać formatowania pogrubienia do kodu wielolinijkowego, albo dodać listy do tabelki - mogłoby to doprowadzić do uszkodzenia składni.

W pewnych odosobnionych przypadkach brak nowej linii przed elementami markdown również mógłby uszkodzić składnie, dlatego edytor dodaje brakujące nowe linie. Dla przykładu, dodanie formatowania pochylenia zaraz po tabelce, mogłoby zostać błędne zinterpretowane, więc edytor doda oddzielającą nową linię pomiędzy tabelką, a pochyleniem.

Skróty klawiszowe

Skróty formatujące, kiedy w edytorze znajduje się pojedynczy kursor, wstawiają sformatowany tekst przykładowy. Jeśli w edytorze znajduje się zaznaczenie (słowo, linijka, paragraf), wtedy zaznaczenie zostaje sformatowane.

  • Ctrl+B - dodaj pogrubienie lub pogrub zaznaczenie
  • Ctrl+I - dodaj pochylenie lub pochyl zaznaczenie
  • Ctrl+U - dodaj podkreślenie lub podkreśl zaznaczenie
  • Ctrl+S - dodaj przekreślenie lub przekreśl zaznaczenie

Notacja Klawiszy

  • Alt+K - dodaj notację klawiszy

Fragment kodu bez oznacznika

  • Alt+C - dodaj pusty fragment kodu

Skróty operujące na kodzie i linijkach:

  • Alt+L - zaznaczenie całej linii
  • Alt+, Alt+ - przeniesienie linijki w której znajduje się kursor w górę/dół.
  • Tab/⌘+] - dodaj wcięcie (wcięcie w prawo)
  • Shit+Tab/⌘+[ - usunięcie wcięcia (wycięcie w lewo)

Dodawanie postów:

  • Ctrl+Enter - dodaj post
  • ⌘+Enter - dodaj post (MacOS)