Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix tabs not being active when restoring state #101

Merged

Conversation

nairaner
Copy link

Regression introduced by 29ebc83

@githubuser0xFFFF githubuser0xFFFF merged commit d4c179c into githubuser0xFFFF:master Jan 31, 2020
@githubuser0xFFFF
Copy link
Owner

Thank you. Could you tell me how to reproduce the problem. I just tested before the merge and yould not reproduce the problem?

@nairaner nairaner deleted the bugfix/active-tab branch February 4, 2020 09:37
@nairaner
Copy link
Author

nairaner commented Feb 4, 2020

I'm not exactly sure why it happens. I just add 6 DockWidgets to DockManager and restore default layout. I also tried using new version of layout XML , but it didn't change anything. You can probably reproduce just this layout XML.

<QtAdvancedDockingSystem Version="0" Containers="1">
  <Container Floating="0">
    <Splitter Orientation="-" Count="3">
      <Splitter Orientation="|" Count="2">
        <Area Tabs="1" Current="Structure">
          <Widget Name="Structure" Closed="0" />
        </Area>
        <Area Tabs="1" Current="Relation">
          <Widget Name="Relation" Closed="0" />
        </Area>
        <Sizes>388 387 </Sizes>
      </Splitter>
      <Area Tabs="1" Current="Viewport">
        <Widget Name="Viewport" Closed="0" />
      </Area>
      <Area Tabs="3" Current="General">
        <Widget Name="General" Closed="0" />
        <Widget Name="Appearance" Closed="0" />
        <Widget Name="Camera" Closed="0" />
      </Area>
      <Sizes>296 697 254 </Sizes>
    </Splitter>
  </Container>
</QtAdvancedDockingSystem>

luelista pushed a commit to luelista/Qt-Advanced-Docking-System that referenced this pull request Mar 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants