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

Improve support when specifying a pre-existing toolbar as the target of agw.aui Minimize #2495

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jmoraleda
Copy link
Contributor

agw.aui includes the option to specify a pre-existing toolbar as the location where the restore icon/tool should go when minimizing panes -- instead of the default behavior which is creating a new automatic toolbar.

But the option of specifying a pre-existing toolbar cannot handle loading new perspectives or closing or detaching minimized panes. The main issues when specifying a pre-existing toolbar are:

  • Using the tool id to find the tool does not work well because multiple tools with the "ID_RESTORE_FRAME" id could have been added to the same toolbar
  • Using the pane label/caption to find the tool does not work well when loading a perspective because the caption of the pane could be changed by the load operation
  • When loading a perspective, a pane that was minimized could become unminimized or vice versa, and when using a pre-existent toolbar, the toolbar would be preserved by the perspective change operation and the restore icon would need to be added or deleted from the toolbar as appropriate, but not added twice if it was already there.

This PR addresses these problems.

@jmoraleda jmoraleda force-pushed the agw-aui-improved-pane-minimize-support branch from 0264c2f to b9fb1c6 Compare December 4, 2023 02:42
…for the restore icon when minimizing a pane in agw.aui
@jmoraleda jmoraleda force-pushed the agw-aui-improved-pane-minimize-support branch from b9fb1c6 to dcf76cb Compare December 8, 2023 17:35
@jmoraleda
Copy link
Contributor Author

Rebased to latest master, since it did not apply cleanly after merging #2494

@Metallicow
Copy link
Contributor

It is still a problem with agw.aui, but it seems you have collapsed two of the issues into a single subject.
These issues may be subject to review.
Look into the depth....

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.

None yet

2 participants