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 #119376 utility functions for save/restore geometry #2752

Merged
merged 3 commits into from
Aug 1, 2016

Conversation

jeetee
Copy link
Contributor

@jeetee jeetee commented Jul 25, 2016

No description provided.

@jeetee
Copy link
Contributor Author

jeetee commented Jul 27, 2016

Just another thought; Currently each window/dialog creates its own settingsGroup as was done previously.
I'm considering making the utility functions use a settingsGroup called "Geometries" and use the objectNames as keys in that group. This should allow us to create a "Reset Window Positions" command more easily as it could then clear out this group.

Reason is that I've often seen the "Reset to Factory Settings" been advised/required to recover from offscreen windows, which is a fairly drastic measure to have to take. The new command would leave all other settings intact.

@lasconic
Copy link
Contributor

Sure. Good one.

@jeetee jeetee force-pushed the save_restore_window_geometry branch from a5af572 to 8d6f0f6 Compare July 28, 2016 14:29
@jeetee jeetee force-pushed the save_restore_window_geometry branch from 8d6f0f6 to d2bf3b8 Compare July 30, 2016 13:37
@jeetee
Copy link
Contributor Author

jeetee commented Jul 30, 2016

Handles a bunch of the known bugs/requests for window dialog geometries.
Still working on fixing up the MainWindow behavior (including maximized/fullscreen handling)

@jeetee jeetee force-pushed the save_restore_window_geometry branch from f09d9c3 to 03453e1 Compare July 31, 2016 09:22
@lasconic lasconic merged commit 93987c8 into musescore:master Aug 1, 2016
@jeetee jeetee deleted the save_restore_window_geometry branch August 1, 2016 19:33
@IsaacWeiss
Copy link
Contributor

@jeetee: Should I log a feature request for "Reset Window Positions"?

@jeetee
Copy link
Contributor Author

jeetee commented Aug 1, 2016

@IsaacWeiss good plan, although strictly speaking, we shouldn't have offscreen windows any more with this approach, so the need for such a function is probably much lower now.

@IsaacWeiss
Copy link
Contributor

IsaacWeiss commented Aug 1, 2016

Oh! I didn't realize that. Great. :) Not needed, then.

@Jojo-Schmitz
Copy link
Contributor

Can we have this or something to the same effect for 2.0.4 too, please? Comes up increasingly in the Forums as being a problem

@jeetee
Copy link
Contributor Author

jeetee commented Sep 25, 2016

@Jojo-Schmitz If 2.0.4 comes into play, it might be easier to update https://musescore.org/en/project/resetdialogpositions to get as close to a reset window position function as possible. This PR influenced the way of storing the information within the configuration file, breaking 2.0.x compatibility.

@Jojo-Schmitz
Copy link
Contributor

I don't think the config file (MuseScore*.ini) has to be compatible between versions, Not being compatibel would possibly only affect a downdgrade to 20.x with x < 4, and then a factory reset would fix it. Maybe that factory reset could even be made part of the uninstall

@lasconic
Copy link
Contributor

I don't think people with a ton of custom shortcuts/palettes/workspace would agree with that.

@Jojo-Schmitz
Copy link
Contributor

Jojo-Schmitz commented Sep 26, 2016

Well, as an option this might work. And we do have -R and -F to chose from?
There are loads of topics in the forums where users unistallated and reinstalled in the wrong assumption that this would be at least as good as a factory reset.

@jeetee
Copy link
Contributor Author

jeetee commented Sep 27, 2016

For this specific PR the 'breaking' of compatibility is actually quite minor. After the PR window positions are stored differently, so if you'd use an older version, it'll have its own set of positioning info, as it uses different Settings parameters.

As for the latter remark about uninstall-reïnstall, we could perhaps go with "the VLC-way"; having an (un-)installer option to reset to defaults (I believe they call it "remove local settings cache"). But that should be its own feature request...

@Jojo-Schmitz
Copy link
Contributor

Jojo-Schmitz commented Sep 27, 2016

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

4 participants