Results 1 to 15 of 122

Thread: Tips for Reviews & Requests

Threaded View

  1. #1
    Join Date
    Jan 2003

    Tips for Reviewers & Requesters


    When reviewing a Web site, do so in a more detailed manner. Often I see replies such as, "Lose the tables for layout, use CSS and semantic, meaningful markup." In fact, I see such replies too often. It casts a careless atomosphere about, and causes a very negative attitude. A better approach would be to explain how and more importantly why. If you're in a hurry or sidetracked, perhaps you should avoid replying until you can dive deeper into the site you are reviewing. Often times, if you are replying hastily, you either miss important things or appear careless. This is frustrating. Parents, though not particularly interested in "childish games," participate to show their love. In the same way, we should review Web sites with care, even if we are not interested in the actual purpose of the site. Don't try to review every site, unless you feel lead; you are not obliged to respond to any one of them. I say this because, again, a careless, quick response often turns out negative. The purpose of this forum is more to help web developers see their errors and possibly learn new design skills, ideas, and useful practices. I believe if we are more specific and careful, our posts will be much more positive, convincing and helpful.

    Review Requesters

    I am going to explain a few things which are relevant to any design you make. Most of these statements are technical.

    - Validate your HTML. In every web site you ask to be reviewed, someone is going to check the validity of your HTML code. You can use the W3C HTML validator to beat them to the punch, however. What is valid HTML? Allow me to explain further. Each document requires a document type declaration (DTD). You have to define, in the first line of your HTML code, just what is allowed. If you don't, browsers revert to a default one (usually, though not always, HTML 4.01 Transitional). A DTD specifies all allowed HTML elements. If you use an HTML element that is not in the DTD specification - whether or not it "works" in the browser - it is invalid. More information can be found at <http://www.w3.org/TR/html4/struct/global.html>.

    - Do not use tables. The purpose of tables is for tabular data - what you see in Excel spreadsheets. Columns and rows. Using tables for layout makes absolutely no sense. For you it may be different, but chances are if you want to design with tables, you don't suffer from any form of disability. There are many blind, dyslexic, and other handicapped users who browse the Internet. And, though you may not realize it, these sites can become very confusing. What's more, the W3C has a specification regarding accessibility. You should always respect and obey the rules set by the W3C; doing otherwise is promoting bad practices and discouraging the proper use of recommended technologies. The Web Content Accessibility Guidelines specify all the rules that you should take into consideration when designing a site. It is generally a good idea to avoid breaking any of them, but you should always strive for priority 2 accessibility (known as "AA"). You may notice, in the WCAG (Web Content Accessibility Guidelines mentioned above), that it says you can use tables for layout if they linearize. This is absolutely not recommended, as it breaks priority 1. It means, basically, if you can see the table-based layout well enough in a text-only browser, then your layout is bearable. However, it is breaking priority one and therefore is a very bad practice. Instead of using tables for your layout, use Cascading Stylesheets. Their purpose is for Web site design. Their use also saves you file space and bandwidth (because of cache). You can include a single file on each page and use minimal markup (excluding actual text content) to turn 8KB files into 4KB ones. Need more reasons or a longer, more in-depth explanation?

    - For users with CSS: use valid CSS. Colored scrollbars, proprietary "filters," and the like are not recommended W3C technologies and are not specified in the CSS2 specification. Hence, they are invalid CSS and should not be used.

    - Do not depend on client-side scripting of any kind. I know, we all love those beautiful JavaScript menus. But what's wrong with using CSS-based menus that do the same thing? Oh, so you want that cool, sliding effect? Great, use JavaScript, but just make sure the menu works without JavaScript. How? Develop the menu to work, and then if you're using JavaScript anyway, replace the CSS menu with a JavaScript one. Also, avoid
    jav&#97;script:function() links. Link to an alternate, non-JavaScript page and use the onclick event handler with a return false statement to run a function and prevent the browser from being redirected to the non-JavaScript page.

    - High-contrast hurts, but helps; high-contrast helps, but hurts. "It's a cool site! It has a black background, red text, and an awesome fiery header image!" So it's valid HTML, CSS, and uses semantic markup. That's great, but how's it on the eyes? Black backgrounds tend to be strenuous. These high-contrast sites can be difficult to view for a lot of users. However, don't turn around and use a white background with light-blue text that is not readable. Make sure your colors contrast enough, but not too much. Often a white background with medium-colored text or a medium-colored background with light text is the best choice (though light text on a light background may work well sometimes). You want your text readable and not strenuous, but you don't want the site to look ugly as a result. This is a key point for those of you who are on free web hosts (see below); when first starting out, you may like the black background and red text, however it is tough on the eyes and 99.9% of the time is the worst choice you can make. Updated: 10/16/04.

    - Test your site in multiple browsers. Do you browse happy in a browser, or do you muck around the Web in a proprietary markup translator? Hopefully the first. There are alternatives for your default browser. Check your site in other browsers to ensure that it functions properly. If it doesn't, you're probably breaking priority 2 somewhere along the line.

    - Avoid free web hosts that give advertisements. It's acceptable to use if you're just beginning, and need somewhere to place your work for others to view, but do not try to make a free host your perminent residence. Advertisements - popups, banners, especially - are very annoying and will cause your site to look much different (worse) than it would in the first place. Added on 10/16/04.

    That's it, I hope this post helps those reviewing sites and requesting their site to be reviewed!
    Last edited by Jona; 01-30-2005 at 04:21 PM.
    Visit Slightly Remarkable to see my portfolio, resumé, and consulting rates.
    Share on Google+

