6.0.0-alpha12
6/10/25

[#10685] Context menu options are not hidden for fixed folders
Summary Context menu options are not hidden for fixed folders
Queue IMP
Queue Version Git master
Type Bug
State Resolved
Priority 1. Low
Owners
Requester goncalo.queiros (at) portugalmail (dot) net
Created 10/27/2011 (4975 days ago)
Due
Updated 11/01/2011 (4970 days ago)
Assigned
Resolved 11/01/2011 (4970 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch Yes

History
11/01/2011 02:05:31 AM Michael Slusarz Comment #3
State ⇒ Resolved
Reply to this comment
Since we now store on the LI element the mbox name, but base64 
encoded, the comparisson
DIMP.conf.fixed_folders.indexOf(baseelt.retrieve('mbox'))
no longer works.
Fixed.
The traditional view is buggy on the folder.php page, so i can't 
make a patch to it, without first looking at other errors, but think 
you can fix it much faster :-)
I have no idea what you mean by "buggy" - works fine for me.  Further, 
on the traditional page, we can't disable fixed folders because not 
all the actions in the drop-down menu are disabled when a mailbox is 
fixed (e.g. download all).  These actions need to be rejected on an 
action-by-action basis.
11/01/2011 02:03:46 AM Git Commit Comment #2 Reply to this comment
Changes have been made in Git for this ticket:

Bug #10685: Hide context menu options on fixed folders
Signed-off-by: Michael M Slusarz <slusarz@horde.org>

  1 files changed, 1 insertions(+), 1 deletions(-)
http://git.horde.org/horde-git/-/commit/5b86247e171ad98768bd3f784a9bb8bf43ccec22
10/27/2011 05:10:20 PM goncalo (dot) queiros (at) portugalmail (dot) net Comment #1
Priority ⇒ 1. Low
New Attachment: 0001-Hide-context-menu-options-on-fixed-folders.patch Download
Patch ⇒ Yes
Milestone ⇒
Queue ⇒ IMP
Summary ⇒ Context menu options are not hidden for fixed folders
Type ⇒ Bug
State ⇒ Unconfirmed
Reply to this comment
Since we now store on the LI element the mbox name, but base64 
encoded, the comparisson
DIMP.conf.fixed_folders.indexOf(baseelt.retrieve('mbox'))
no longer works.

The attached patch fixes this for dynamic view, but not for 
traditional view. The traditional view is buggy on the folder.php 
page, so i can't make a patch to it, without first looking at other 
errors, but think you can fix it much faster :-)

Saved Queries