From kfm-devel Wed Sep 04 03:23:29 2002 From: Doug Hanley Date: Wed, 04 Sep 2002 03:23:29 +0000 To: kfm-devel Subject: Re: Right-Click Tab Behavior X-MARC-Message: https://marc.info/?l=kfm-devel&m=103110978830346 On Monday 02 September 2002 05:56 am, Luk=E1=B9 Tinkl wrote: > Dne po 2. z=E1=F8=ED 2002 11:31 David Faure napsal(a): > > On Sunday 01 September 2002 23:17, Doug Hanley wrote: > > > Hi, > > > > > > Regarding that last commit to konq_frame.cc about 4 days ago by lukas, > > > I noticed that when you try and use "Duplicate Current Tab", "Detatch > > > Current Tab", or "Remove Current Tab" it yields undesireable behavior > > > because it removes the currently active tab, not the tab you > > > right-clicked on. I think it was better the old way because with the > > > new way, there is no way to just click on a tab and say "remove this > > > tab", you have to activate it then do remove current tab. To test > > > this, just get a Konq window, create a new tab, then click on an > > > inactive tab and do remove current tab. It will remove the active ta= b, > > > not the tab you clicked on. Personally, I think this commit should be > > > backed out. > > > > ... or fixed. > > > > Cc'ing Lukas, I'm not sure he's on kfm-devel. > > OK, should I revert my change? I think it would be best to back it out first, then recommit when everythin= g=20 works properly. I mean, I see what you're trying to do, add some continuity to the interfac= e=20 without much code duplication, but I think in this case you're gonna have t= o=20 it differently, because it brings on undesired results. I think if you add= =20 in the icons and accels manually but still using the standard QPopupMenu=20 calls, it would work out very well. =2D- Doug