<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>276872</bug_id>
          
          <creation_ts>2011-07-01 07:51:45 +0000</creation_ts>
          <short_desc>New tab should be placed right after current tab, not at the end.</short_desc>
          <delta_ts>2011-09-27 21:15:13 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>2</classification_id>
          <classification>Applications</classification>
          <product>konsole</product>
          <component>general</component>
          <version>2.7.999</version>
          <rep_platform>Arch Linux</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>NOR</priority>
          <bug_severity>wishlist</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Jekyll Wu">adaptee</reporter>
          <assigned_to name="Konsole Bugs">konsole-bugs-null</assigned_to>
          <cc>ivo</cc>
          
          <cf_commitlink></cf_commitlink>
          <cf_versionfixedin>4.8</cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1137301</commentid>
    <comment_count>0</comment_count>
      <attachid>61521</attachid>
    <who name="Jekyll Wu">adaptee</who>
    <bug_when>2011-07-01 07:51:45 +0000</bug_when>
    <thetext>Created attachment 61521
put the new tab right after current tab

Version:           2.7.999 (using KDE 4.6.4) 
OS:                Linux

Konsole provides the option of &apos;start in same directory as current tab&apos;. This
is nice, but konsole always put new tab at the end of tab list, which make this
nice option not as useful as it appears.

For example, I have 5 tabs open. In tab #2 I am compiling some code and I
suddenly have a nice idea and want to grep something, So I open a new tab.
The &apos;grep&apos; tab start at the same directory as the &apos;compilation&apos; tab , but is
placed at the end(#6). After I finish greping and want to switch back to the
&apos;compilation&apos; tab(#2), I have 3 choices:

  a). Use mouse to click tab #2, which is slow compared with keyboard.
  b). Press &apos;Shift+Left&apos; 4 times
  c). Press &apos;Ctr+Shift+Alt&apos; 3 times to move the &apos;grep&apos; tab to place #3, then
      press &apos;Shift+left&apos;

Has the &apos;grep&apos; tab been placed right after the &apos;compilation&apos; tab, at #3, I
would only need to press &apos;Shift+left&apos; one time to switch the focus .

So this example shows that placing new tab right after current tab can make
(some) users&apos; life much easier. And I think that&apos;s the natural result following
the &apos;start in same directory&apos; option: users expect tabs doing related work to be
placed near each other.

The attached patch is a quick hack. It always puts new tab right after current
tab, regardless of whether &apos;start in same directory&apos; is enabled or not. 


Reproducible: Always

Steps to Reproduce:
1. open 5 tabs in konsole
2. change their title to #1,#2,#3,#4,#5
3. switch focus to #2
4. open a new tab


Actual Results:  
the new tab is placed after #5

Expected Results:  
the new tab is placed after #2</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1147530</commentid>
    <comment_count>1</comment_count>
    <who name="Kurt Hindenburg">khindenburg</who>
    <bug_when>2011-07-31 02:38:47 +0000</bug_when>
    <thetext>Git commit 676df0c26e5c01652da64690756b74fbe180a080 by Kurt Hindenburg.
Committed on 31/07/2011 at 04:17.
Pushed by hindenburg into branch &apos;master&apos;.

New tabs will be placed to the right of the current tab.

Currently new tabs are placed at the end which is not very convenient.

Patch by Jekyll Wu &lt;adaptee@gmail.com&gt;
FEATURE: 276872
FIXED-IN: 4.8

M  +11   -1    src/ViewManager.cpp

http://commits.kde.org/konsole/676df0c26e5c01652da64690756b74fbe180a080</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1153532</commentid>
    <comment_count>2</comment_count>
    <who name="Ivo Anjo">ivo</who>
    <bug_when>2011-08-13 15:06:20 +0000</bug_when>
    <thetext>Can this at least be configurable?
All KDE applications I&apos;ve tried do not do this (dolphin, konqueror, kate, k3b, ktorrent), nor does gnome-terminal.
Firefox does it, but offers an option to disable (which I use).

Please consider that this is quite a big change, and lots of users (like me) have used the old behavior for years.

