פרסום | Orange - קידוד אתר

[אביב]

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

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

מה שכן, אכן מדובר בעיצוב פשוט מאוד ואין יותר מדיי על מה לתת ביקורת, חוץ מאולי הנקודות הבאות:

קוד ה-css אכן לא מעוצב יפה.

ב- Features ו-Prices האיקון שלידן מיושר לא נכון אנכית. בכלל, הייתי מיישם את האייקון כתמונת רגע אולי ולא כ-img.

כמו כן, את החלק של ה- prices הייתי משנה ככה. מ-

<ul>
	<li class="money">Text Text Text Text Text Text Text Text Text Text Text Text </li>
	<li class="money">Text Text Text Text Text Text Text Text Text Text Text Text </li>
	<li class="money">Text Text Text Text Text Text Text Text Text Text Text Text </li>
	<li class="money">Text Text Text Text Text Text Text Text Text Text Text Text </li>
</ul>

ל-

<ul class="money">
	<li>Text Text Text Text Text Text Text Text Text Text Text Text </li>
	<li>Text Text Text Text Text Text Text Text Text Text Text Text </li>
	<li>Text Text Text Text Text Text Text Text Text Text Text Text </li>
	<li>Text Text Text Text Text Text Text Text Text Text Text Text </li>
</ul>

בוצעו קצת שינויים בסורס לפי מה שאני רואה, בכול מקרה לא מקובל לתת TAB לתגיות בסיס כמו HEAD או BODY.

[IgalSt]

נכתב במקור על ידי Aviv.Hazor

בוצעו קצת שינויים בסורס לפי מה שאני רואה, בכול מקרה לא מקובל לתת TAB לתגיות בסיס כמו HEAD או BODY.

אני יכול להסכים לגבי ה-body, אבל לא לגבי ה-head.

בכל אופן, זה לא עקרוני.

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