[sugar] [PATCH] Change tabbing to show a preview by opening the frame
Hello, Some patches that implement nicer tabbing will follow. With the patches applied, pressing +Tab will show the frame. The activity that will be selected as soon as is released again is shown by popping up its palette. Unfortunately there are a couple of issues with palettes. One issue is that that at first only the primary palette is shown, but after a while the secondary palette will pop up (as the timeout is started automatically). Then I am accessing the _update_position function of the Palette directly, to handle disappearing activities properly. The third thing I noticed is that there is a race condition in the Palette (which I think is my fault ...). I'll need to have a look at how to fix this. (The reason for the race is the code that delays the "popup" signal until the window has been mapped. IIRC that was needed so the gab drawing would work properly.) Benjamin signature.asc Description: This is a digitally signed message part ___ Sugar mailing list Sugar@lists.laptop.org http://lists.laptop.org/listinfo/sugar
Re: [sugar] [PATCH] Change tabbing to show a preview by opening the frame
Hi Benjamin, On Thu, Jun 5, 2008 at 12:20 AM, Benjamin Berg <[EMAIL PROTECTED]> wrote: > Hello, > > Some patches that implement nicer tabbing will follow. With the patches > applied, pressing +Tab will show the frame. The activity that will > be selected as soon as is released again is shown by popping up > its palette. Eben, could you apply these patches to your jhbuild and comment on the user experience? > Unfortunately there are a couple of issues with palettes. > > One issue is that that at first only the primary palette is shown, but > after a while the secondary palette will pop up (as the timeout is > started automatically). Can you look at how the secondary palette is displayed by the right click? I understand that's what you want, right? > Then I am accessing the _update_position function of the Palette > directly, to handle disappearing activities properly. You mean to move left the palette when another activity icon disappears? Perhaps the Invoker should be listening for position changes on the observed widget and move the palette accordingly? > The third thing I noticed is that there is a race condition in the > Palette (which I think is my fault ...). I'll need to have a look at how > to fix this. > (The reason for the race is the code that delays the "popup" signal > until the window has been mapped. IIRC that was needed so the gab > drawing would work properly.) Ok. Before looking in detail at the code, have you considered relying on the window manager to do the tabbing and have the shell to just observe the changes on the tabbing stack? Perhaps wnck can help on that? Although I could find some time this week to look at these patches, I would be happier if Marco could find time do do it, as he knows better about these window manager stuff. I know he's very busy, though... Thanks, Tomeu ___ Sugar mailing list Sugar@lists.laptop.org http://lists.laptop.org/listinfo/sugar
Re: [sugar] [PATCH] Change tabbing to show a preview by opening the frame
On Tue, Jun 10, 2008 at 9:58 AM, Tomeu Vizoso <[EMAIL PROTECTED]> wrote: > Although I could find some time this week to look at these patches, I > would be happier if Marco could find time do do it, as he knows better > about these window manager stuff. I know he's very busy, though... I could do an high level review, and then you guys help me out with the details... Marco ___ Sugar mailing list Sugar@lists.laptop.org http://lists.laptop.org/listinfo/sugar
Re: [sugar] [PATCH] Change tabbing to show a preview by opening the frame
On Tue, Jun 10, 2008 at 10:56 AM, Marco Pesenti Gritti <[EMAIL PROTECTED]> wrote: > On Tue, Jun 10, 2008 at 9:58 AM, Tomeu Vizoso <[EMAIL PROTECTED]> wrote: >> Although I could find some time this week to look at these patches, I >> would be happier if Marco could find time do do it, as he knows better >> about these window manager stuff. I know he's very busy, though... > > I could do an high level review, and then you guys help me out with > the details... Ok, seems like a good deal to me. Thanks, Tomeu ___ Sugar mailing list Sugar@lists.laptop.org http://lists.laptop.org/listinfo/sugar
Re: [sugar] [PATCH] Change tabbing to show a preview by opening the frame
On Tue, 2008-06-10 at 09:58 +0200, Tomeu Vizoso wrote: > On Thu, Jun 5, 2008 at 12:20 AM, Benjamin Berg > > Unfortunately there are a couple of issues with palettes. > > > > One issue is that that at first only the primary palette is shown, but > > after a while the secondary palette will pop up (as the timeout is > > started automatically). > > Can you look at how the secondary palette is displayed by the right > click? I understand that's what you want, right? Yup, something like that. Though it will need to be handled differently than the right click code. I guess just adding another parameter to popdown will work. > > Then I am accessing the _update_position function of the Palette > > directly, to handle disappearing activities properly. > > You mean to move left the palette when another activity icon > disappears? Perhaps the Invoker should be listening for position > changes on the observed widget and move the palette accordingly? Yeah, something like that. Though I am not sure what is needed to get it working right now. > Before looking in detail at the code, have you considered relying on > the window manager to do the tabbing and have the shell to just > observe the changes on the tabbing stack? Perhaps wnck can help on > that? To say the truth I did not even consider it, as tabbing is already handled by Sugar in the existing code. That said, I just had a quick look, but could not find anything in metacity or libwnck that suggests that leaving tabbing to the window manager is possible. Benjamin signature.asc Description: This is a digitally signed message part ___ Sugar mailing list Sugar@lists.laptop.org http://lists.laptop.org/listinfo/sugar
Re: [sugar] [PATCH] Change tabbing to show a preview by opening the frame
I had a quick look to the patches and the approach looks good to me. I asked Eben to comment on the UI side of this before we make a more careful review. Would be nice to get this in by the feature freeze. Marco ___ Sugar mailing list Sugar@lists.laptop.org http://lists.laptop.org/listinfo/sugar