On Wed, Sep 06, 2006 at 11:29:58PM +0200, Michael Gerz wrote:
> John, JMarc,
> 
> what do you think about the patch below? The idea is to have a submenu 
> for character styles that combines layout-specific char styles, 
> bold/emphasize/noun style, and the font dialog. The layout-specific char 
> styles have been moved from menu "Insert". Although the styles are 
> actually implemented as insets, the user does not think in terms of 
> "inserting" something. Bold/emphasize/noun style - whether you like it 
> or not - are frequently requested features. There is at least one 
> bugzilla report for re-adding "bold" to the menu. The font dialog is 
> listed at the end ("Other...") because users should not be encouraged to 
> use it often. Regarding the word-* functions:  We might move them to the 
> submenu as well to shorten the "Edit" menu (as done in the patch). 
> However, after testing "word-capitalize", I really doubt that this 
> particular functions is very useful (why do we have it at all?).
> 
> Any comments?
> 
> Michael

 
This looks very nice to me.

- Martin
 
> Index: stdmenus.ui
> ===================================================================
> --- stdmenus.ui    (revision 14919)
> +++ stdmenus.ui    (working copy)
> @@ -89,14 +89,10 @@
>         Separator
>         Item "Find & Replace...|F" "dialog-show findreplace"
>         Separator
> -        Item "Word in Small Capitals" "word-capitalize"
> -        Item "Word in Uppercase" "word-upcase"
> -        Item "Word in Lowercase" "word-lowcase"
> -        Separator
>         Item "Move Paragraph Up|o" "paragraph-move-up"
>         Item "Move Paragraph Down|v" "paragraph-move-down"
>         Separator
> -        Item "Text Style...|S" "dialog-show character"
> +        Submenu "Character Style...|C" "insert_charstyles"
>         Item "Paragraph Settings...|P" "layout-paragraph"
>         Separator
> # Mathed b0rkage means these don't work properly
> @@ -136,6 +132,19 @@
>         PasteRecent
>     End
> 
> +    Menu "insert_charstyles"
> +        CharStyles
> +        Separator
> +        Item "Emphasize Style|E" "font-emph"
> +        Item "Noun Style|N" "font-noun"
> +        Item "Bold Style|B" "font-bold"
> +        Separator
> +        Item "Other...|O" "dialog-show character"
> +        Separator
> +        Item "Uppercase" "word-upcase"
> +        Item "Lowercase" "word-lowcase"
> +    End
> +
> # not much we can do to help here
>     Menu "edit_tabular"
>         Item "Multicolumn|M" "tabular-feature multicolumn"
> @@ -253,7 +262,6 @@
> # YUCK
>         Submenu "Note|N" "insert_note"
>         OptSubmenu "Branch|B" "insert_branches"
> -        OptSubmenu "Character Style|y" "insert_charstyles"
>         Submenu "File|e" "insert_file"
>         Item "Box" "box-insert Frameless"
>         Separator
> @@ -377,10 +385,6 @@
>         Branches
>     End
> 
> -    Menu "insert_charstyles"
> -        CharStyles
> -    End
> -
> #
> # DOCUMENT MENU
> #
> 

Attachment: pgpH4y6x618o6.pgp
Description: PGP signature

Reply via email to