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 1682 - Konsole: Tab popup context menu does not include Move Session Left/Right options
Summary: Konsole: Tab popup context menu does not include Move Session Left/Right options
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: tdebase (show other bugs)
Version: R14.0.0 [Trinity]
Hardware: Other Linux
: P5 minor
Assignee: Michele Calgaro
URL:
Depends on:
Blocks:
 
Reported: 2013-10-18 14:08 CDT by Darrell
Modified: 2013-10-23 14:09 CDT (History)
4 users (show)

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


Attachments
Konsole tab popup menu "move session left/right " patch (8.34 KB, patch)
2013-10-19 09:25 CDT, Michele Calgaro
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Darrell 2013-10-18 14:08:17 CDT
The Konsole View menu includes options to move a tab session to the left or right of the current position in the tab bar. These options do not exist in the tab popup context menu.

Refer to bug report 1067 for similar changes in Konqueror.
Comment 1 Michele Calgaro 2013-10-18 14:50:36 CDT
I will look into this bug as well, sometimes next week.
Comment 2 Michele Calgaro 2013-10-19 09:25:54 CDT
Created attachment 1561 [details]
Konsole tab popup menu "move session left/right " patch

Attached proposed patch.
If applied after the patch for bug 909, there should be no fuzzs or offsets
Comment 3 Michele Calgaro 2013-10-19 09:26:41 CDT
Changed status to PATCHAVAIL
Comment 4 Slávek Banko 2013-10-19 17:56:48 CDT
Pushed to GIT in hash 1041337.
Comment 5 Darrell 2013-10-23 10:06:13 CDT
Patch looks great here. Sorry I could not test earlier.
Comment 6 Slávek Banko 2013-10-23 10:08:53 CDT
(Odpověď na komentář #5)
> Patch looks great here. Sorry I could not test earlier.

I have tested it and pushed :)
Comment 7 Darrell 2013-10-23 14:09:54 CDT
> I have tested it and pushed 
Yes, I saw that. :-) I was only adding a comment because I opened the bug report but was busy the past few days and could not test.