Re: mod_mbox bug squashing

2005-08-29 Thread Eli Marmor
Maxime Petazzoni wrote:

  5) If you know that the AJAX view only works in Gecko based browsers
  do you plan on sniffing the browser type and redirecting to the
  non-AJAX view in that case?  This is probably a bit of a
  controversial subject, but it seems like the kind of thing that
  would have to be resolved before the ASF could roll something like
  this out.
 
 I'll try to make the AJAX view work on as many browser as
 possible. I'd like it to work on Konqueror and Opera. The IE problem
 does not seem to be fixable.

I agree that it is better to dedicate the time to support more browsers,
than to invest too much in cases where AJAX is not supported.

However, what is this problem with IE that doesn't seem to be fixable?

If it was discussed here and I missed it, I apologize, but please give
me a pointer to that discussion.

P.S. It's cool.

Thanks,
-- 
Eli Marmor
[EMAIL PROTECTED]
Netmask (El-Mar) Internet Technologies Ltd.
__
Tel.:   +972-9-766-1020  8 Yad-Harutzim St.
Fax.:   +972-9-766-1314  P.O.B. 7004
Mobile: +972-50-5237338  Kfar-Saba 44641, Israel


Re: mod_mbox bug squashing

2005-08-29 Thread Maxime Petazzoni
Hi,

First of all, thanks for your reply, I was looking forward to having
mod_mbox's original author comments !

 It crashes Safari (Mac OS X 10.4.x) when I try to select a message
 in the list when I already have a message open.  Ouch!

Ok, if it crashes it means we won't have any debugging information. If
you have a couple of minutes today on IRC, can we make step by step
debugging ? I'll add alert box in the toggleMessage() function, and
you'll tell me where it crashes so I can isolate the problem.

 With Firefox (Deer Park Alpha 2), it works.

It seems to work perfectly with any version of Firefox (tested from
1.0.4). Unfortunately that's only 20~30% of the browsers.

 The subject summaries in 'Message List' doesn't list the month - it
 just says 'Monday 01, 11:03:35' - I really think we need to emit the
 month there.  (Yes, it's on the top, but that's easy to miss.)

I just changed that yesterday evening because I find that the date is
too long in the message list, especially with lower resolutions. What
would you like to see there ?

Tuesday 12 April 2005, 14:29:51 (very, very long)
Tue, 12 Apr 2005, 14:29:51 (like RFC822 dates but without the GMT stuff)
12 April 2005, 14:29:51
Tue, 12 Apr, 14:29
... or any other combination you like. I must admit I don't know which
one to choose here :P

 What is the pattern for the 'Message List' for showing the next
 messages in the sequence?  I'm not sure how many messages it is
 trying to show - it doesn't seem consistent to me.

What do you call the pattern ? If you are referring to the paging
system, in date and author sort, 100 messages per page are
displayed. In thread sort, 42 full threads are shown (this explain why
you often have less pages in threaded mode than in date sort).

 One feature I like is having the 'Site Index' link - this is in
 mod_mbox trunk.  How about having the 'Mailing list archives' also
 having a 'site index' link so you can view what other archives are
 present fro that site.

Since this feature is actually in trunk/, you've said it, it will be
backported. I just don't considerer it as a key feature for the
moment. It's been an almost non-stop work+sleep+work+eat+sleep+work
session since I'm back home, and I had more important features to code
before this one :)

I looked at your recent changes to mod_mbox, and I value them.

 I agree with Garrett that having the 'next' and 'prev' links should
 also appear at the *bottom* of the message - this is so that when
 you read the message, you can then click to read the next message in
 the thread w/o scrolling up.

 Most critically, I think the 'next' and 'prev', and numbered links
 can become confusing when trying to read a message.  If you want to
 read the next message in the thread - clicking the 'message list'
 'next'/'prev' links brings you out.  So, finding some way to resolve
 that to make it clearer for the user - perhaps merely adding
 prev/next message in 'message view' list would be good. Or, have
 some way of hiding (or disabling) the list navigation when we have a
 message view active.

