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 245 - Konqueror: Restore context menu options
Summary: Konqueror: Restore context menu options
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: tdebase (show other bugs)
Version: 3.5.11 [Trinity]
Hardware: Other All
: P5 enhancement
Assignee: Timothy Pearson
URL:
Depends on:
Blocks:
 
Reported: 2010-08-25 14:51 CDT by Darrell
Modified: 2012-10-19 15:46 CDT (History)
3 users (show)

See Also:
Compiler Version:
TDE Version String: 3.5.10
Application Version:
Application Name: konqueror

Attachments
Restores the context menu option 'Open in Background Tab' (8.63 KB, patch)
2011-12-03 18:04 CST, Darrell 		Details | Diff
New patch that keeps configurable behaviour (3.99 KB, patch)
2012-01-12 17:36 CST, Timothy Pearson 		Details | Diff
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-25 14:51:23 CDT 
Once upon a time the Konqueror web link context menu provided both the option to open a web link in a foreground tab or a background tab. That option was removed and the context menu now shows only the preferred option selected in the Settings. The history of the the refusal to restore the options can be read at bug 63706 (http://bugs.kde.org/show_bug.cgi?id=63706).

This removal is inconsistent with the design of many other web browsers that provide both options in the context menu.

The option was removed somewhere after KDE 3.1.3.

Historical screen capture is available here (http://humanreadable.nfshost.com/images/context-menu-konq313.png).

Request is to restore both options to the context menu.
Comment 1 Ilya 2011-11-04 07:24:03 CDT 
What's up with this bug?
Comment 2 Ilya 2011-11-04 07:24:47 CDT 
sorry wrong bug number
Comment 3 Darrell 2011-11-29 19:52:35 CST 
The original revision when the context menu was modified:

http://websvn.kde.org/?view=revision&revision=246742
Comment 4 Darrell 2011-12-03 18:04:41 CST 
Created attachment 208 [details]
Restores the context menu option 'Open in Background Tab'

Patch uploaded to restore the old context menu! Like this:

Open in Background Tab
Open in New Tab

The patch includes removing from the Konqueror Settings, Web Behavior, Advanced Options, the "Open new tabs in the background" option because that option no longer is needed or serves a purpose.

Woo hoo!
Comment 5 Timothy Pearson 2012-01-11 18:00:20 CST 
(In reply to comment #4)
> The patch includes removing from the Konqueror Settings, Web Behavior, Advanced
> Options, the "Open new tabs in the background" option because that option no
> longer is needed or serves a purpose.

Are you sure that it no longer serves a purpose?  It seems to me that it would control the default new tab behaviour...
Comment 6 Darrell 2012-01-11 18:25:36 CST 
I tested that option before removing. The check box had no effect. Of course, perhaps somebody else can see what I could not and the option could still be useful. I only can say that I saw no effect with the older menu restoration. I'm not offended if you want somebody else to test further. :)
Comment 7 Timothy Pearson 2012-01-12 17:36:58 CST 
Created attachment 270 [details]
New patch that keeps configurable behaviour

Try this patch; it should keep the configuration option active.
Comment 8 Darrell 2012-01-12 18:16:27 CST 
Does the check box do anything? Under the new menu structure, there is no default opening scheme. The web link context menu popup controls how new tabs are opened, either in the foreground or background.

The check box has no effect on web page links, but does the option make sense when in file management mode? I don't know. Might be a day or two before I can test.
Comment 9 Darrell 2012-03-07 18:38:12 CST 
Final patch merged in GIT hash 769054446b8e43589e94869240e8642e5b93e9df.
This resolves this enhancement request.