Critiques

My Critique for the Others (Homework 5)
Critique for Homework 5
Critique for Homework 3
Critique for Homework 2

My Critique for the Others

This is the my critique I gave the class members I am in charge of.

Hi, Nick

Here is my critique of you assignment 5, ICS 415 Web Programming.

Contents
-language: good. No spell miss when executing spell checker.
-structure: good. You did discriminate each subtitle by changing the size.
-contents: Need some explanation about example. See the comment below.

Presentation
-text formatting: good, You made a paragraph with some indents.
-graphics and other media: N/A
-CSS: Need CSS to define font too to improve your productivity. See the comment below.

Interaction
-links: good. No dead links
-forms: good.
-JavaScript: good

<Problems>
(1) No Explanation About Your Example
You give user a actual example with Javascript code.However, it would
be good idea to give user how it works on each line of your source code,
or describe the process of your example separately.

(2) Need CSS to define font attribute and color
It seems to me that you use CSS as the "hw4.css". However, you do not
use this to define the font so that you change the font face many times
as "Verdana, Arial, Helvetica, sans-serif". It would be better to
define the font too in terms of your productivity and ease of
maintenance. You can define your own class to define your favorite font
style like:
.mainparagraphs {
font-family: Verdana, Arial, Helvetica, sans-serif;
font-size: 12px;
color: #666666;
margin-top: 12px; margin-right: 36px;
margin-bottom: 12px;
margin-left: 36px;
line-height: 20px
}
and use in the <body>:
<P CLASS="mainparagraphs">the .mainparagraphs must be applied</P>

<Good Feature I noticed>
(1) Description about both navigator object and its userAgent property.
I like this style. This make user more understandable for the navigator object.I should
follow you.

<Other Suggestion>
This is trivial thing, but how about adding actual URL link in the
reference.The reason is because Heavy user wants to find the new web
site which have never visited before so that it would help his/her
decision on whether or not avoid the site which was already visited.
Some times, the tile of the link is the same as another page title.

Jodi

Hi, Jodi

Here is my critique of you assignment 5, ICS 415 Web Programming.

Contents
-language: good. No spell miss when executing spell checker.
-structure: good. You did descriminate each subtitle as a purple color.

Presentation
-text formating: good, but have a problem on error.html.
See the comment below
-graphics and other media: N/A

Interaction
-links: good. No dead links
-forms: good, but have a problem on table size. See the comment below
-JavaScript: good

