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

[MRG] GUI fix minor glitches #899

Merged
merged 3 commits into from
Oct 4, 2024

Conversation

gtdang
Copy link
Collaborator

@gtdang gtdang commented Sep 26, 2024

  1. Removed tight_layout method in favor of the constrained layout argument

    • The tight layout was displaying a warning for spectrogram plots. This change removes that message.
  2. fixed set selected index for figure controls after new figure is generated

    • When creating a blank figure, the GUI was not automatically opening/selecting the new figure tab. This was because the selected index was only being set for the figure tabs but not the figure control tabs. This made them not synchronous, and when the two tab groups are later linked, it reverted to the un-updated index.

…ated

When creating a blank figure, the GUI was not automatically opening/selecting the new figure tab. This was because the selected index was only being set for the figure tabs but not the figure control tabs. This made them not synchronous, and when the two tab groups are later linked, it reverted to the un-updated index.
…rgument

The tight layout was displaying a warning for spectrogram plots. This change removes that message.
@gtdang gtdang marked this pull request as ready for review September 26, 2024 15:25
@gtdang gtdang changed the title Gui fix minor glitches [MRG] GUI fix minor glitches Sep 26, 2024
@ntolley
Copy link
Contributor

ntolley commented Oct 4, 2024

Everything looks good on me end, thanks @gtdang!!

@ntolley ntolley merged commit 2376a64 into jonescompneurolab:master Oct 4, 2024
12 checks passed
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