We are still talking about the AJAX browsing interface, eh ? If yes,
there is currently no message navigation in the message view. I agree
with your comments, and these links will be added soon. Also, please
notice that prev/next links will only be able to go through the
currently loaded page. I can eventually make the last 'Next' click
load the next page if it exists, but it won't be possible to load the
first message of this new page directly (blame the first 'A' of AJAX
(asynchronous)).

 But, again, it looks a lot better than what we have today.  Cool.
 -- justin

Thanks,
- Sam

-- 
Maxime Petazzoni (http://www.bulix.org)
 -- gone crazy, back soon. leave message.


signature.asc
Description: Digital signature


Re: mod_mbox bug squashing

2005-08-29 Thread Maxime Petazzoni
Hi,

* Eli Marmor [EMAIL PROTECTED] [2005-08-29 11:57:50]:

 Maxime Petazzoni wrote:
 
  I'll try to make the AJAX view work on as many browser as
  possible. I'd like it to work on Konqueror and Opera. The IE problem
  does not seem to be fixable.
 
 I agree that it is better to dedicate the time to support more browsers,
 than to invest too much in cases where AJAX is not supported.
 
 However, what is this problem with IE that doesn't seem to be fixable?
 
 If it was discussed here and I missed it, I apologize, but please give
 me a pointer to that discussion.

It was not discussed here, so no need to apologize.

Basically, I have an body onLoad function that calls the box list and
message list creation. The getMsgList() function create an
XmlHttpRequest call to the YY./ajax/thread URL, and waits for the
result. When it gets it, it saves the resulting XML document into a
global variable '_msglist' and then call a processing routine,
drawFullMsgList().

But at the first attempt of drawFullMsgList() to access the _msglist
variable, IE thraw a Javascript error, saying that _msglist is null or
not defined.

I don't have a clue on how to fix this. If someone has, please show
up.

- Sam
-- 
Maxime Petazzoni (http://www.bulix.org)
 -- gone crazy, back soon. leave message.


signature.asc
Description: Digital signature


Re: mod_mbox bug squashing

2005-08-29 Thread Justin Erenkrantz
--On August 29, 2005 11:52:18 AM +0200 Maxime Petazzoni 
[EMAIL PROTECTED] wrote:



First of all, thanks for your reply, I was looking forward to having
mod_mbox's original author comments !


Ha!  =)


Ok, if it crashes it means we won't have any debugging information. If
you have a couple of minutes today on IRC, can we make step by step
debugging ? I'll add alert box in the toggleMessage() function, and
you'll tell me where it crashes so I can isolate the problem.


Perhaps later on today.  But, you're in France, right?  I'm in California 
(GMT-7), so that might be hard to arrange.



With Firefox (Deer Park Alpha 2), it works.


It seems to work perfectly with any version of Firefox (tested from
1.0.4). Unfortunately that's only 20~30% of the browsers.


Is it that high now?  =)


The subject summaries in 'Message List' doesn't list the month - it
just says 'Monday 01, 11:03:35' - I really think we need to emit the
month there.  (Yes, it's on the top, but that's easy to miss.)


I just changed that yesterday evening because I find that the date is
too long in the message list, especially with lower resolutions. What
would you like to see there ?

Tuesday 12 April 2005, 14:29:51 (very, very long)
Tue, 12 Apr 2005, 14:29:51 (like RFC822 dates but without the GMT stuff)
12 April 2005, 14:29:51
Tue, 12 Apr, 14:29
... or any other combination you like. I must admit I don't know which
one to choose here :P


I think the last option is probably best.  (Dayname, Day Mon, 
Hour:Min) - no need for second granularity in the summary.



What is the pattern for the 'Message List' for showing the next
messages in the sequence?  I'm not sure how many messages it is
trying to show - it doesn't seem consistent to me.


What do you call the pattern ? If you are referring to the paging
system, in date and author sort, 100 messages per page are
displayed. In thread sort, 42 full threads are shown (this explain why
you often have less pages in threaded mode than in date sort).


When you are reading a message - it still shows a part of the message list 
- but it seems a bit non-intuitive as to how many messages it is showing 
when you are reading a message.



moment. It's been an almost non-stop work+sleep+work+eat+sleep+work
session since I'm back home, and I had more important features to code
before this one :)


Sure.  No worries.


I looked at your recent changes to mod_mbox, and I value them.


Thanks!


We are still talking about the AJAX browsing interface, eh ? If yes,
there is currently no message navigation in the message view. I agree
with your comments, and these links will be added soon. Also, please
notice that prev/next links will only be able to go through the
currently loaded page. I can eventually make the last 'Next' click
load the next page if it exists, but it won't be possible to load the
first message of this new page directly (blame the first 'A' of AJAX
(asynchronous)).


Yes, I'm talking about the AJAX interface.

So, I look forward to seeing how you address this.  ;-)


Thanks,
- Sam


Hey, you're the one doing the work not me.  ;-)

Thanks!  -- justin


Re: mod_mbox bug squashing

2005-08-29 Thread Maxime Petazzoni

 Perhaps later on today.  But, you're in France, right?  I'm in
 California (GMT-7), so that might be hard to arrange.

I use to stay late on IRC, just ping me on #apache-modules when you
have some time.

 I just changed that yesterday evening because I find that the date is
 too long in the message list, especially with lower resolutions. What
 would you like to see there ?
 
 Tuesday 12 April 2005, 14:29:51 (very, very long)
 Tue, 12 Apr 2005, 14:29:51 (like RFC822 dates but without the GMT stuff)
 12 April 2005, 14:29:51
 Tue, 12 Apr, 14:29
 ... or any other combination you like. I must admit I don't know which
 one to choose here :P
 
 I think the last option is probably best.  (Dayname, Day Mon,
 Hour:Min) - no need for second granularity in the summary.

Ok, I like this one too. It's already set up on
http://skikda.bulix.org/archives/dev/

 What is the pattern for the 'Message List' for showing the next
 messages in the sequence?  I'm not sure how many messages it is
 trying to show - it doesn't seem consistent to me.
 
 What do you call the pattern ? If you are referring to the paging
 system, in date and author sort, 100 messages per page are
 displayed. In thread sort, 42 full threads are shown (this explain why
 you often have less pages in threaded mode than in date sort).
 
 When you are reading a message - it still shows a part of the
 message list - but it seems a bit non-intuitive as to how many
 messages it is showing when you are reading a message.

This small context view display the messages before and after the one
you are reading. The context size is -3 and +3 messages, except if you
are in the first or last 3 messages of course.

 We are still talking about the AJAX browsing interface, eh ? If yes,
 there is currently no message navigation in the message view. I agree
 with your comments, and these links will be added soon. Also, please
 notice that prev/next links will only be able to go through the
 currently loaded page. I can eventually make the last 'Next' click
 load the next page if it exists, but it won't be possible to load the
 first message of this new page directly (blame the first 'A' of AJAX
 (asynchronous)).
 
 Yes, I'm talking about the AJAX interface.
 
 So, I look forward to seeing how you address this.  ;-)

I've added message view navigation and some other tweaks. Please
referer to my last commits.

- Sam

-- 
Maxime Petazzoni (http://www.bulix.org)
 -- gone crazy, back soon. leave message.


signature.asc
Description: Digital signature


mod_mbox bug squashing

2005-08-28 Thread Maxime Petazzoni
Hi,

As the Summer Of Code deadline approaches, I'd like to put mod_mbox to
the test in order to fix as many bugs as possible.

If you are willing to help, here follows some useful information :

- mod_mbox offers an AJAX browser (YY.mbox/browser) for dynamic
  archive browsing to Javascript capable browsers. Still, the AJAX
  browser is known to be working only in Gecko-based browser for the
  moment.

- Whatever your browser is, if the AJAX browsing interface does not
  work for you, fall back to flat mode by deactivating Javascript or
  changing the URL from YY.mbox/browser to YY.mbox/thread

- Please submit reports as answers to this thread, including your
  operating system version, your browser version and the URL where you
  noticed the problem. If it's in the AJAX browsing interface, please
  give the month, year, and relevant mail information (author, date
  and subject).

The latest version of mod_mbox running an up-to-date rsync of the
httpd-dev mail archives is available at
http://skikda.bulix.org/archives/dev/

Thanks for your help,
- Sam

-- 
Maxime Petazzoni (http://www.bulix.org)
 -- gone crazy, back soon. leave message.


signature.asc
Description: Digital signature


Re: mod_mbox bug squashing

2005-08-28 Thread Garrett Rooney

Maxime Petazzoni wrote:

Hi,

As the Summer Of Code deadline approaches, I'd like to put mod_mbox to
the test in order to fix as many bugs as possible.

If you are willing to help, here follows some useful information :

- mod_mbox offers an AJAX browser (YY.mbox/browser) for dynamic
  archive browsing to Javascript capable browsers. Still, the AJAX
  browser is known to be working only in Gecko-based browser for the
  moment.

- Whatever your browser is, if the AJAX browsing interface does not
  work for you, fall back to flat mode by deactivating Javascript or
  changing the URL from YY.mbox/browser to YY.mbox/thread

- Please submit reports as answers to this thread, including your
  operating system version, your browser version and the URL where you
  noticed the problem. If it's in the AJAX browsing interface, please
  give the month, year, and relevant mail information (author, date
  and subject).

The latest version of mod_mbox running an up-to-date rsync of the
httpd-dev mail archives is available at
http://skikda.bulix.org/archives/dev/


I played around with this a bit, and I have a few comments.

1) It looks very cool, much nicer than the existing mod_mbox UI.

2) The way you let the user page through different parts of a given 
month is easy to miss.  It took me a while to notice that the 1 2 3 4 
links were up at the top.  Adding next and prev links at the top and 
bottom would solve that problem.


3) In the AJAX UI visiting a link doesn't result in it being marked as 
visited in the same way that visiting a link in the non-AJAX UI does. 
 If there's some CSS/JavaScript magic that can fix that it would be a 
good thing, as it's useful when searching through large amounts of mail 
to see where you've already looked.


4) The Back button is broken in the AJAX view, but you probably 
already knew that.  Don't know if there's much that can be done about it.


5) If you know that the AJAX view only works in Gecko based browsers do 
you plan on sniffing the browser type and redirecting to the non-AJAX 
view in that case?  This is probably a bit of a controversial subject, 
but it seems like the kind of thing that would have to be resolved 
before the ASF could roll something like this out.


-garrett


Re: mod_mbox bug squashing

2005-08-28 Thread Maxime Petazzoni
Hi,

 1) It looks very cool, much nicer than the existing mod_mbox UI.

Thanks, that was the main goal of httpd-mbox-if, I'm happy to hear
that people find the new version nice and good-looking.

 2) The way you let the user page through different parts of a given
 month is easy to miss.  It took me a while to notice that the 1 2 3
 4 links were up at the top.  Adding next and prev links at the
 top and bottom would solve that problem.

I've added Previous and Next links to the page selector. How is it ?

 3) In the AJAX UI visiting a link doesn't result in it being marked
 as visited in the same way that visiting a link in the non-AJAX UI
 does.  If there's some CSS/JavaScript magic that can fix that it
 would be a good thing, as it's useful when searching through large
 amounts of mail to see where you've already looked.

I've already took a look to this problem, and I don't think we can
solve them. Switching a link (a a markup) to the Css meta-class
:visited does not seem to be possible in Javascript.

