Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [WWW] Moja pierwsza strona
Synek.
post 28.08.2012, 12:59:40
Post #1





Grupa: Zarejestrowani
Postów: 35
Pomógł: 0
Dołączył: 21.07.2012

Ostrzeżenie: (0%)
-----


Proszę o ocenę mojej pierwszej strony. Piszcie co trzeba zmienić itd.
LINK
Hosting tymczasowy.
Go to the top of the page
+Quote Post
 
Start new topic
Odpowiedzi (1 - 14)
Bateria
post 28.08.2012, 13:21:46
Post #2





Grupa: Zarejestrowani
Postów: 129
Pomógł: 13
Dołączył: 23.11.2011

Ostrzeżenie: (0%)
-----


Tak wygląda na najnowszym internet explorerze:


--------------------
Go to the top of the page
+Quote Post
Synek.
post 28.08.2012, 13:28:45
Post #3





Grupa: Zarejestrowani
Postów: 35
Pomógł: 0
Dołączył: 21.07.2012

Ostrzeżenie: (0%)
-----


Naprawione.
Go to the top of the page
+Quote Post
Evinek
post 28.08.2012, 13:32:35
Post #4





Grupa: Zarejestrowani
Postów: 280
Pomógł: 46
Dołączył: 23.03.2010

Ostrzeżenie: (0%)
-----


Prosta i moim zdaniem ładna.
Do poprawy zakładka "Realizacje" - nie czytelne. Musisz wymyślić inny układ projektów. Może obrazki same, po najechaniu jakieś dodatkowe informacje?

Formularz kontaktowy nie sprawdza poprawności emaila.
W kod HTML się nie zagłębiam.
Wszystkie podstrony w .html, a kontakt w .php.
Rób od razu wszystko w PHP.

Ocena: 7/10
Go to the top of the page
+Quote Post
Synek.
post 28.08.2012, 13:35:00
Post #5





Grupa: Zarejestrowani
Postów: 35
Pomógł: 0
Dołączył: 21.07.2012

Ostrzeżenie: (0%)
-----


Cytat(Evinek @ 28.08.2012, 14:32:35 ) *
Do poprawy zakładka "Realizacje" - nie czytelne. Musisz wymyślić inny układ projektów. Może obrazki same, po najechaniu jakieś dodatkowe informacje?

Formularz kontaktowy nie sprawdza poprawności emaila.


1. Coś pomyślę.
2. Ale nie wysyła maila gdy jest nie poprawny.


Go to the top of the page
+Quote Post
binprogrammer
post 28.08.2012, 13:37:04
Post #6





Grupa: Zarejestrowani
Postów: 249
Pomógł: 8
Dołączył: 10.08.2011
Skąd: Bełchatów

Ostrzeżenie: (10%)
X----


Cytat
"Oprócz stron wykonuję fora internetowe oparte na języku phpBB3."


Chciałbym cię poprawić, phpbb to nie język smile.gif

Cytat
Rób od razu wszystko w PHP.


Albo zamień php na html w htaccess smile.gif

Ocena 6/10 smile.gif Jestem trochę od ciebie młodszy i dawno nie zrobiłem szablonu, oj chyba trzeba się wziąć znowu za to na poważnie smile.gif

Ten post edytował binprogrammer 28.08.2012, 13:39:50
Go to the top of the page
+Quote Post
Synek.
post 28.08.2012, 13:41:32
Post #7





Grupa: Zarejestrowani
Postów: 35
Pomógł: 0
Dołączył: 21.07.2012

Ostrzeżenie: (0%)
-----


Poprawiłem.
Go to the top of the page
+Quote Post
kamil4u
post 28.08.2012, 14:28:18
Post #8





Grupa: Zarejestrowani
Postów: 2 350
Pomógł: 512
Dołączył: 4.01.2009
Skąd: Wrocław / Świdnica

Ostrzeżenie: (0%)
-----


Nie wiem czy to dobry pomysł oferować swoje usługi. Za mało wiesz. Najpierw, moim zdaniem, powinieneś się wiele nauczyć, a dopiero później na tym zarabiać. A może tylko ja już w dzisiejszych czasach mam coś takiego, że nie lubię sprzedawać ...., no coś na czym bardzo dobrze się nie znam.

