By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.
Bug 251 - Kicker: has no configuration option to stop scrolling among apps
Summary: Kicker: has no configuration option to stop scrolling among apps
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: tdebase (show other bugs)
Version: 3.5.12 [Trinity]
Hardware: Other All
: P1 major
Assignee: Calvin Morrison
URL:
Depends on:
Blocks:
 
Reported: 2010-08-30 14:37 CDT by Darrell
Modified: 2012-10-19 15:37 CDT (History)
2 users (show)

See Also:
Compiler Version:
TDE Version String: 3.5.10
Application Version:
Application Name: kicker/panel

Attachments
A patch to implement a kcm option to solve the issue (4.24 KB, text/x-diff)
2012-03-10 14:42 CST, Calvin Morrison 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Darrell 2010-08-30 14:37:53 CDT 
Sometimes I do not pay attention to the location of the mouse pointer when I start scrolling with the mouse wheel. :)

When the mouse pointer is located over an application, then as expected the application is affected by the mouse wheel scrolling.

When I happen not to pay attention that the mouse pointer is hovering over the KDE kicker/panel, then the mouse wheel scrolling has the effect of paging/scrolling/cycling through all open applications. Hardly fatal, but distracting.

Recommended solution is to provide a configuration option to stop kicker/panel from cycling through the apps.
Comment 1 Darrell 2010-09-20 18:07:38 CDT 
Similarly, if the mouse pointer is hovering over the desktop pager in the panel, the mouse scrolls through each virtual desktop. Therefore an option to disable scrolling through the apps should disable scrolling through the desktops too. Or create two options to control both actions.
Comment 2 Darrell 2012-03-10 13:23:22 CST 
Not to forget that hovering the mouse pointer over the Konqueror tab bar and moving the scroll wheel then scrolls through the tabs.

I see the usefulness for these options, but like so much we discuss, everybody has their own work flow style. These automatic scrolling behaviors frustrate me much. I don't want any scrolling at all in any of these areas.

To satisfy both types of users, the only solution is to add KControl check boxes.

I don't know whether one check box should control all scrolling or one check box should be added for each area (panel, virtual desktops, Konqueror tabs, etc.)
Comment 3 Calvin Morrison 2012-03-10 13:26:45 CST 
I think a seperate option for each would be favored. I tend to rely on twin 
scrolling, and I don't like the scrolling for kicker... I think other people 
might have similar feelings.

I am having a lot of trouble with the configuration bits. Not with the actual 
coding (pfft it was just one if statement) but with getting it to save and load 
in my kcontrol module.

Sometimes I shake my head at TDE. A lot of this seems to complicated for it's 
own good :p
Comment 4 Calvin Morrison 2012-03-10 14:42:31 CST 
Okay, 

I have just uploaded a patch to solve the problem for the taskbar ONLY.

Please report NEW enhancement requests for other applications that require 
similar options. 

Calvin
Comment 5 Calvin Morrison 2012-03-10 14:42:36 CST 
Created attachment 477 [details]
A patch to implement a kcm option to solve the issue
Comment 6 Darrell 2012-03-10 17:43:46 CST 
Woohoo! By golly Calvin! Works great with an existing and new profile. Setting is persistent across sessions.

One note: the text string has a period at the end and none of the other strings do. I fixed my copy of the patch so don't worry about that. :)

How much work do you think to apply the same fix to virtual desktops and the Konqueror tab bar?

Thanks!

Tim, should I push this patch?
Comment 7 Darrell 2012-03-10 21:39:29 CST 
One more candidate: Konsole tab bar.
Comment 8 Calvin Morrison 2012-03-11 11:08:47 CDT 
If it works I say push :-)
Comment 9 Calvin Morrison 2012-03-11 11:41:41 CDT 
seperate bugs would make it easier to keep track of! :-)
Comment 10 Darrell 2012-03-11 13:53:56 CDT 
Bug reports 908 (Desktop Pager), 909 (Konsole), and 910 (Konqueror) submitted! Hopefully you can clone the patch here to those bug reports. :)
Comment 11 Darrell 2012-03-13 11:21:25 CDT 
Patch merged in GIT hash 14b7d88851378ac5ee85198634a984a665668237.

Thank you for the patch!