| Summary | Suggestions for showing mailbox size in IMP (Patch) |
| Queue | IMP |
| Queue Version | HEAD |
| Type | Enhancement |
| State | Resolved |
| Priority | 1. Low |
| Owners | slusarz (at) horde (dot) org |
| Requester | ekg2002 (at) columbia (dot) edu |
| Created | 06/20/2006 (6962 days ago) |
| Due | |
| Updated | 07/20/2006 (6932 days ago) |
| Assigned | 06/21/2006 (6961 days ago) |
| Resolved | 07/20/2006 (6932 days ago) |
| Milestone | |
| Patch | No |
page - like the delete confirmation page. Showing the size next to
the folders in the mailbox list, with this information disappearing on
the next folders page load, is not totally intuitive given the current
file size. I will see if I can come up with a solution.
essentially does the same thing as the given JS.
At least on my IMP test rig, the size goes away on the next page
refresh. This was intentional, as I think it's overkill to have a
"Show Size" and a "Do Not Show Size" action. In other words, I don't
think the action should be sticky.
All that said, I'm open to change it.
selected "Show Size" on a folder, or did I just merge the patch
incorrectly?
In addition, I had to add "else
if(action=='mbox_size'){submitAction(action);}" to
imp/templates/javascript/folders.js at line 12. Again, is this a
problem that only I had, or is it something that should be put in the
patch?
Assigned to Michael Slusarz
Priority ⇒ 1. Low
Type ⇒ Enhancement
Summary ⇒ Suggestions for showing mailbox size in IMP (Patch)
Queue ⇒ IMP
New Attachment: mbox_size.diff
State ⇒ New
a new action on the Folder tree to show the size of a mailbox. It is
attached. I'd like to ask the community if they have any suggestions
as to the implementation.
The main problem I see is that I'm not sure that IMAP/Tree::build() is
the proper place to do the logic required. The problem I see is that I
need to cache something in the $row array, only once. The patch works,
but I'm not sure the implementation is the Horde (or the best) way.