comments and constructive criticism wanted

M

Michael Winter

http://www.nenya.be/41229-1/cal.htm

(Only the calendar navigation links work)

Considering that majority of the work isn't yours, what precisely do you
want critiqued?

Aside from the problems that were pointed out in ciwas, the code clutters
the global namespace. Reusable code like this would best be presented as
an object, adhering to practices you'd expect for software development in
general.

Mike


Please don't multi-post.
<URL:http://www.cs.tut.fi/~jkorpela/usenet/xpost.html>
 
S

steven

Hywel Jenkins said:
It's nice, but having the week/month/day navigation as you have is a
PITA. Why not just make the days clickable?

Yeah, that's what I also said in the to-do section.
Have the week/month
navigation in a more logical position.

Like above or below the calendar, you mean? I did have plain buttons for
this purpose, but they didn't look very attractive; I may create bitmaps
instead.
Thanks for the comment.

Steven
 
H

Hywel Jenkins

Yeah, that's what I also said in the to-do section.


Like above or below the calendar

Yes. This way, perhaps the entire thing could be self-contained as
Michael suggests.
 
M

McKirahan

steven said:
Yeah, that's what I also said in the to-do section.


Like above or below the calendar, you mean? I did have plain buttons for
this purpose, but they didn't look very attractive; I may create bitmaps
instead.
Thanks for the comment.

Steven

"<<", "<", ">", and ">>" are often used to go to previous year, previous
month, next month, and next year, respectively; no images needed.

Also, I would right justify (followed by " &nbsp; ") the day numbers.

Why the minus sign before each week?
 
S

steven

McKirahan said:
"<<", "<", ">", and ">>" are often used to go to previous year, previous
month, next month, and next year, respectively; no images needed.

Also, I would right justify (followed by " &nbsp; ") the day numbers.

Why the minus sign before each week?

ISO-8601 "week only of the implied year". But I admit that it's a bit
confusing. I may change it to this other ISO-8601 format "year and week only
in the implied century": YYWww (e.g. "04W52").
Thanks for the reply.

Steven
 

Ask a Question

Want to reply to this thread or ask your own question?

You'll need to choose a username for the site, which only take a couple of moments. After that, you can post your question and our members will help you out.

Ask a Question

Members online

No members online now.

Forum statistics

Threads
473,755
Messages
2,569,536
Members
45,007
Latest member
obedient dusk

Latest Threads

Top