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

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

עוד קצת מידע:
בגוגל אני מקום 8-6 במילה "מדריכים"!, כל יום יש באתר מעל 100 גולשים שונים.
ואני גם מרוויח לא רע ממנו. מגוגל הרווחתי 28 דולר ומחברת פרסום ישראלית אחרת 50 שקל.

אז אשמח לקבל טיפים והצעות לשיפור מכל התותחים פה

חג שמח.

http://uzone.co.il

לא יודע איפה לייעץ לך... מלבד הדברים הבאים שאני מניח שאתה כבר יודע:

- נסה לצמצם קוד מיותר:
יש שם המון אלמנטים של טבלאות, ואני לא בטוח שיש צורך בכולם...
יש תגים עם מאפיין onmouseover וכד'. בימינו עדיף להשתמש במאפיין ID וסקריפט JS הנמצא בקובץ חיצוני. נסה להכניס כמה שיותר קוד JS לתוך קובץ חיצוני, וכן לגבי CSS.
זאת כדי שהאתר שלך יהיה קריא יותר עבור מנועי החיפוש.

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

מה הבעייה ב ONMOUSEOVER?
זה מאט את האתר ? לא תקני? לא ממש הבנתי.

זה פשוט לא נהוג.
אתה מערב סקריפט בשפת סימון...

הקוד הבא:

קוד:

<div onmouseover='...' onmouseout='...'></div>
<div onmouseover='...' onmouseout='...'></div>
<div onmouseover='...' onmouseout='...'></div>
<div onmouseover='...' onmouseout='...'></div>
<div onmouseover='...' onmouseout='...'></div>

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


כנס לולידטור של W3C:
http://validator.w3.org/#validate_by_input

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

איך אני עושה את זה עם JS ?
לקחתי את הטבלה בעזרת GETELEMENTBYID

עכשיו, איך אני עושה שבעת מעבר עכבר ישתנה הצבע ?

תודה

קודם תודא שהקוד JS ירוץ אחרי שהאלמנטים (שאתה רוצה להפעיל עליהם את הקוד) ייטענו.

עכשיו, מה שנהוג הוא ליצור אלמנט אב (יש גם שיטות אחרות), ולצאצאים תחתיו לא יהיה אותו שם של התג כמו הבנים:

קוד:

<div id='father'>
	<div>...</div>
	<div>...</div>
	<div>...</div>
	<div>...</div>
</div>
<script type='text/javascript'>//<![CDATA[
var sons = document.getElementById("father").getElementsByTagName("div");
var x;
for (x in sons)
{
	sons[x].onmouseover = function() { /* TODO */ }
	sons[x].onmouseout = function() { /* TODO */ }
}
//]]>
</script>

יש גם דרכים אחרות אבל זה תלוי איך הקוד כתוב אצלך

אוקיי.
מה אני כותב בתוך הפונקצייה עצמה לשינוי הצבע?
ניסיתי ככה - ניסיתי גם במקום this לכתוב x
שניהם לא עובדים....

קוד PHP:

 var sons = document.getElementById("menu1o").getElementsByTagName("div");
var x;
for (x in sons)
{
    sons[x].onmouseover = function() {this.style.backgroundColor='#F0FCFC'; }
    sons[x].onmouseout = function() { /* TODO */ }
} 


הקוד עצמו מופיע בתוך קובץ JS שנטען אטומטית לעמוד

אתה צריך להשתמש ב- this, ולא ב-x, בין היתר כי x מוגדר רק בתוך הפונקציה שהצהרת עליו (כי יש לו קידומת var). קרא את התגובה שלי (בסוף למטה) למידע נוסף: http://www.fresh.co.il/vBulletin/sh...030#post2862919

האם מופיעה לך שגיאת JS בעקבות הקוד?
אם כן, מה היא?

אאל"ט, קוד ה-JS שנמצא בין תגי ה-head אמור להתבצע לאחר טעינת העמוד.
ליתר ביטחון, עדיף שתדרוש במפורש שהקוד ירוץ לאחר שהעמוד יטען:

קוד:

window.onload = function()
{
var sons = document.getElementById("menu1o").getElementsByTagName("div");
var x;
for (x in sons)
{
    sons[x].onmouseover = function() {this.style.backgroundColor='#F0FCFC'; }
    sons[x].onmouseout = function() { /* TODO */ }
}
}

הוספתי את הקוד בתוך
http://uzone.co.il/includes/dropdowncontent.js

ועדיין זה לא עובד...