Uwagi:
- Masz doctype HTML5, a w ogóle z niego nie korzystasz
- brak znacznika <html>, za to go zamykasz
- znacznik <center> - znacznik wycofano - teraz używa się CSS
-
Kod
<div id="logo">
- niepotrzebny znacznik, nic nie wnoszący
-
Kod
<div id="menu">
<a href="index.html">strona główna</a>
<a href="omnie.html">o mnie</a>
<a href="oferta.html">oferta</a>
<a href="portfolio.html">realizacje</a>
<a href="kontakt.php">kontakt</a>
</div>
Menu tworzymy na listach -> <ul>, <li>
-
Kod
<div id="prawa">
Wszystkie prawa zastrzeżone / Copyright &copy; 2012 by
<a href="index.html">Arkadiusz Chodór</a>
</div>
Użycie <div> tu nie pasuje. Osobiście użyłbym <span>
- kontakt w stopce to samo co z menu - użyj <ul>
- formularz -> nie używamy tabelek - tylko do danych tabelarycznych
-
Kod
input type="submit" name="submit" value="Wyślij" style="border-radius: 3px; border: 1px solid orange;  background-color: #000000; opacity: 0.7; color: #FFFFFF; padding: 5px 5px 5px 5px; width: 75px;">&nbsp;
CSS nie używamy w atrybucie style="..." tylko dajemy w oddzielnym pliku
-
Kod
<font size="4">
- znacznik wycofany
- i wiele innych mniejszych błędów, ale nie chce mi się przypatrywać

Oczywiście, nie piszę tego żebyś się zniechęcił - wręcz przeciwnie - chcę Ci jedynie uświadomić, że czeka Cię wiele pracy przed sobą i zachęcam Cię do tego, żebyś wiele czytał, pisał, ale hobbistycznie, a nie od razu za pieniądze.

Wyglądu nie oceniam, bo to nie moja branża. CSS i JS też na razie nie patrzyłem, ale jak będziesz chciał to mogę popatrzeć.

Kod: (4 lub 5)/10


--------------------
Go to the top of the page
+Quote Post
Synek.
post 28.08.2012, 14:46:06
Post #9





Grupa: Zarejestrowani
Postów: 35
Pomógł: 0
Dołączył: 21.07.2012

Ostrzeżenie: (0%)
-----


Cytat(kamil4u @ 28.08.2012, 15:28:18 ) *
Nie wiem czy to dobry pomysł oferować swoje usługi. Za mało wiesz. Najpierw, moim zdaniem, powinieneś się wiele nauczyć, a dopiero później na tym zarabiać. A może tylko ja już w dzisiejszych czasach mam coś takiego, że nie lubię sprzedawać ...., no coś na czym bardzo dobrze się nie znam.

Uwagi:
- Masz doctype HTML5, a w ogóle z niego nie korzystasz
- brak znacznika <html>, za to go zamykasz
- znacznik <center> - znacznik wycofano - teraz używa się CSS
-
Kod
<div id="logo">
- niepotrzebny znacznik, nic nie wnoszący
-
Kod
<div id="menu">
<a href="index.html">strona główna</a>
<a href="omnie.html">o mnie</a>
<a href="oferta.html">oferta</a>
<a href="portfolio.html">realizacje</a>
<a href="kontakt.php">kontakt</a>
</div>
Menu tworzymy na listach -> <ul>, <li>
-
Kod
<div id="prawa">
Wszystkie prawa zastrzeżone / Copyright &copy; 2012 by
<a href="index.html">Arkadiusz Chodór</a>
</div>
Użycie <div> tu nie pasuje. Osobiście użyłbym <span>
- kontakt w stopce to samo co z menu - użyj <ul>
- formularz -> nie używamy tabelek - tylko do danych tabelarycznych
-
Kod
input type="submit" name="submit" value="Wyślij" style="border-radius: 3px; border: 1px solid orange;  background-color: #000000; opacity: 0.7; color: #FFFFFF; padding: 5px 5px 5px 5px; width: 75px;">&nbsp;
CSS nie używamy w atrybucie style="..." tylko dajemy w oddzielnym pliku
-
Kod
<font size="4">
- znacznik wycofany
- i wiele innych mniejszych błędów, ale nie chce mi się przypatrywać

Oczywiście, nie piszę tego żebyś się zniechęcił - wręcz przeciwnie - chcę Ci jedynie uświadomić, że czeka Cię wiele pracy przed sobą i zachęcam Cię do tego, żebyś wiele czytał, pisał, ale hobbistycznie, a nie od razu za pieniądze.

Wyglądu nie oceniam, bo to nie moja branża. CSS i JS też na razie nie patrzyłem, ale jak będziesz chciał to mogę popatrzeć.

Kod: (4 lub 5)/10


Ja nie chcę na tym zarabiać, tylko ćwiczę. Napisałem ta stronę, w celach ćwiczenia. Aktualnie uczę się PHP, MySQL i jQuery.
Dzięki za wytknięcie błędów. Bardzo mi to pomoże.
Go to the top of the page
+Quote Post
Tuminure
post 29.08.2012, 07:12:55
Post #10





Grupa: Zarejestrowani
Postów: 178
Pomógł: 49
Dołączył: 16.04.2012
Skąd: Bytom

Ostrzeżenie: (0%)
-----


Cytat
Użycie <div> tu nie pasuje. Osobiście użyłbym <span>

