Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
Reply to this topicStart new topic
> [www] Prosty projekt strony
MissMoneypenny
post
Post #1





Grupa: Zarejestrowani
Postów: 37
Pomógł: 0
Dołączył: 11.06.2005

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


To jest dopiero mój drugi że tak powiem "projekt", więc nie oczekujcie jakichś cudów. Chodziło mi głównie o przejrzystość i wykorzystanie css-a do formatowania wyglądu(najlepiej się uczę css-a w praktyce (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) ). Proszę o konstruktywną ocenę w skali od 1 do 10. Sprawdzałam w Firefoxie, Operze i IE-nic sie nie rozjeżdżało (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) .

Oto adres: Strona

Ten post edytował MissMoneypenny 26.01.2006, 21:02:10
Go to the top of the page
+Quote Post
tiraeth
post
Post #2





Grupa: Przyjaciele php.pl
Postów: 1 789
Pomógł: 41
Dołączył: 30.10.2003
Skąd: Wrocław

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


Cytat
This Page Is Valid XHTML 1.0 Transitional!

No to już jakiś plus (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Graficznie top mi się nie podoba, kolory troche mało współgrające...

5/10 (+1 za Valid XHTML)
Go to the top of the page
+Quote Post
Zajec
post
Post #3





Grupa: Zarejestrowani
Postów: 1 086
Pomógł: 8
Dołączył: 10.12.2003

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


Całkiem dobrze. Widzę jedna kilka dziwnie wyglądających znaczników.

Pierwszy to pusty tag
Kod
<div class="tresc_foot"> </div>
który z tego co widzę, to robi tylko odstęp... Jeśli chcesz mieć newsy w odpowiedniej odległości to nadaj po prostu odpowiedni margines dla klasy .tresc

Druga sprawa: lekki tag soup w postaci
Kod
<div class="tresc_main">
<p class="tresc">
(...)
</p></div>
nie dało się tego jako jednego znacznika zrobić?

Oczywiście to już jest czepianie się, wymaganie kompletnej dokładności :-) Kod ogólnie mi pasuje.


P.S.
Prawie wszystkim <li> nadajesz klasę .li - czy nie łatwiej byłoby właściwości tej klasy przypisać domyślnie tagowi li, a jedynie te, które mają wyglądac inaczej - oznaczać dodatową klasą?
Go to the top of the page
+Quote Post
hwao
post
Post #4


Developer


Grupa: Moderatorzy
Postów: 2 844
Pomógł: 20
Dołączył: 25.11.2003
Skąd: Olkusz




masz tam cos wstawione na chama nie powinno tka byc (jak daje ignoruj css to ma grafike)

kod wyglada prawie ok
Go to the top of the page
+Quote Post
MissMoneypenny
post
Post #5





Grupa: Zarejestrowani
Postów: 37
Pomógł: 0
Dołączył: 11.06.2005

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


