![]() | |
![]() |
| | Thread Tools | Display Modes |
#11
| |||
| |||
|
|
Nije Nego wrote: Nije Nego wrote: Not only html, but css as well, accessibility, design, content, language ... whatever catches your eye. http://www.media-marketing.biz Is that all for critique? Few possible answers. 1. Site is good - no need for critique. 2. Site is bad - too much to critique. 3. Everybody on ahc are elsewhere. 4. It's the weekend? |
|
5. There is a mess of warnings given by the CSS validator: http://jigsaw.w3.org/css-validator/v...marketing.biz/ Warnings are not a consideration at this stage, mainly they point to |
#12
| |||
| |||
|
|
5. There is a mess of warnings given by the CSS validator: http://jigsaw.w3.org/css-validator/v...marketing.biz/ Warnings are not a consideration at this stage, mainly they point to missing bg color, and sporadic mixture of px and ems (needed for fine tuning). |
|
I'd be happy if the warnings are my main consideration. |
#13
| |||
| |||
|
|
Nije Nego wrote: 5. There is a mess of warnings given by the CSS validator: http://jigsaw.w3.org/css-validator/v...marketing.biz/ Warnings are not a consideration at this stage, mainly they point to missing bg color, and sporadic mixture of px and ems (needed for fine tuning). Don't forget about them. You may have neglected a background color for .. oh .. navigation where my default background color is the same as your text color. <g |
|
I'd rather use some amount of em for most widths and margins, instead of pixels or percents. That way, your page will expand properly when a visitor increases text size. |
|
I'd be happy if the warnings are my main consideration. I'd be happy if it was above freezing outside. |
#14
| |||
| |||
|
|
Beauregard T. Shagnasty wrote: Nije Nego wrote: 5. There is a mess of warnings given by the CSS validator: http://jigsaw.w3.org/css-validator/v...marketing.biz/ Warnings are not a consideration at this stage, mainly they point to missing bg color, and sporadic mixture of px and ems (needed for fine tuning). Don't forget about them. You may have neglected a background color for .. oh .. navigation where my default background color is the same as your text color. <g Bg color is missing only in few "child" definition, where it has been already defined with the parent. This should not be an issue, hopefully. |
|
I'd be happy if the warnings are my main consideration. I'd be happy if it was above freezing outside. It's +14 here - blessed by Adriatic Sea. |
#15
| |||
| |||
|
|
Don't forget about them. You may have neglected a background color for .. oh .. navigation where my default background color is the same as your text color. <g Bg color is missing only in few "child" definition, where it has been already defined with the parent. This should not be an issue, hopefully. Assigning { background-color: transparent; } will eliminate the warning. |
|
I'd be happy if the warnings are my main consideration. I'd be happy if it was above freezing outside. It's +14 here - blessed by Adriatic Sea. -4°C here at the moment. My nearest body of water is Lake Ontario, which blesses us with a phenomenon that supposedly doesn't happen anywhere else except in the Great Lakes region of the US: Lake Effect Snow |
#16
| |||
| |||
|
|
Not only html, but css as well, accessibility, design, content, language ... whatever catches your eye. http://www.media-marketing.biz Overall: Excellent work! An appealing design and layout. The HTML code |
#17
| ||||||
| ||||||
|
|
Nije Nego wrote: Not only html, but css as well, accessibility, design, content, language ... whatever catches your eye. http://www.media-marketing.biz Overall: Excellent work! An appealing design and layout. The HTML code is clean. |
|
The CSS needs only a bit of cleanup (color + bg-color, px + ems, etc.). Also you could do with a bit of cascading for the <hx> elements, especially for the left column. For instance create a default "#tijelo_l h3" that has the style of ".left_title2". This eliminates all those "<h3 class="left_title2">". Use a similar approach for <p> in "#tijelo_r". Add a "tijelo_r p" to replace <p class="maintekst"> with <p>. |
|
- There is a link to the current page on each page. |
|
- "Tax Benefits": You are using <br> to create a new paragraph; use <p instead. Needs space around the & in "Bosnia&Herzegovina"; the column is narrow. |
|
- Center justification should be used sparingly. It is harder to read large blocks of text center-justified. Ah! I just noticed the default color style, crveni, is left-justified. Looks better that way. (I was looking a zlatni.) - The red on black for the nav is hard to read, it has poor contrast even on mouseover. |
|
- Hopefully this is just a first draft for the English translation. Yes, English is not my native language (obvious!), but couldn't wait for |
#18
| |||
| |||
|
|
- There is a link to the current page on each page. There should be only # inside #menu <li> (serving to maintain <li display), and # in the footer under "top" (obvious - return to the top)! Do you consider this wrong? It's a matter of opinion. I show a non-link style to give further |
![]() |
| Thread Tools | |
| Display Modes | |
| |