<Problems>
(1) Error Page(http://www2.hawaii.edu/~nanbu/error.html)

Tile is still "Untitiled Document".
I tried to right click and open a new window to jump the error page.The
page seems defaul page tilte "Untitled Documment".It is good idea to
change your title name.

No Back Link
After going to the error.html, I did not find the link to go back to
the previous page. It may be fine even though you write the link since
current browsers have the button to the previous page.However, I think
It would be user-friendly to set the link to the previous page.

(2) Table size (Width)
When I open your page with Netscape 6.2, I found that the width of the
table located in the middle spreads. This is the part you can still
improve on your page.

<Good Feature I noticed>
(1) Description about non-Javascript
I like the description about "none Javascript support". I did not
consider if users turn off Javascript or use browsers which do not
support Javascript. I have to consider it too.

How about explaing how users can set javscript off when since you
describe the situation of javascript-off. As one of users, I would try
to do it on whether or not it really works well.

Preston

Hi, Preston

Here is my critique of you assignment 5, ICS 415 Web Programming.

Contents
-language: Spell miss when executing spell checker. "recieve". See the comment below
-structure: good. You did discriminate each subtitle by changing the size.
-contents: good. You give user comment as "Style Tips".s

Presentation
-text formatting: good, You made a paragraph with some indents.
-graphics and other media: N/A
-CSS: almost good. You defined tags in your CSS file so that your
code is pretty neat. However, See the comment below

Interaction
-links: Dead link was found in Contents. See the comment below
-forms: good.
-JavaScript: good. It works except for Opera 6.01

<Problems>
(1) Spell miss
I found the "recieve" in your page. Even though you might check spell
check with spell checker, it would be good idea to check them in the
last minutes.

(2) Dead Link in the Netscape6.2 and Opera6.01
Although the link in the Contents of the top works well in the Internet
Explore 6, it does not work in Netscape and Opera. The possible reason
is, I think, that you use ">" in the HTML code <a href="#anchor1"><Location
.hash></a> so that the browsers misunderstand the code. What do you
think? Try to change ">" to "<".

(3) CSS for table
You might define tags in your "MyTemplate.css" except table. you seems
to define the table color each time so that it would be take long time
if you want to change the color. It would be good idea to define the
table color in the CSS too.

Cherise

Hi, Cherise

Here is my critique of you assignment 5, ICS 415 Web Programming.

Presentation
-text formatting: good, You made a paragraph with some indents.
-graphics and other media: N/A
-CSS: good. You defined tags in your CSS file so that your code is pretty neat

Interaction
-links: Dead link was found. "Reviews for this topic note" in the bottom of your page
-forms: good.
-JavaScript: good. It works except for Opera 6.01

<Other Suggestion>
This is my favorite style for the link to another site which is not my
site. I used "_blank" target attribute to distinguish between my site
and outer site. Whenever user go to the outer site from my site, I use
the "_blank" target attribute, which opens a new window. The example is
like:

<A HREF="http://www.w3schools.com/js/js_window.asp" TARGET="_blank">This link triggers a new window</A>

Critique (Homework 5)

Mark Akira

Contents:
- Spelling error in the overview section: method instead of methods (first sentence), alert() instead of alert() method as in the first sentence (second sentence), massage instead of message (third sentence), “give them OK” instead of “gives them an OK” (third sentence), formatted instead of formatted (fourth sentence).

- Second paragraph of overview section: “the alert() method do not return” instead of “the alert() method does not return” (first sentence), “pushing OK button” instead of “pushing the OK button” (second sentence).

Presentation:
- Very nice layout
- You need a section that talks about design issues.

Interaction:
- Neat button/window

Overall, you have a very good layout however the spelling errors are excessive. I understand English may be your second language, however, you should use the spell checker in Microsoft Word or whatever word processing software you use.

Mark

Alice Sin Mee

Gabriel Ka Ho

Contents
-language: good but u have a spelling error in your overview " .... display to the users, ???sand??? give them OK ....."
-structure: since your subtopics are so short, you might want to think about putting them all in 1 line to conserve space

Presentation
-text formatting: try using a darker text color because it is kind of hard to read
-graphics and other media: I like the printscreen, very well done

Interaction
-links: Worked
-forms: Worked

Gabe Kwok

Paravy

Chris

This is your critique for your JavaScript webpages, that
you did for Assignment 5 in ICS 415.

I will critique your page concerning the
3 main elements of a web page. I will tell you
in which parts you have to improve your web
page.

Contents
-language: good
-structure: good

Presentation
-text formating: good
-graphics and other media: good
-CSS: improve: e.g. no bgcolor attributes in html / don't use "class" or span so often!

Interaction
-links: good
-forms: good
-JavaScript: good

Chris

Critique (Homework 3)

Topic : CSS -Text properties (Overview) & Units - URL

Lance

Sandra

Design of the page is nice. I like the compact format in the tables. It's very neat, and your information is to the point. It is good for reference, for someone who knows what they are already doing and are looking for something specific. I also like the coloring of the code and such.

I like how you included what features were available in which browser for your text properties note. I wish you could have shown us some examples of what each code produces for the text properties.

The URL section was pretty good; a couple other examples might have been nice, but it's fine the way it is too.

Aaron

There should be an example of some of the text properties since you have an
overview for units
Some of the tags don't have IE or NN values, so if you couldn't find them,
you should say that instead of leaving it blank.
For the direction tag, you should explain what ltr and rtl are
You should make a discussion section and say not to use text-decoration's
blink or underline tags (he talked about it in class)
Explain what the unicode-bidi tag does
Instead of writing 'color' or 'length' for parameters, you should define
them (#000000 or 5px for example)
Some of the code seems to be in a smaller font than other parts of code
Nice colors on the page, good layout.

For the overview for units, you should change 'yourURL' to an actual URL so
people don't mistake it for an attribute
Talk about what kind of things you can use the URL tag on

Michael

Chris

I will critique your page concerning the
3 main elements of a web page. I will tell you
in which parts you have to improve your web
page.

> > Contents
> > -language: good
> > -structure: write more
> >
> > Presentation
> > -text formating: good
> > -graphics and other media: improve colors
> >
> > Interaction
> > -links: good
> > -forms: good

Critique (Homework 2)

Topic : TABLE overview (Multi-Column Formatting)

Gordon K

I checked out your assignment page on tables and thought it was prepared and
displayed very well. It seemed to have all the necessary attributes for a
web page. The coloring was good with the examples to make things more
distinct. And the structure was laid out well.

Melissa Daric

Reed Alan

  The background color of your table might not have been the best choice. It tends to blend a little with the color you chose for your keywords like TABLE and TR. You can still read the words, but it might be a better idea to chose more contrasting colors.
  The headings "Code Example" and "HTML Example" are also confusing. The code is in html. You might want to change "HTML Example" to "Example Output" and "Code Example" to "HTML Code Example" or something similar to that.
  Other than that, you did a good job. The format is simple, yet easy to understand and you chose decent examples.

Ruri

*Visually neat and organized since you used a table and colors well. I like it.
*Do you also need "column" for the first table? It only has "Row" now.
*You might want to add "attributes" section (if you have time...
I got feedback from Chris for heading notes, saying "better to have attributes". Especially table has several useful attributes(width, height, border, align, valign ...).

Chris

-Some text is hard readable because the background and text color is nearly the same
-Make a clearer overview of Multi-Column formatting.