Thanks.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1153853</commentid>
    <comment_count>3</comment_count>
    <who name="Jekyll Wu">adaptee</who>
    <bug_when>2011-08-15 04:05:27 +0000</bug_when>
    <thetext>(In reply to comment #2)
&gt; Can this at least be configurable? 

I will try to make it configurable before it is shipped in 4.8.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1167363</commentid>
    <comment_count>4</comment_count>
    <who name="Jekyll Wu">adaptee</who>
    <bug_when>2011-09-26 22:46:57 +0000</bug_when>
    <thetext>Git commit acd37f7a739ffac0863b2580c775b0956c45baa8 by Jekyll Wu.
Committed on 27/09/2011 at 00:43.
Pushed by jekyllwu into branch &apos;master&apos;.

Allow users to control where the newly created tab should be placed.

CCBUG:276872
REVIEW:102685

M  +13   -1    src/EditProfileDialog.cpp
M  +1    -0    src/EditProfileDialog.h
M  +21   -1    src/EditProfileDialog.ui
M  +2    -0    src/Profile.cpp
M  +13   -0    src/Profile.h
M  +15   -6    src/ViewManager.cpp

http://commits.kde.org/konsole/acd37f7a739ffac0863b2580c775b0956c45baa8</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1167702</commentid>
    <comment_count>5</comment_count>
    <who name="Ivo Anjo">ivo</who>
    <bug_when>2011-09-27 21:15:13 +0000</bug_when>
    <thetext>Thanks a lot for looking into this! Cheers :)</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>61521</attachid>
            <date>2011-07-01 07:51:45 +0000</date>
            <delta_ts>2011-07-01 07:51:45 +0000</delta_ts>
            <desc>put the new tab right after current tab</desc>
            <filename>0001-put-new-tab-after-current-tab.patch</filename>
            <type>text/plain</type>
            <size>983</size>
            <attacher name="Jekyll Wu">adaptee</attacher>
            
              <data encoding="base64">RnJvbSBhMjJkM2UyZmM2NGY2ZDU5MzllY2M2YjAyOGIwZTk3YzM1NDY3MTliIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBKZWt5bGwgV3UgPGFkYXB0ZWVAZ21haWwuY29tPgpEYXRlOiBG
cmksIDEgSnVsIDIwMTEgMTU6Mzk6MDcgKzA4MDAKU3ViamVjdDogW1BBVENIXSBwdXQgbmV3IHRh
YiBhZnRlciBjdXJyZW50IHRhYgoKLS0tCiBzcmMvVmlld01hbmFnZXIuY3BwIHwgICAxMyArKysr
KysrKysrKystCiAxIGZpbGVzIGNoYW5nZWQsIDEyIGluc2VydGlvbnMoKyksIDEgZGVsZXRpb25z
KC0pCgpkaWZmIC0tZ2l0IGEvc3JjL1ZpZXdNYW5hZ2VyLmNwcCBiL3NyYy9WaWV3TWFuYWdlci5j
cHAKaW5kZXggMGFhMjRkNS4uNzVhNWY2NyAxMDA2NDQKLS0tIGEvc3JjL1ZpZXdNYW5hZ2VyLmNw
cAorKysgYi9zcmMvVmlld01hbmFnZXIuY3BwCkBAIC02MTAsNyArNjEwLDE4IEBAIHZvaWQgVmll
d01hbmFnZXI6OmNyZWF0ZVZpZXcoU2Vzc2lvbiogc2Vzc2lvbikKICAgICB3aGlsZSAoIGNvbnRh
aW5lckl0ZXIuaGFzTmV4dCgpICkKICAgICB7CiAgICAgICAgIFZpZXdDb250YWluZXIqIGNvbnRh
aW5lciA9IGNvbnRhaW5lckl0ZXIubmV4dCgpOwotICAgICAgICBjcmVhdGVWaWV3KHNlc3Npb24s
Y29udGFpbmVyLC0xKTsKKworICAgICAgICBpbnQgaW5kZXggICAgID0gLTE7CisgICAgICAgIFFX
aWRnZXQqIHZpZXcgPSBhY3RpdmVWaWV3KCk7CisKKyAgICAgICAgaWYgKHZpZXcpCisgICAgICAg
IHsKKyAgICAgICAgICAgIFFMaXN0PFFXaWRnZXQqPiB2aWV3cyA9IGNvbnRhaW5lci0+dmlld3Mo
KSA7CisgICAgICAgICAgICBjb25zdCBpbnQgY3VycmVudEluZGV4ID0gdmlld3MuaW5kZXhPZih2
aWV3KSA7CisgICAgICAgICAgICBpbmRleCA9IGN1cnJlbnRJbmRleCArMSA7CisgICAgICAgIH0K
KworICAgICAgICBjcmVhdGVWaWV3KHNlc3Npb24sY29udGFpbmVyLGluZGV4KTsKICAgICB9CiAK
IH0KLS0gCjEuNy42Cgo=
</data>

          </attachment>
      

    </bug>

</bugzilla>