Skip to content

Conversation

@SiboVG
Copy link
Member

@SiboVG SiboVG commented Jun 13, 2022

This PR fixes #974: Moving from component to component, configuration tab selected could open to the same tab.

The current behavior is this:

  • If a tab is selected the component config dialog of component 1, and you edit another component, component 2 (either by clicking component 2 while the edit dialog of component 1 is still open, or by first closing component 1's edit dialog, and then editing component 2):

    • If the previously selected tab of component 1 is also present for component 2 (e.g. both components have a ‘General’ tab or ‘Appearance’ tab), then that same tab is opened for component 2

    • If component 2 does not have the tab of component 1, the first tab of component 2 is selected (e.g. component 1 is a FinSet with the ‘Fin tabs’ tab selected, component 2 is a Body Tube, then the first tab of component 2 - ‘General’ - will be opened)

  • If a new component is added, the automatic dialog pop-up will always be opened in the first tab (e.g. ‘General’)

Demo:

Screen.Recording.2022-06-13.at.12.47.00.mp4

@SiboVG
Copy link
Member Author

SiboVG commented Jun 13, 2022

I'm wondering though: should a user be able to disable this new behavior in the system preferences or do we want to make this default behavior?

@neilweinstock
Copy link
Contributor

I would definitely make it the default behavior... hide it in preferences and no one will ever notice it. Those that are bothered by the new behavior might be motivated to go find the preference (or ask on the forum, or whatever).

I think it's worth a checkbox in preferences. If it's easier to have the default options be disabled by default, or if it's too darn hard to come up with good wording for the option, then you can do it opposite: have the option be "Always open leftmost tab when opening a component edit dialog", unchecked by default.

@SiboVG SiboVG marked this pull request as draft June 13, 2022 23:47
@hcraigmiller
Copy link
Collaborator

My preference would be to have the new behavior be the default behavior.

@SiboVG
Copy link
Member Author

SiboVG commented Jun 14, 2022

Latest commits add option in preference panel (couldn't find a good enabled wording, so I went for the disabled wording proposed by Neil)
image

@SiboVG SiboVG marked this pull request as ready for review June 14, 2022 00:33
@neilweinstock
Copy link
Contributor

Typo in tooltip: almost -> always

@hcraigmiller
Copy link
Collaborator

hcraigmiller commented Jun 14, 2022

I would change "leftmost" to "General", ditto on @neilweinstock.

Otherwise, works beautifully, no anomalied found.

@SiboVG
Copy link
Member Author

SiboVG commented Jun 14, 2022

Typo in tooltip: almost -> always

My bad, it was too late when I made that change

@SiboVG
Copy link
Member Author

SiboVG commented Jun 14, 2022

I would change "leftmost" to "General", ditto on @neilweinstock.

I would not do this because it's not always General, for stages and boosters it's 'override' and 'separation' respectively.

@SiboVG SiboVG merged commit 3730f78 into openrocket:unstable Jun 14, 2022
@SiboVG SiboVG deleted the issue-974 branch September 30, 2022 12:27
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.

New feature request: Moving from component to component, configuration tab selected could open to the same tab

3 participants