Ocena Portfolio

0

Siema , jest taka sprawa i prośba o ocenienie mojego świeżo powstałego portfolio i w głównej mierze kodu , gdyż chciałbym wiedzieć Co robię nie tak , i Co ewentualnie miałbym poprawić. Programowania stron webowych uczę się od Września tego roku. Do frameworków typu : bootstrap , angular , itp jeszcze się nie zabierałem. Tutaj link do mojej strony: http://www.mrjowik.ayz.pl , domena oczywiście tymczasowa przez parę dni. Cóż... forum obserwuję od dłuższego czasu więc liczę że aż takiego hejtu nie będzie :D

0

Szczerze mówiąc 2/10. Masz jakieś elementy material design, trochę elementów sprzed 10lat. W sliderze obrazek wystaje poza slider. Na listę tego co zrobiłeś nie da się patrzeć. Strona jest czarno biała, ale przy kontakcie podświetla się coś na żółto. Czemu wyróżniasz gg dając tam obrazek?

1

user image

0

user image
Lepiej jeszcze nad tym popracuj

0

sporo jeszcze pracy (i nerwów) przed tobą ;)

spróbuj pościągać sobie gotowe layouty i zobacz jak są zbudowane, jak nazwane są klasy, gdzie umieszczamy skrypty, jak robimy opisy, jak robimy przerwy, spacje itp.
od angielskiego tak czy siak nie uciekniesz, więc wdrażaj od początku język i dobre praktyki

jeżeli nie wykorzystujesz w pełni bootstrapa to wystarczy sam grid, jak strona nie przechowuje ciasteczek to info o tym jak i sam skrypt jest zbędny

resztę napisali inni, więc pomijam, powodzenia dude

0

Dzięki za krytykę ;)

0

Poza tym wyrzuć zdanie "Witam na stronie mojego portfolio !". Jest to zbędne i lepiej brzmi jak pierwszym zdaniem jest przedstawienie się.

2

Koledzy już sie na temat wyglądu wypowiedzieli, może teraz troch o kodzie:

Uwagi ogólne:

  • śmietnik w repo - pogrupuj rzeczy w foldery,
  • plik index nie zawiera pehapa, może mieć rozszerzenie .html,

HTML:
https://validator.w3.org/nu/?doc=http%3A%2F%2Fwww.mrjowik.ayz.pl%2F

PHP:

JS:

  • ładujesz na raz dwie wersje jQuery! (btw taka kobyła jak jQuery nie jest Ci w ogóle potrzebna),
  • podepnij sobie do edytora ESLint / JSLint / JSHint bo formatowanie kodu to koszmar, zajrzyj też tu1: https://github.com/airbnb/javascript/tree/es5-deprecated/es5,
  • znowu niepotrzebne globale,
  • niepotrzebne redeklaracje zmiennych,
  • magic numbers,
  • klasy zostawa dla CSSa, w JS używaj id i atrybutów data-
  • nie używaj skomplikowanych selektorów typu a[href*="#"]:not([href="#"], nadawaj elementom atrybuty data- i po nich wyszukuj,
  • nie uzywaj domyślnie operatora ==, używaj ===,
  • $(document).ready(function () {}) i $(function () {}) to to samo - z czego wynika, że masz to niepotrzebnie dwa razy,
  • animacje powinno się robić w CSS,
  • zainteresuj się ES6 bo używasz przestarzałej wersji języka (tu masz style guide1: https://github.com/airbnb/javascript),
  • ogólnie bardzo słaby, nieczytelny kod niestety.

CSS sobie daruję bo już mi się nie chce ;)

1 - są różne style guides, ale te są bardzo popularne i bardzo dobre

1 użytkowników online, w tym zalogowanych: 0, gości: 1