2 strony początkującego web-designera

2 strony początkującego web-designera
  • Rejestracja: dni
  • Ostatnio: dni
0

Witam,
chciałbym wam przedstawić moje dwie strony (a w sumie jedną kompletną i drugiej jedynie stronę główną). Zależy mi na ocenie kodu, biorąc pod uwagę, że to moje pierwsze projekty :) Oraz powiedzenie co jest nie tak. Dziękuje za wszelkie porady, jak i za konstruktywną krytykę.

http://algogeek.cba.pl/
http://algogeek.cba.pl/algogeek-test/index.html

PS. Niestety, ale stronki na darmowym hostingu, stąd reklamy i możliwe rozjechania. Niestety niezbyt wiem jak to zlikwidować :/

LI
  • Rejestracja: dni
  • Ostatnio: dni
  • Postów: 518
0

Postaw na openshift - tam nie ma reklam.

Typografia nie do końca mi jakoś pasuje.

W placeholderach masz nierówny tekst.

Toolbar by chyba lepiej wyglądał z samym kolorem, zamiast tej grafiki.

To moje prywatne odczucia ;) Ogólnie nie jest źle.

  • Rejestracja: dni
  • Ostatnio: dni
0

Jutro postaram się zmienić hosting :)
Co do typografii i ogólnie wyglądu - tym zajmował się grafik, przekaże mu Twoje uwagi. Zaraz zobaczę na placeholdery. Mi najbardziej zależy na opinii dotyczącej kodu - ewentualne błędy itp.

Ale dziękuję za Twoją opinię :)

Spine
  • Rejestracja: dni
  • Ostatnio: dni
  • Postów: 7103
0
Kopiuj
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">

Już szkoda pisać w XHTML, pisz współcześnie:

Kopiuj
<!DOCTYPE html>

Do pisania wzorów/formułek mógłbyś się pokusić o podpięcie LaTeX'u, zamiast wstawiać obrazek.
http://algogeek.cba.pl/algogeek-test/ciag_fibonacciego.html

Design całkiem średni, jak nie kiepski. Biały tekst na jasnoszarym tle jest nieczytelny. Tabelki (np. tabela ligowa) powinna mieć jakiś minimalny padding, żeby cyferki nie zlewały się z obramowaniem.

mr_jaro
  • Rejestracja: dni
  • Ostatnio: dni
  • Lokalizacja: Grudziądz/Bydgoszcz
  • Postów: 5300
0

w drugim linku masz niedomknięte tagi i się sypie lekko.

kasiaKasia
  • Rejestracja: dni
  • Ostatnio: dni
  • Postów: 259
1

mała wskazówka:
Strona nie jest responsywna, inaczej nazywana "mobile first". Czyli strona zbudowana z metod i klas wyświetla się na każdej wielkości ekranu estetycznie , elementy skalują się

Przykładowa biblioteka:
http://getbootstrap.com/

Przykłady :
http://startbootstrap.com/template-categories/all/

mały przykład do zdjęć na których masz tekst:

Kopiuj
<div class="container" >
    <div class="row">
        <div class="col-lg-6">
            <div class="panel">
                <div class="panel-heading">
                    <div class="row">

                        <div class="text-right">
                                    tekst
                        </div>
                    </div>

                </div>

            </div>
        </div>
        <div class="col-lg-6">
                    tekst
        </div>   

    </div>
</div>
    • ** ładnie nazywasz klasy w div
Patryk27
  • Rejestracja: dni
  • Ostatnio: dni
  • Lokalizacja: Wrocław
  • Postów: 13042
0

Brak RWD skreśla tę stronę na samym początku :P

Freerider
  • Rejestracja: dni
  • Ostatnio: dni
  • Lokalizacja: Kraków
  • Postów: 67
0
  • nie pasuje mi typografia
  • stopka nieczytelna
  • biała czcionka na szarym tle (prawy panel) - dla mnie mało czytelna, tabela przez bold również
  • lewy panel, a dokładniej te szare przyciemnienia są wg mnie za wielkie w stosunku do wielkości całego zdjęcia, tekst ładny
  • sprawdziłbym jak wyglądała by nawigacja na dole przy użyciu << >> , itp. zamiast tylu liczb
  • nieczytelna stopka
  • miejscami nieczytelne zielone napisy
  • lewy panel bym przeskalował, dużo miejsca a tekst procentowo mało zajmuje, a przeglądam na 1366x768 i dlatego mam takie mniejsze

Zarejestruj się i dołącz do największej społeczności programistów w Polsce.

Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.