Bug 440382 - Main toolbar elements messed up on (un)maximize when using split view.
Summary: Main toolbar elements messed up on (un)maximize when using split view.
Status: RESOLVED DUPLICATE of bug 430521
Alias: None
Product: dolphin
Classification: Applications
Component: general (show other bugs)
Version: 21.07.80
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Dolphin Bug Assignee
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-07-29 11:38 UTC by Antonio Orefice
Modified: 2021-08-02 07:57 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Video of the bug (743.60 KB, video/x-matroska)
2021-07-29 11:38 UTC, Antonio Orefice
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Antonio Orefice 2021-07-29 11:38:29 UTC
Created attachment 140386 [details]
Video of the bug

SUMMARY
When using split view and maximizing and unmaximizing the main dolphin window, toolbar elements got messed up.
See Video attached.

Offending commit:
koko@Gozer# git bisect good
0cee94ce82ccb82afd0c4e22d77e251276e7a447 is the first bad commit
commit 0cee94ce82ccb82afd0c4e22d77e251276e7a447
Author: Felix Ernst <fe.a.ernst@gmail.com>
Date:   Wed Jan 6 01:38:45 2021 +0000

    Fix location bar being wrongly aligned on first startup
    
    When starting Dolphin the very first time, the spacing in front of the
    location bar is wrong. This commit fixes this by completely updating
    all cached geometry every time adjustSpacing() is called. Because this
    happens once on a timer 100 ms after every url change, it will happen
    once shortly after the window is shown. At that point all geometry is
    where it should be and spacing calculation works as expected.
    
    The ViewContainer geometry retrieval is refactored into a small nested
    helper class in DolphinNavigatorsWidgetAction by the name
    ViewGeometriesHelper.
    
    Previously the logic of that class was divided between DolphinTabPage
    and DolphinNavigatorsWidgetAction.
    
    BUG: 429447
    FIXED-IN: 21.04.0

 src/dolphinnavigatorswidgetaction.cpp | 216 ++++++++++++++++++++--------------
 src/dolphinnavigatorswidgetaction.h   |  78 ++++++++----
 src/dolphintabpage.cpp                |  34 +-----
 src/dolphintabpage.h                  |   5 -
 4 files changed, 191 insertions(+), 142 deletions(-)
Comment 1 Antonio Orefice 2021-08-02 07:57:11 UTC

*** This bug has been marked as a duplicate of bug 430521 ***