Furthermore, changing link's color could be good, but they will
disappear as soon as you reload the AJAX browsing interface ...

 4) The Back button is broken in the AJAX view, but you probably
 already knew that.  Don't know if there's much that can be done
 about it.

What do you expect the back button to do ? Since we don't change the
page, the browser won't help you here. As you've said, there's not
much we can do here.

 5) If you know that the AJAX view only works in Gecko based browsers
 do you plan on sniffing the browser type and redirecting to the
 non-AJAX view in that case?  This is probably a bit of a
 controversial subject, but it seems like the kind of thing that
 would have to be resolved before the ASF could roll something like
 this out.

I'll try to make the AJAX view work on as many browser as
possible. I'd like it to work on Konqueror and Opera. The IE problem
does not seem to be fixable.

I think I'm going to put a link into the Loading... box to the flat
version, so if the browser get stuck to the loading part, the user can
fall back into flat mode.

I'm also working on XHTML validation.

Thanks a lot for your comments,
- Sam

-- 
Maxime Petazzoni (http://www.bulix.org)
 -- gone crazy, back soon. leave message.


signature.asc
Description: Digital signature


Re: mod_mbox bug squashing

2005-08-28 Thread Garrett Rooney

Maxime Petazzoni wrote:


2) The way you let the user page through different parts of a given
month is easy to miss.  It took me a while to notice that the 1 2 3
4 links were up at the top.  Adding next and prev links at the
top and bottom would solve that problem.



I've added Previous and Next links to the page selector. How is it ?


Nice.  Thanks.


3) In the AJAX UI visiting a link doesn't result in it being marked
as visited in the same way that visiting a link in the non-AJAX UI
does.  If there's some CSS/JavaScript magic that can fix that it
would be a good thing, as it's useful when searching through large
amounts of mail to see where you've already looked.



I've already took a look to this problem, and I don't think we can
solve them. Switching a link (a a markup) to the Css meta-class
:visited does not seem to be possible in Javascript.

Furthermore, changing link's color could be good, but they will
disappear as soon as you reload the AJAX browsing interface ...


That's too bad, but hey, if it can't be done then it can't be done.

-garrett


Re: mod_mbox bug squashing

2005-08-28 Thread Justin Erenkrantz
--On August 28, 2005 7:26:16 PM +0200 Maxime Petazzoni 
[EMAIL PROTECTED] wrote:



- mod_mbox offers an AJAX browser (YY.mbox/browser) for dynamic
  archive browsing to Javascript capable browsers. Still, the AJAX
  browser is known to be working only in Gecko-based browser for the
  moment.


Looks really nice!  Here's a few comments:

It crashes Safari (Mac OS X 10.4.x) when I try to select a message in the list 
when I already have a message open.  Ouch!


With Firefox (Deer Park Alpha 2), it works.

The subject summaries in 'Message List' doesn't list the month - it just says 
'Monday 01, 11:03:35' - I really think we need to emit the month there.  (Yes, 
it's on the top, but that's easy to miss.)


What is the pattern for the 'Message List' for showing the next messages in 
the sequence?  I'm not sure how many messages it is trying to show - it 
doesn't seem consistent to me.


One feature I like is having the 'Site Index' link - this is in mod_mbox 
trunk.  How about having the 'Mailing list archives' also having a 'site 
index' link so you can view what other archives are present fro that site.


I agree with Garrett that having the 'next' and 'prev' links should also 
appear at the *bottom* of the message - this is so that when you read the 
message, you can then click to read the next message in the thread w/o 
scrolling up.


Most critically, I think the 'next' and 'prev', and numbered links can become 
confusing when trying to read a message.  If you want to read the next message 
in the thread - clicking the 'message list' 'next'/'prev' links brings you 
out.  So, finding some way to resolve that to make it clearer for the user - 
perhaps merely adding prev/next message in 'message view' list would be good. 
Or, have some way of hiding (or disabling) the list navigation when we have a 
message view active.


But, again, it looks a lot better than what we have today.  Cool.  -- justin