StackedViewContainer is a plain view container with no navigiations.
It can be replaced using TabbedViewContainer and hiding the tabbar.
This will minimize duplicate code and allow easier control of code
in Konsole and KonsolePart.
Details
Details
- Reviewers
hindenburg - Group Reviewers
Konsole - Commits
- R319:12fa4737ab66: Remove StackedViewContainer class
Run konsole and konsole part test, and then
use the shortcuts to create tab.
Diff Detail
Diff Detail
- Repository
- R319 Konsole
- Branch
- removeStackedView
- Lint
No Linters Available - Unit
No Unit Test Coverage
src/ViewManager.cpp | ||
---|---|---|
705 | I would take this to a method: if (tabbedNavigation()) { container->setTabBarVisible(true); } or container->setTabBarVisible(tabbedNavigation()); If it is clear enough. |
src/ViewManager.cpp | ||
---|---|---|
705 | I can't set it to "container->setTabBarVisible(tabbedNavigation());" as it would show the tabbar even when there's only one view, and that's not what konsole expects. also, I don't think a new method that only does a single boolean operation is really needed. :) |