ביקורת | קידוד - Xenon Lamps

[Shon12]

שלום,

לאחר זמן רב שלא קידדתי, אשמח לבקש דעות על הקידוד הבא:
http://shongefen.freehost.co.il/xenon.html
אשמח אם תוכלו להסתכל ב-CSS,html אם תוכלו למצוא משהו שאפשר לייעל אותו...
תודה רבה!

[Erez | TrustMedia.co.il]

נתחיל מהתמונות רקע:
http://shongefen.freehost.co.il/imag...itled-1_02.png
יכלת לשים תמונה אחת ברוחב אחד פיקסל בתור רקע ועוד תמונה אחת ללוגו,זה היה הרבה יותר יעיל
http://shongefen.freehost.co.il/imag...itled-1_05.png
?! בשביל מה לשים תמונת רקע שחורה?שים פשוט צבע שחור בתור רקע
http://shongefen.freehost.co.il/images/bottom.jpg
כנ"ל
קוד HTML:
יש מקומות שיכלת לקצר,לדוגמא:

<div class="header">

			<ul>
				<li>FAQ</li>
				<li>Installation Guide</li>
				<li>Xenon Lamps</li>
				<li>Xenon Kits</li>
				<li>Home</li>

			</ul>
		</div>

להפוך ל:

			<ul class="header">
				<li>FAQ</li>
				<li>Installation Guide</li>
				<li>Xenon Lamps</li>
				<li>Xenon Kits</li>
				<li>Home</li>

			</ul>

ובנוסף יש מספר מקומות שבהם שמת CSS בinline,עדיף שתכניס את זה לקובץ CSS
הCSS דיי סבבה

[Shon12]

אוקיי,תודה רבה, יתוקן.
חוץ מזה, עוד משהו?

[Shon12]

אשמח לשמוע עוד הצעות חוץ מההצעות שהוצעו...

[reXtea]

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

							<div class="videos_links"><a href="#">How to replacement Xenon lamp</a></div> 
							<div style="margin:0; font-size:7px;">...............................................................................</div>

אתה יכול להעיף לגמריי את הDIV שבתוכו הנקודות, ובקלאס videos_links תוסיף:

border-bottom: 1px dotted #FFFFFF;