Re: runtime accel changes

2012-09-15 Thread Tristan Van Berkom
On Sun, Sep 16, 2012 at 8:05 AM, Benjamin Otte wrote: > Michael Natterer gimp.org> writes: > >> I think my actual point here is: There are so many usecases >> and levels of complexity in peoples' workflows, and >> in the applications that can be written in GTK+, we should >> not disable useful th

Re: runtime accel changes

2012-09-15 Thread Benjamin Otte
Michael Natterer gimp.org> writes: > I think my actual point here is: There are so many usecases > and levels of complexity in peoples' workflows, and > in the applications that can be written in GTK+, we should > not disable useful things because they are either not in > fashion any longer accor

Re: runtime accel changes

2012-09-14 Thread Philip Chimento
On Thu, Sep 13, 2012 at 11:01 PM, Paul Davis wrote: > > On Thu, Sep 13, 2012 at 4:43 PM, Juan Pablo Ugarte > wrote: >> >> On Thu, 2012-09-13 at 06:22 +, Tristan Van Berkom wrote: >> > From what I see... all one should *need* to do is specify the >> > accelerator keys >> > desired to trigger a

Re: runtime accel changes

2012-09-13 Thread Paul Davis
On Thu, Sep 13, 2012 at 4:43 PM, Juan Pablo Ugarte < juanpablouga...@gmail.com> wrote: > On Thu, 2012-09-13 at 06:22 +, Tristan Van Berkom wrote: > > From what I see... all one should *need* to do is specify the > accelerator keys > > desired to trigger a given GtkAction and add that action to

Re: runtime accel changes

2012-09-13 Thread Juan Pablo Ugarte
On Thu, 2012-09-13 at 06:22 +, Tristan Van Berkom wrote: > From what I see... all one should *need* to do is specify the accelerator keys > desired to trigger a given GtkAction and add that action to an action group > (either with GtkBuilder constructs or with > gtk_action_group_add_action_with

Re: runtime accel changes

2012-09-13 Thread Phil Clayton
On 13/09/12 14:20, Ryan Lortie wrote: Is there a usecase for users who regularly switch keybindings as part of their normal workflow, or is this a sort of "customise the app once, then use it forever" type of situation? If you sit down at someone else's computer it is useful to be able to swit

Re: runtime accel changes

2012-09-13 Thread Michael Natterer
On Thu, 2012-09-13 at 09:20 -0400, Ryan Lortie wrote: > hi, > > Thanks for the feedback. > > On 12-09-12 05:15 PM, Paul Davis wrote: > > On Wed, Sep 12, 2012 at 5:07 PM, Michael Natterer wrote: > >> I object. Do you really want to disable shortcut editors in > >> complex applications such as GIM

Re: runtime accel changes

2012-09-13 Thread Paul Davis
On Thu, Sep 13, 2012 at 9:20 AM, Ryan Lortie wrote: > hi, > > Thanks for the feedback. > > > On 12-09-12 05:15 PM, Paul Davis wrote: > >> On Wed, Sep 12, 2012 at 5:07 PM, Michael Natterer wrote: >> >>> I object. Do you really want to disable shortcut editors in >>> complex applications such as G

Re: runtime accel changes

2012-09-13 Thread Ryan Lortie
hi, Thanks for the feedback. On 12-09-12 05:15 PM, Paul Davis wrote: On Wed, Sep 12, 2012 at 5:07 PM, Michael Natterer wrote: I object. Do you really want to disable shortcut editors in complex applications such as GIMP? I echo Mitch's objections. Ardour users find it very convenient to do

Re: runtime accel changes

2012-09-12 Thread Tristan Van Berkom
On Wed, Sep 12, 2012 at 9:15 PM, Paul Davis wrote: > > > On Wed, Sep 12, 2012 at 5:07 PM, Michael Natterer wrote: >> >> On Wed, 2012-09-12 at 16:27 -0400, Ryan Lortie wrote: >> > hi, >> > >> > I recently wrote a patch[1] to re-enable accel labels in GtkMenu >> > generated from GMenuModel. They g

Re: runtime accel changes

2012-09-12 Thread Paul Davis
On Wed, Sep 12, 2012 at 5:07 PM, Michael Natterer wrote: > On Wed, 2012-09-12 at 16:27 -0400, Ryan Lortie wrote: > > hi, > > > > I recently wrote a patch[1] to re-enable accel labels in GtkMenu > > generated from GMenuModel. They got lost in the shuffle during some > > related recent changes. >

Re: runtime accel changes

2012-09-12 Thread Michael Natterer
On Wed, 2012-09-12 at 16:27 -0400, Ryan Lortie wrote: > hi, > > I recently wrote a patch[1] to re-enable accel labels in GtkMenu > generated from GMenuModel. They got lost in the shuffle during some > related recent changes. > > Essentially, the new approach means that the accel='' attribute o

runtime accel changes

2012-09-12 Thread Ryan Lortie
hi, I recently wrote a patch[1] to re-enable accel labels in GtkMenu generated from GMenuModel. They got lost in the shuffle during some related recent changes. Essentially, the new approach means that the accel='' attribute of each menuitem directly determines what the accel label will be