Bug 168223 - Unintuitive web-shortcut checkbox functionality
Summary: Unintuitive web-shortcut checkbox functionality
Status: RESOLVED FIXED
Alias: None
Product: konqueror
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Ubuntu Linux
: NOR wishlist
Target Milestone: ---
Assignee: Konqueror Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-08-03 16:26 UTC by Mel Collins
Modified: 2012-05-09 22:36 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In: 4.9.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Mel Collins 2008-08-03 16:26:27 UTC
Version:            (using KDE 3.5.9)
Installed from:    Ubuntu Packages

As implied by the presence of bug 122879, the checkbox that appears next to each entry in the "web shortcuts" configuration panel has an unintuitive function (ie. marking it's presence in a single, arguably unrelated, contextual right-click menu). The actual function of these boxes is not immediately apparent either, requiring either viewing the documentation or making use of the '?' button in the window decorations.
The logical functionality of checkboxes in this position would be a straight activate/deactivate of the entry. This is the case with pretty much all other similar GUI layouts I have ever encountered, even in Konqueror itself: "Previews & Meta-Data" and "Crypto" configuration panels, for example.

IMO the "add to menu" checkboxes should be in their own, suitably headed, column in the table at the very least. Certainly not the first column, though!

As a slight aside: Given that the context menu in question is only really tangentially related to location-bar shortcuts, I can see no real reason why they should be so closely joined in the configuration GUI anyway. There is no way to have a context-menu entry without a location-bar shortcut sequence, for example.
Comment 1 Dawit Alemayehu 2012-05-09 22:36:51 UTC
Git commit 0707b07d3afb14870d5c31149238e0f32e0d1187 by Dawit Alemayehu.
Committed on 07/05/2012 at 09:36.
Pushed by adawit into branch 'master'.

- Moved the checkbox used to select preferred shortcuts into its own column to
  prevent user confusion.

- Allow the shortcut list to be filtered using the shortcut texts.
Related: bug 218164
FIXED-IN: 4.9.0
REVIEW: 104900
GUI: New translatable text

M  +37   -21   kurifilter-plugins/ikws/ikwsopts.cpp
M  +5    -5    kurifilter-plugins/ikws/ikwsopts_p.h
M  +1    -1    kurifilter-plugins/ikws/ikwsopts_ui.ui

http://commits.kde.org/kde-runtime/0707b07d3afb14870d5c31149238e0f32e0d1187