[BuildDream]

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

בכל מקרה בקשר לקידוד יש לא מעט דברים שכדאי לשפר, כמו הpadding של הבלוקים - הטקסט צמוד מידי למסגרת.
במקום להשתמש באופציית הרקע בCSS שיכולה לשכפל תמונה כדי שתופיע לאורך כל האלמנט - שמתי לב שהמקודד פשוט מיקום את התמונה הזאת - http://www.ecom.freehost.co.il/kidud...s/kidud_31.gif ומתח אותה ומיקם אותה מספר פעמים לאורך האתר כדי להציג את הרקע.
מלבד שהשיטה לא טובה במיוחד ופחות דינמית, אפשר להסתפק באותה תמונה רק ברוחב של 1 פיקסל - דבר שיקטין משמעותית את התמונה (ותמונות רקע נוספות שבוצע בהן שימוש) ולחסוך בתעבורה ובזמן העליה של האתר.

הקידוד מבחוץ נראה סביר, אבל מבפנים הקוד לא יעיל ולא מסודר ויש עוד מקום לשיפור, לא מספיק רק לעמוד בתקן של W3C.

[IgalSt]

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

נכתב במקור על ידי almog12

טבלה לא נמצאת מחוץ לתקן ואין סיבה לא להשתמש בה.

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

[BuildDream]

נכתב במקור על ידי IgalSt

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


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

אבל יש מקומות, כדגומת תפריטים שלפעמים סידור שלהם בטבלה יכול להיות יעיל יותר מאשר סידור בDiv.
רוב השימוש שלי לטבלאות הוא באמת להצגת תוכן בטבלה, או בטפסים שסידור שלהם בטבלה הוא יעיל ומסודר יותר.

אבל במקרה של הקידוד שהוא הביא פה זה שימוש מוגזם שפוגע בדינמיות של האתר וביעילות של הקוד.

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

[IgalSt]

נכתב במקור על ידי almog12

אבל יש מקומות, כדגומת תפריטים שלפעמים סידור שלהם בטבלה יכול להיות יעיל יותר מאשר סידור בDiv.
רוב השימוש שלי לטבלאות הוא באמת להצגת תוכן בטבלה, או בטפסים שסידור שלהם בטבלה הוא יעיל ומסודר יותר.

אבל במקרה של הקידוד שהוא הביא פה זה שימוש מוגזם שפוגע בדינמיות של האתר וביעילות של הקוד.

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

התג table קיים ומבורך. עם זאת, מבחינתי הוא קיים אך ורק להצגת תוכן טבלאי.

בחיים אני לא אעביר code review אם יש שם טבלאות ליצירת layout.
גם אם זה בטופס או בתפריט.
בטופס חובה להשתמש ב- label עם הגדרת for בהתאם לשדה.
בתפריט מאוד מאוד מומלץ להשתמש ברשימה (ממוינת או לא ממוינת) כאשר כל קישור יושב בתוך LI.

[NetanelEdri]

נכתב במקור על ידי IgalSt

התג table קיים ומבורך. עם זאת, מבחינתי הוא קיים אך ורק להצגת תוכן טבלאי.

בחיים אני לא אעביר code review אם יש שם טבלאות ליצירת layout.
גם אם זה בטופס או בתפריט.
בטופס חובה להשתמש ב- label עם הגדרת for בהתאם לשדה.
בתפריט מאוד מאוד מומלץ להשתמש ברשימה (ממוינת או לא ממוינת) כאשר כל קישור יושב בתוך LI.

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

לגבי הקידוד , כמו שהחברים אמרו לך פה מדובר בקידוד ברמת קוד מאוד נמוכה ומיושנת.
בהצלחה.