Re: View / Preview Pane accelerator (was: Re: [Evolution] I type a single-quote and get an acute accent)

[Patrick O'Callaghan <poc usb ve>]

On Mon, 2005-01-10 at 18:33 +0100, guenther wrote:
> On Mon, 2005-01-10 at 12:53 -0400, Patrick O'Callaghan wrote:
> > On Fri, 2005-01-07 at 17:23 +0100, guenther wrote:
> > > The View / Preview Pane accelerator is defined in your
> > > $prefix/share/evolution/2.0/ui/evolution-mail-global.xml file. It's
> > > this
> > > chunk:
> > >
> > >     <cmd name="ViewPreview"
> > >      _tip="Show message preview window"
> > >      accel="*Control*grave"
> > >      type="toggle"/>
> > >
> > > Change the accel attribute to suite your needs and keyboard.
> >
> > OK, I got this to work after several tries.
>
> Great. :)
>
>
> > The syntax for the accel
> > value is not entirely obvious: I finally settled on Ctrl+Alt+P but when
> > I tried setting *Control*Alt*p (by analogy with *Control*Alt*s which is
> > used for "Save As ...") it didn't work. However it did work with
> > *Control**Alt*p. Is this a feature?
>
> No, a bug. :-/
>
> $ grep Control *.xml | grep Alt | tail -2
> evolution-comp-editor.xml:        accel="*Control*Alt*s"
> evolution-mail-message.xml:     accel="*Control**Alt*s"
>
> $ egrep "\*Control\*\*(Alt|Shift)\*" *.xml | wc -l
> 17
> $ egrep "\*Control\*(Alt|Shift)\*" *.xml | wc -l
> 1
>
> Unlucky you used the only buggy accelerator setting as sample. That
> particular one doesn't even work. Each modifier needs to be surrounded
> by asterisks, so *Control**Alt*p is correct.

I suspected as much, just didn't like to shout BUG without checking.

poc