Wiem, te puste znaczniki miałam poprawić i zapomniałam, bo wcześniej wpisywałam sobie przykładowe treści (zanim trochę uporzadkowałam kod (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) .
@Zajec wezmę sobie twoje uwagi do serca odnośnie tych tagów li, rzeczywiscie trochę nie przemyślałam tego.
Go to the top of the page
+Quote Post
pawel_k
post
Post #6





Grupa: Zarejestrowani
Postów: 211
Pomógł: 3
Dołączył: 29.07.2005
Skąd: Szczebrzeszyn

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


mi sie jedna rzecz nie podoba:

div#all
{
width: 700px;
margin:auto;
background:url(yellow.jpg);
}


nie lepiej okreslic kolor dla div? (yellow.jpg to "obrazek" o wymiarach 1px na 1px)
Go to the top of the page
+Quote Post
MissMoneypenny
post
Post #7





Grupa: Zarejestrowani
Postów: 37
Pomógł: 0
Dołączył: 11.06.2005

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


Cytat
nie lepiej okreslic kolor dla div? (yellow.jpg to "obrazek" o wymiarach 1px na 1px)


Zastosowałam tak, ponieważ w palecie kolorów dostępnych dla web-a niestety nie było odpowiedniego koloru, gdyby takowy był, to oczywiście zrobiłabym tak, jak mówisz (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) .
Go to the top of the page
+Quote Post
hwao
post
Post #8


Developer


Grupa: Moderatorzy
Postów: 2 844
Pomógł: 20
Dołączył: 25.11.2003
Skąd: Olkusz




Ale poza "slownym" masz jeszcze pare innych (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Podpowiem

Szesnastkowy: #D9D095
RGB: rgb(217, 208, 149) LUB rgb(85%, 82%, 58%)
Go to the top of the page
+Quote Post
MissMoneypenny
post
Post #9





Grupa: Zarejestrowani
Postów: 37
Pomógł: 0
Dołączył: 11.06.2005

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


Dzięki hwao! Tak to bym żyła w niewiedzy (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) . Poprawiam te obrazki o których mówiłeś, jutro poprawie resztę kodu zdognie z waszymi sugestiami. Jeszcze raz dzięki za wszystkie sugestie.
Go to the top of the page
+Quote Post
ikioloak
post
Post #10





Grupa: Zarejestrowani
Postów: 416
Pomógł: 0
Dołączył: 8.01.2004

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


Mi sie za to bardzo podoba. Kolory wg mnie pasuja do siebie, wszystko czytelne. W kod sie nie wglebialem ale na 1 rzut oka tez prezentuje sie bardzo przejzyscie.
Nie podobaja mi sie 2 rzeczy:
- w topie napis 'Miss Moneypenny's workshop' jest za maly.
- biale tlo.
Biorac pod uwage ze strona ta ma byc homepage lub blog daje 7/10
Go to the top of the page
+Quote Post
SirK
post
Post #11





Grupa: Zarejestrowani
Postów: 14
Pomógł: 0
Dołączył: 27.05.2005

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


Jak dla mnie bardzo ok, kod niemal idealny (tego moglbym sie od Ciebie uczyc (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) , kolorki moim zdaniem dopasowane elegancko, tylko tak jak juz to bylo wspomniane tytul w top'ie dalbym wiekszy, zeby sie rzucal w oczy.
Fajny projekt.
Go to the top of the page
+Quote Post
vieri_pl
post
Post #12





Grupa: Zarejestrowani
Postów: 406
Pomógł: 9
Dołączył: 24.07.2005
Skąd: Bydgoszcz

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


Nie w moim stylu, ale ok. Kod jak już poprzednicy pisali na prawdę ok, kolory hmmm tu mam więcej zastrzeżeń, ale to chyba dlatego że nie lubię tego typu stron... (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

Ogólnie bardzo ładnie wykonane (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Pozdrawiam
Go to the top of the page
+Quote Post
hwao
post
Post #13


Developer


Grupa: Moderatorzy
Postów: 2 844
Pomógł: 20
Dołączył: 25.11.2003
Skąd: Olkusz




nadal masz fragmenty "nie zalezne od css" (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) a idea xhtml'a polega na tym ze zmieniajac css mozesz zmienic calkowicie wyglad strony.

  1. <img id="note" src="1.jpg" alt="kartka">


Powinnas zastowic

  1. <div class="imgBlock1"></div>


Teraz podpinasz do tego blocku css'a odpowiedniego (width heigh i backgrounda: url( './1.jpg' ); ) i masz ok (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) (zrob tak ze wszytkimi poza reklama)

Troche duzo Div'ów masz (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) a moglas ladnie uzyc innych znacznikow.
Go to the top of the page
+Quote Post
MissMoneypenny
post
Post #14





Grupa: Zarejestrowani
Postów: 37
Pomógł: 0
Dołączył: 11.06.2005

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


Poprawiłam kod, usunęłam zbędne znaczniki, obrazki. Teraz powinno być wszystko jak trzeba. Hwao, jeszcze raz dzięki za wskazówki (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) .
Go to the top of the page
+Quote Post
Zajec
post
Post #15





Grupa: Zarejestrowani
Postów: 1 086
Pomógł: 8
Dołączył: 10.12.2003

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


Cały czas mnie męczy sposób, w jaki zrobiłaś to menu. Połowa znaczników jest tam nadal zbędna. Zrobiłem Ci menu o takiej samej funkcjonalności, a dwa razy krótszym kodzie:

Kod
#menu1 { width: 160px; float: right; list-style: none; text-align: center; background-color: #CC9; font-style: italic; color:black; }
#menu1 * { height: 30px; }
#menu1 a { background: #9CC; display: block; margin: 5px 0; height: 30px; color: #5A5A5A; text-decoration: none; border-top:1px dotted gray; border-bottom:1px dotted gray; font: normal 10px/18px "Lucida Grande", "Lucida Sans Unicode", verdana, lucida, sans-serif; }
#menu1 a:hover { background: #A0D8D9; color: #6C0088; }




Natomiast zamiast obecnego
  1. <div id="menu1">
  2.  
  3. <div id="menu2">
  4.  
  5. <div class="linki">
  6.  
  7. <p class="category">Kategoria</p>
  8.  
  9. <ul class="ul">
  10.  
  11. <li><a href="index.html#">link 1</a></li>
  12.  
  13. <li><a href="index.html#">link 1</a></li>
  14.  
  15. </ul>
  16. (....)
  17. (....)
  18. (....)
  19. </div>
  20. </div>
dajesz tylko takie coś:
  1. <ul id="menu1">
  2. <li>Kategoria</li>
  3. <li><a href="index.html#">link 1</a></li>
  4. <li><a href="index.html#">link 1</a></li>
  5. <li>Kategoria</li>
  6. <li><a href="index.html#">link 1</a></li>
  7. <li><a href="index.html#">link 1</a></li>
  8. <li>Kategoria</li>
  9. <li><a href="index.html#">link 1</a></li>
  10. <li><a href="index.html#">link 1</a></li>
  11. </ul>


Ten post edytował Zajec 27.01.2006, 19:35:25
Go to the top of the page
+Quote Post
revyag
post
Post #16





Grupa: Przyjaciele php.pl
Postów: 2 258
Pomógł: 16
Dołączył: 21.09.2004
Skąd: Kielce

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


Layout fajny, chociaż kolory dałbym ciut bardziej nasycone.
Jak go zobaczyłem od razu z czymś mi się skojarzył --> mtv5 i taki program pimp my ride (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Ale to chyba przez tą czcionkę (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
Go to the top of the page
+Quote Post
strife
post
Post #17





Grupa: Przyjaciele php.pl
Postów: 2 605
Pomógł: 96
Dołączył: 22.10.2004
Skąd: UK

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


Witam,

+ Kobieta dała do oceny projekt (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)
+ Fajny klimatowy top, kojarzy mi się z latami '70

- Nazwa autora wpisu nie dochodzi do końca, czyli do góry.

Walidacji nie sprawdzałem, ponieważ zrobili to moi poprzednicy. Ogólnie bardzo ładnie.

Pozdrwiam!
Go to the top of the page
+Quote Post

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

 



RSS Aktualny czas: 7.10.2025 - 07:06