From: Marc G. K. <ma...@it...> - 2003-09-12 00:18:41
|
p dont think said: >> Does anybody have a preference for where the "refresh folder list" > link >> should go? It's currently at the bottom of the list. Personally I > think >> it should be up under the "refresh unseen/total info" or alternately > in > > on top is best, since if someone has a long list of folders, it might b= e > off the page and difficult to just click when they want a refresh > >> the folders page (since it's unlikely that the list of folders will be >> changed from anywhere else). Anybody have an opinion on it? > > the refresh isn't just for the list of folders is it? it's also to > refresh the number of new messages, which means that it's actually a > highly used link, at least when the autorefresh time is either zero or > very large. > No, the normal refresh folders link now only refresh the unseen/total inf= o and is located above the folder list just as the previous refresh folder link (same location). The refresh folder list currently located below the folder list does a full refresh which means it also does a LSUB request to the imap server. This full refresh is only needed when you add/rename/delete folders from ANOTHER imap-client during your SquirrelMail session. That was thew reason I placed it below the folderlist. The link normally isn't needed because we use a cached folder list created in the first request to the sqimap_get_folder_list during a SquirrelMail session. I know the current location looks a little bit ugly BUT the thought behin= d it was to stop users to do an unneeded full refresh and therefor the location had to be "out of side" which is below the list. So from performance perspective it would be nice to see a better solution for the link at a difficult place to reach. Of course we can also remove the link entirely which means that users hav= e to logout and login again in case of parrallel imap sessions with changed subscribed folderlist OR enter the folder page which default refresh the cached list. Regards, Marc Groot Koerkamp. |