Thread Information

Users Browsing this Thread

There are currently 1 users browsing this thread. (0 members and 1 guests)

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
HTML5 Development Center



X vBulletin 4.2.2 Debug Information

  • Page Generation 0.23739 seconds
  • Memory Usage 4,942KB
  • Queries Executed 14 (?)
More Information
Template Usage (31):
  • (1)ad_footer_end
  • (1)ad_footer_start
  • (1)ad_global_above_footer
  • (1)ad_global_below_navbar
  • (1)ad_global_header1
  • (1)ad_global_header2
  • (1)ad_navbar_below
  • (3)bbcode_code
  • (19)bbcode_quote
  • (1)footer
  • (1)forumjump
  • (1)forumrules
  • (1)gobutton
  • (1)header
  • (1)headinclude
  • (1)headinclude_bottom
  • (41)memberaction_dropdown
  • (1)navbar
  • (4)navbar_link
  • (1)navbar_moderation
  • (1)navbar_noticebit
  • (1)navbar_tabs
  • (2)option
  • (41)postbit
  • (41)postbit_onlinestatus
  • (41)postbit_wrapper
  • (1)showthread_list
  • (1)spacer_close
  • (1)spacer_open
  • (1)tagbit_wrapper 

Phrase Groups Available (6):
  • global
  • inlinemod
  • postbit
  • posting
  • reputationlevel
  • showthread
Included Files (27):
  • ./showthread.php
  • ./global.php
  • ./includes/class_bootstrap.php
  • ./includes/init.php
  • ./includes/class_core.php
  • ./includes/config.php
  • ./includes/functions.php
  • ./includes/functions_navigation.php
  • ./includes/class_friendly_url.php
  • ./includes/class_hook.php
  • ./includes/class_bootstrap_framework.php
  • ./vb/vb.php
  • ./vb/phrase.php
  • ./includes/functions_facebook.php
  • ./includes/functions_calendar.php
  • ./includes/functions_bigthree.php
  • ./includes/class_postbit.php
  • ./includes/class_bbcode.php
  • ./includes/functions_reputation.php
  • ./includes/functions_notice.php
  • ./includes/functions_threadedmode.php
  • ./packages/vbattach/attach.php
  • ./vb/types.php
  • ./vb/cache.php
  • ./vb/cache/db.php
  • ./vb/cache/observer/db.php
  • ./vb/cache/observer.php 

Hooks Called (73):
  • init_startup
  • friendlyurl_resolve_class
  • init_startup_session_setup_start
  • database_pre_fetch_array
  • database_post_fetch_array
  • init_startup_session_setup_complete
  • global_bootstrap_init_start
  • global_bootstrap_init_complete
  • cache_permissions
  • fetch_postinfo_query
  • fetch_postinfo
  • fetch_threadinfo_query
  • fetch_threadinfo
  • fetch_foruminfo
  • load_show_variables
  • load_forum_show_variables
  • global_state_check
  • global_bootstrap_complete
  • global_start
  • style_fetch
  • global_setup_complete
  • showthread_start
  • showthread_getinfo
  • strip_bbcode
  • friendlyurl_clean_fragment
  • friendlyurl_geturl
  • forumjump
  • cache_templates
  • cache_templates_process
  • template_register_var
  • template_render_output
  • fetch_template_start
  • fetch_template_complete
  • parse_templates
  • fetch_musername
  • notices_check_start
  • notices_noticebit
  • process_templates_complete
  • friendlyurl_redirect_canonical
  • showthread_post_start
  • showthread_query_postids_threaded
  • showthread_threaded_construct_link
  • showthread_query
  • bbcode_fetch_tags
  • bbcode_create
  • showthread_postbit_create
  • postbit_factory
  • postbit_display_start
  • bbcode_parse_start
  • postbit_imicons
  • bbcode_parse_complete_precache
  • bbcode_parse_complete
  • postbit_display_complete
  • memberaction_dropdown
  • tag_fetchbit_complete
  • forumrules
  • navbits
  • navbits_complete
  • build_navigation_data
  • build_navigation_array
  • check_navigation_permission
  • process_navigation_links_start
  • process_navigation_links_complete
  • set_navigation_menu_element
  • build_navigation_menudata
  • build_navigation_listdata
  • build_navigation_list
  • set_navigation_tab_main
  • set_navigation_tab_fallback
  • navigation_tab_complete
  • fb_like_button
  • showthread_complete
  • page_templates