HighDots Forums  

critique would be apreciated

Websites/HTML pages critique & reviews Discuss and review existing WWW material (alt.html.critique)


Discuss critique would be apreciated in the Websites/HTML pages critique & reviews forum.



Reply
 
Thread Tools Display Modes
  #11  
Old   
Nije Nego
 
Posts: n/a

Default Re: critique would be apreciated - 02-12-2006 , 06:36 PM






Beauregard T. Shagnasty wrote:
Quote:
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?
It's already Monday here :-)
Quote:
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.


Reply With Quote
  #12  
Old   
Beauregard T. Shagnasty
 
Posts: n/a

Default Re: critique would be apreciated - 02-12-2006 , 08:27 PM






Nije Nego wrote:

Quote:
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.

Quote:
I'd be happy if the warnings are my main consideration.
I'd be happy if it was above freezing outside.

--
-bts
-Warning: I brake for lawn deer


Reply With Quote
  #13  
Old   
Nije Nego
 
Posts: n/a

Default Re: critique would be apreciated - 02-13-2006 , 05:02 AM



Beauregard T. Shagnasty wrote:
Quote:
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.

Quote:
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.
Pixels were used only in header section, ems were not applicable because
of IE.

Quote:
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.

Regards.


Reply With Quote
  #14  
Old   
Beauregard T. Shagnasty
 
Posts: n/a

Default Re: critique would be apreciated - 02-13-2006 , 09:24 AM



Nije Nego wrote:

Quote:
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.
Assigning { background-color: transparent; } will eliminate the warning.

Quote:
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

--
-bts
-Warning: I brake for lawn deer


Reply With Quote
  #15  
Old   
Nije Nego
 
Posts: n/a

Default Re: critique would be apreciated - 02-13-2006 , 02:02 PM



Beauregard T. Shagnasty wrote:

Quote:
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 know about transparent - but would it really make any change to my
sheet other than having less warnings. Don't think so. I think some
warnings could easily be neglected if you know what do they stand for.

Quote:
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

So far we had only: Fake Effect Snow


Reply With Quote
  #16  
Old   
Jim Moe
 
Posts: n/a

Default Re: critique would be apreciated - 02-14-2006 , 05:11 PM



Nije Nego wrote:
Quote:
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.

--
jmm (hyphen) list (at) sohnen-moe (dot) com
(Remove .AXSPAMGN for email)


Reply With Quote
  #17  
Old   
Nije Nego
 
Posts: n/a

Default Re: critique would be apreciated - 02-14-2006 , 05:49 PM



Jim Moe wrote:
Quote:
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.
You're being too kind.

Quote:
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>.
Will do.

Quote:
- 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?

Quote:
- "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.
Already there!

Quote:
- 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.
Noted!

Quote:
- Hopefully this is just a first draft for the English translation.

Yes, English is not my native language (obvious!), but couldn't wait for
the translation.

Thank you indeed for your time & effort, and valuable advices.


Reply With Quote
  #18  
Old   
Jim Moe
 
Posts: n/a

Default Re: critique would be apreciated - 02-15-2006 , 01:49 AM



Nije Nego wrote:
Quote:
- 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
feedback to the visitor about the location within the site. It's a bit of
navigation redundancy. There is, of course, <title> and <h1>. But the
visitor is likely looking at that link's position when the page loads. If
the link does not become "inactive," there is a small, brief confusion
about having successfully selected the page.

--
jmm (hyphen) list (at) sohnen-moe (dot) com
(Remove .AXSPAMGN for email)


Reply With Quote
Reply




Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

vB code is On
Smilies are On
[IMG] code is On
HTML code is Off



Powered by vBulletin Version 3.5.4
Copyright ©2000 - 2008, Jelsoft Enterprises Ltd.