Created attachment 104935 [details] How to delete an entry in the customize toolbar As a user who customizes his toolbar, i think it would be great to have a 'delete' button underneath the 'add' and 'modify' entries, rather than having to click the modify button and then click the delete entry. Also think it would be good to have a 'reset' button underneath the 'delete' button, rather than having to click the toolbar button and then clicking 'restore default settings' entry.
Seems reasonable as usual actions are Add, Modify, and Delete. Component changed to UI (applies to Writer, Calc, Impress, Draw, Base, and Math). Status set to NEW.
An 'Add Separator' button would also be useful, as presently its located under the modify button.
EasyHack: Source Code: http://opengrok.libreoffice.org/xref/core/cui/source/customize/cfg.cxx#806 Dialog file (edit with Glade): http://opengrok.libreoffice.org/xref/core/cui/uiconfig/ui/menuassignpage.ui
I have submitted a patch for this bug report.[0] It adds the delete button. I appreciate comments/reviews. I'll continue with the other buttons after this patch. Can we say that we are going to add "Delete" (already submitted the related patch), "Reset", and "Add Separator" buttons; so the final appearance of the buttons would be like this? -"Add" -"Modify" -"Delete" -"Reset" -"Add Separator" The order of the first four buttons seems right, but I don't know where to put the "Add Separator" button. -- [0] https://gerrit.libreoffice.org/#/c/28323/
What if we move all those buttons in a toolbar below the "Commands" list box?
(In reply to Samuel Mehrbrodt (CIB) from comment #5) > What if we move all those buttons in a toolbar below the "Commands" list box? Do you mean something like this? http://i.cubeupload.com/iTwE8l.png
Muhammet Kara committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=5214eba02c011b5d15cc2ba33f4c6bd811e6cef1 tdf#82840 Add 'delete' button to toolbar customization It will be available in 5.3.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
(In reply to Muhammet Kara from comment #4) > The order of the first four buttons seems right, but I don't know where to > put the "Add Separator" button. Would suggest you order them like this Add Command Add Separator -margin or padding separator- Modify Delete -margin or padding separator- Reset (In reply to Samuel Mehrbrodt (CIB) from comment #5) > What if we move all those buttons in a toolbar below the "Commands" list box? Dont think it looks good that way because it increases the height of the dialog and especially as all the other tabs and even the other section of the current tab have buttons on the right. @Muhammet: If you are feeling so inclined, it would be good to remove the radio buttons from the 'Toolbar' button and have them underneath the toolbar drop down menu, so they are more visible/accessible.
Created attachment 127041 [details] Customize Toolbar Mockup Toolbars tab of Customize dialog.
(In reply to Yousuf (Jay) Philips from comment #8) > (In reply to Muhammet Kara from comment #4) > > The order of the first four buttons seems right, but I don't know where to > > put the "Add Separator" button. > > Would suggest you order them like this > > Add Command > Add Separator > -margin or padding separator- > Modify > Delete > -margin or padding separator- > Reset > > (In reply to Samuel Mehrbrodt (CIB) from comment #5) > > What if we move all those buttons in a toolbar below the "Commands" list box? > > Dont think it looks good that way because it increases the height of the > dialog and especially as all the other tabs and even the other section of > the current tab have buttons on the right. > > @Muhammet: If you are feeling so inclined, it would be good to remove the > radio buttons from the 'Toolbar' button and have them underneath the toolbar > drop down menu, so they are more visible/accessible. What do you think about the attached screenshot? (Customize Toolbar Mockup) (Note that the "Reset" button here resets the selected "toolbar" and restores its default values.)
(In reply to Muhammet Kara from comment #10) > What do you think about the attached screenshot? (Customize Toolbar Mockup) > (Note that the "Reset" button here resets the selected "toolbar" and > restores its default values.) Screenshot looks good, but i'd make these amendments 1) rename the 'visible' label to 'style' 2) restore the 'toolbar' group button which had rename and delete options in it and label it 'modify' 3) vertically center the toolbar command buttons and the move up and move down buttons with the commands listbox 4) replace the new visible separators with margin or padding, similar to how it is in the keyboard tab
(In reply to Yousuf (Jay) Philips from comment #11) > (In reply to Muhammet Kara from comment #10) > > What do you think about the attached screenshot? (Customize Toolbar Mockup) > > (Note that the "Reset" button here resets the selected "toolbar" and > > restores its default values.) > > Screenshot looks good, but i'd make these amendments > > 1) rename the 'visible' label to 'style' Sure. > 2) restore the 'toolbar' group button which had rename and delete options in > it and label it 'modify' Wouldn't it be confused with the other 'modify' button on the same dialog? > 3) vertically center the toolbar command buttons and the move up and move > down buttons with the commands listbox Sure. > 4) replace the new visible separators with margin or padding, similar to how > it is in the keyboard tab In the keyboard tab, it is done by grouping the buttons as 'primary' and 'secondary'. I couldn't do it the same way because we have three groups. I'll try to find a way to align them properly.
(In reply to Muhammet Kara from comment #12) > > 2) restore the 'toolbar' group button which had rename and delete options in > > it and label it 'modify' > > Wouldn't it be confused with the other 'modify' button on the same dialog? They are in two separate sections, so not a problem. > > 4) replace the new visible separators with margin or padding, similar to how > > it is in the keyboard tab > > In the keyboard tab, it is done by grouping the buttons as 'primary' and > 'secondary'. I couldn't do it the same way because we have three groups. > I'll try to find a way to align them properly. Margins or padding should be sufficient enough to achieve the same look, but we dont need visible separators.
Muhammet Kara committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=4aca836a468bb0e2a7aae0800164e2b81f96243f tdf#82840 Add 'Reset' button to toolbar customization It will be available in 5.3.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
seems ready to close
As modifying one of the tabs, modifies all the tabs, it would also be useful to move the 'add submenu' entry out of the modify button in the menus and context menus tabs. Is there a reason why 'restore default command' is only available in the toolbars tab?
Muhammet Kara committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=3b3feeff98385bccce14b83bf32f079f624550c6 tdf#82840 Add 'Add Separator' button to toolbar customization It will be available in 5.3.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Muhammet Kara committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=58abb0fd301c235ea3f8d2a00be299ca0e9c7a8e tdf#82840 Add radio buttons for toolbar styles It will be available in 5.3.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.