קידוד שעשיתי - ביקורת

[ofekns]

שלום, בעבר ההיתי מקודד (וגם קצת מתכנת) ועוד כמה דברים...
המון זמן לא נגעתי בזה....
כרגע אני חייל.... אני רוצה בקרוב לפתוח עסק שבו יוצבו מספר שירותים, אחד מהם הוא קידוד.
לכן אני רוצה לקודד בצורה הטובה ביותר שאני יכול. אז ככה...

לקחתי עיצוב (מאתר חינם של PSDים) וקידדתי את העיצוב (כמובן 100% ידני בלי עזרה מקודים שהיו מצואים באתר)
אני ישמח לשמוע תגובות על כל נושא שקשור לקידוד.... מבנה קוד (סדר בקוד)... הכל!...

נ.ב בגלל שאני כרגע מתכון לבנות תיק עבודות אני עבודות במחירי רצפה/חינם (תלוי בעיצוב).
אז תודה רבה למי שמגיב....

קישור לתמונה של העיצוב:
http://termoed.co.il/ofek/golf_site/golfsite.jpg

קישור לקידוד:

http://termoed.co.il/ofek/golf_site/

[raNge]

קידוד נחמד ביותר :]
בהצלחה

[Labs]

<span>|</span> - מיותר
<div id="logo"></div> אם אתה לא הולך לשים עליו לינק... - מיותר

אישית אני מעדיף להשתמש בCLASS ולשמור את הID לJS... אבל אפשר גם ככה

לא מותאם לIE6

אני רואה בCSS שרציתה כנראה לעשות את התפריט כרשימה וויתרתה וזה חבל...

בכל מקרה יש עוד כמה בעיות דברים מיותרים וכמה בעיות תמשיך לקודד
והסוד הגדול לפחות בישבלי הוא תמשיך לנסות עד שתצליח אתה רוצה לעשות משהו ככה תעשה ככה עד שזה יראה טוב ויפה בכל הדפדפנים וזה לא בושה להעזר בגוגול
תריץ FLOAT IE6 אני בטוח שתיהיה מבסוט

[ofekns]

raNge - תודה =]...

Labs -

<span>|</span> - מיותר

מה אתה מציע במקום?

ובקשר ל:

<div id="logo"></div>

אז מה אתה מציע במקום? בdiv שמעליו כבר יש תמונה רקע... (בגלל זה עשיתי את זה כך)

לא מותאם ל IE6 - צודק... שחכתי להתאים.... (אני יודע להתאים באופן מלא.. הרבה זמן של חוסר תעסוקה בקידוד גרמו לי לשכוח.. )

בקשר לCSS צודק ... אני תמיד ההיתי עושה תפריט ברשימה.. חבל שויתרתי על זה...טוב שהערת את צומת ליבי בקשר לזה...

תודה על הביקורת =] (אם יש למישהו עוד הערות אני ישמח לשמוע)