Restore new tab behavior
ClosedPublic

Authored by roberts on Sep 29 2018, 8:48 AM.

Details

Summary

As part of the QTabBar refactoring, honouring of the new tab behaviour
setting was lost. Restore using a simpler implementation that simply
asks QTabWidget for the current index.

BUG: 396701

Diff Detail

Repository
R319 Konsole
Branch
restore_new_tab_behavior
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 3323
Build 3341: arc lint + arc unit
roberts created this revision.Sep 29 2018, 8:48 AM
Restricted Application added a project: Konsole. · View Herald TranscriptSep 29 2018, 8:48 AM
Restricted Application added a subscriber: konsole-devel. · View Herald Transcript
roberts requested review of this revision.Sep 29 2018, 8:48 AM

I tougth I fixed that a while ago

Em dom, 30 de set de 2018 às 04:34, Nathaniel Graham <
noreply@phabricator.kde.org> escreveu:

ngraham added reviewers: Konsole, hindenburg, tcanabrava. View Revision
https://phabricator.kde.org/D15828
*REPOSITORY*
R319 Konsole

*REVISION DETAIL*
https://phabricator.kde.org/D15828

*To: *roberts, Konsole, hindenburg, tcanabrava

*Cc: *konsole-devel, herrold, ngraham, maximilianocuria, hindenburg

I thought this was fixed as well but the 'New tab behavior" setting appears to be ignored.

roberts edited the summary of this revision. (Show Details)Oct 6 2018, 7:32 AM

@hindenburg, @tcanabrava: I cannot find a pending a review that would re-add this functionality. Should I push this or do you want to deal with it separately?

Push.
My time is limited unfortunately.
Next week I'll have more time for konsole but work has eating my time.

Em sáb, 6 de out de 2018 às 09:38, Lindsay Roberts <
noreply@phabricator.kde.org> escreveu:

roberts added a comment. View Revision
https://phabricator.kde.org/D15828

@hindenburg https://phabricator.kde.org/p/hindenburg/, @tcanabrava
https://phabricator.kde.org/p/tcanabrava/: I cannot find a pending a
review that would re-add this functionality. Should I push this or do you
want to deal with it separately?

*REPOSITORY*
R319 Konsole

*REVISION DETAIL*
https://phabricator.kde.org/D15828

*To: *roberts, Konsole, hindenburg, tcanabrava
*Cc: *konsole-devel, herrold, ngraham, maximilianocuria, hindenburg

This revision was not accepted when it landed; it landed in state Needs Review.Oct 6 2018, 12:58 PM
This revision was automatically updated to reflect the committed changes.