Mógłbyś rozwinąć myśl? Jak dla mnie div jest dobrym rozwiązaniem - do span musiałby przypisać dodatkowo kilka atrybutów.
Inna sprawa, że skoro ma doctype html5, to mógłby zastosować <footer>, jednak div jest jak najbardziej dobrym pomysłem.

W formularzu brakuje mi znacznika label.

Ten post edytował Tuminure 29.08.2012, 07:15:47
Go to the top of the page
+Quote Post
dr_NO
post 29.08.2012, 08:45:12
Post #11





Grupa: Zarejestrowani
Postów: 197
Pomógł: 15
Dołączył: 10.09.2006
Skąd: Siemianowice Śląskie / Katowice

Ostrzeżenie: (0%)
-----


HTML5 - wystarczy użyć header aside, section i footer, 3/4 divów odpada. Dodatkowo jest tam treści co kot napłakał i każesz użytkownikom scrolować...


--------------------
woop woop.
Go to the top of the page
+Quote Post
kamil4u
post 29.08.2012, 12:55:22
Post #12





Grupa: Zarejestrowani
Postów: 2 350
Pomógł: 512
Dołączył: 4.01.2009
Skąd: Wrocław / Świdnica

Ostrzeżenie: (0%)
-----


Najlepiej użyć HTML5, ale założyłem, że skoro nie użył ani jednego znacznika z HTML5 to używa w sumie czwórkę smile.gif

Cytat
Mógłbyś rozwinąć myśl?

Moim zdaniem <div> tu nie pasuje. <div> ma grupować jakieś elementy blokowe. Tu jest tylko jedna linia tekstu, gdyby było ich więcej to nawet pokusiłbym się o użycie <p>, ale że jest jedna to zastosowanie <span>, który ma za zadanie grupować elementy liniowe, jest moim zdaniem słusznym wyborem.

Ale zobaczmy co na to W3C:
Cytat
The DIV and SPAN elements, in conjunction with the id and class attributes, offer a generic mechanism for adding structure to documents. These elements define content to be inline (SPAN) or block-level (DIV) but impose no other presentational idioms on the content. Thus, authors may use these elements in conjunction with style sheets, the lang attribute, etc., to tailor HTML to their own needs and tastes.


Jak dla mnie to właśnie mamy w tym przypadku, czyli definiujemy zawartość w "inline", czyli używamy <span>.

Autor wątku jest dość mało doświadczony, ale jeśli Ty uważasz, że się na tym dobrze znasz i używasz <div> bo masz za tym konkretne argumenty to mógłbyś użyć tego elementu, ale jak dla mnie zdecydowanie lepszy będzie tu <span>.

Teraz Ty napisz swoje argumenty za "div" i o jakich atrybutach piszesz przy <span>? Wystarczy id/class a nawet i to nie, bo można się odwołać przez #id_stopki span

Ten post edytował kamil4u 29.08.2012, 12:55:39


--------------------
Go to the top of the page
+Quote Post
dr_NO
post 29.08.2012, 13:18:42
Post #13





Grupa: Zarejestrowani
Postów: 197
Pomógł: 15
Dołączył: 10.09.2006
Skąd: Siemianowice Śląskie / Katowice

Ostrzeżenie: (0%)
-----


Zrób w sopce listę i tyle. display inline, img vertical-align middle, i lecisz.


--------------------
woop woop.
Go to the top of the page
+Quote Post
Posio
post 29.08.2012, 22:56:48
Post #14





Grupa: Zarejestrowani
Postów: 417
Pomógł: 44
Dołączył: 23.06.2011

Ostrzeżenie: (0%)
-----


Jak dla mnie jest to aż zbyt minimalistyczne (a uwielbiam minimalistyczny styl). Wszystko jest w miarę czytelne i nie dostaje się oczopląsu przeglądając stronę. Oprócz tego rzadko korzystasz z możliwości CSS. Praktycznie użyłeś go tylko do okreslenia czcionki, ustawienia odpowiednich bloków i uzyskania hoverów.

Podsumując:
-prostu szablon html (aczkolwiek jeśli się dopiero uczysz - powodzenia smile.gif )
Go to the top of the page
+Quote Post
Tuminure
post 30.08.2012, 06:39:51
Post #15





Grupa: Zarejestrowani
Postów: 178
Pomógł: 49
Dołączył: 16.04.2012
Skąd: Bytom

Ostrzeżenie: (0%)
-----


Cytat
Teraz Ty napisz swoje argumenty za "div" i o jakich atrybutach piszesz przy <span>

Muszę zwrócić honor, gdyż myślałem, że mowa jest o całej stopce, a nie tylko o jej fragmencie z tekstem (nie zwróciłem uwagi, że jest w niej jeszcze drugi div z ikonami). W takim wypadku faktycznie span jest lepszym wyborem, niż div.
Go to the top of the page
+Quote Post

Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Wersja Lo-Fi Aktualny czas: 22.06.2025 - 14:11