| ngraham | |
| elvisangelaccio |
| Dolphin |
When opening a new tab in the background and switching to that tab the focus is set on the location bar instead of the files view. If you switch back to the original tab and then to the new tab again focus will be set on the files view. The problem is caused by creation of DolphinTabPage in an active state which leads to skipping by return in DolphinView::setActive(bool active) without setting the focus on the view. This patch fixes this defect.
BUG: 407604
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
Hmm, this feels like a work-around because we initialize m_active(true) everywhere else (DolphinView, DolphinSearchBox).
If possible it should be initialized to false only if the tab page is a background tab.
It does not look correct either, the problem is that the focus on location bar is delayed and setActive(false) isn't applied, this is workaround. But we have problems on that before, exactly that part with focusIn on location bar. You should correct there but it can break something other.
When tab is switched, DolphinTabWidget::currentTabChanged deactivates last-viewed tab page and activates new current tab, and there should be only one active tab at same moment. The problem is that DolphinTabWidget::openNewTab creates tabs in active state and now there several tabs in active state, which don't get the focus on the files view after switching on them first time, because DolphinView::setActive doesn't do anything if new active state(true) is equal to old state(true), while it should set focus on the view.
I like this solution better. I'll push on master only for now, and we'll see if we break something ;)