Dont show tabs if in KParts mode
AbandonedPublic

Authored by tcanabrava on Nov 4 2018, 7:30 PM.

Details

Summary

In kparts mode we just want to show the TerminalDisplay,
but right now this is very hard, so just hide the tabbar.

Test Plan

Opened instances of dolphin / kate and looked the kparts output.
Opened konsole, everything seems fine.

Diff Detail

Repository
R319 Konsole
Branch
fixTabKParts
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 4563
Build 4581: arc lint + arc unit
tcanabrava created this revision.Nov 4 2018, 7:30 PM
Restricted Application added a project: Konsole. · View Herald TranscriptNov 4 2018, 7:30 PM
Restricted Application added a subscriber: konsole-devel. · View Herald Transcript
tcanabrava requested review of this revision.Nov 4 2018, 7:30 PM
tcanabrava edited the test plan for this revision. (Show Details)Nov 4 2018, 7:31 PM

Did you consider just checking isKonsolePart() and hiding the tabbar? I didn't actually try coding that but since other parts of the code use thought it was a possibility.

I didn't, I looked for isPart, isKPart but my grep didn't try
isKonsokePart. I'll update the code.

Em ter, 6 de nov de 2018 às 02:28, Kurt Hindenburg <
noreply@phabricator.kde.org> escreveu:

hindenburg added a comment. View Revision
https://phabricator.kde.org/D16669

Did you consider just checking isKonsolePart() and hiding the tabbar? I
didn't actually try coding that but since other parts of the code use
thought it was a possibility.

*REPOSITORY*
R319 Konsole

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

*To: *tcanabrava, hindenburg, sandsmark, ngraham
*Cc: *konsole-devel, ngraham, maximilianocuria, hindenburg

tcanabrava abandoned this revision.Nov 6 2018, 12:23 PM

arc diff created a new revision, sorry for that. https://phabricator.kde.org/D16706