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

"Indicator" settings failed to save on Windows 10 22H2 #160

Closed
vibperson opened this issue Mar 2, 2023 · 16 comments
Closed

"Indicator" settings failed to save on Windows 10 22H2 #160

vibperson opened this issue Mar 2, 2023 · 16 comments
Assignees
Labels
bug Something isn't working upstream
Milestone

Comments

@vibperson
Copy link

what i need to do to save this option? If i want to use bar - not pointers. It resets after close
изображение

and similar problem with displays. when i mark to use second - h mirrors, its also resets after close (on this pic i 've i screen but now i can't make screen 2nd)
изображение

@Cuperino
Copy link
Owner

Cuperino commented Mar 2, 2023

Thanks for reporting. Always try to to separate each issue in a different report, as that helps with our tracking.

I'll dedicate this issue to the bar issue exclusively and make a new issue regarding the screens.

@Cuperino Cuperino self-assigned this Mar 2, 2023
@Cuperino
Copy link
Owner

Cuperino commented Mar 2, 2023

What operating system and QPrompt version are you using? You can find out QPrompt's version in the About Page. I ask because I wasn't able to replicate your issue with Bar not saving on my system.

@Cuperino Cuperino added unconfirmed bug more info needed More information is required to work on this issue labels Mar 2, 2023
@vibperson
Copy link
Author

Qpromt v.1.1.5 (v.1.1/720a13) and Windows 10 x64 22H2

@Cuperino
Copy link
Owner

Cuperino commented Mar 2, 2023

Excellent! Thanks.

@videosmith
Copy link

videosmith commented Mar 4, 2023

what i need to do to save this option? If i want to use bar - not pointers. It resets after close изображение

and similar problem with displays. when i mark to use second - h mirrors, its also resets after close (on this pic i 've i screen but now i can't make screen 2nd) изображение

I have 1.1.5 on an x86_64 platform. the option selected persists correctly through a restart/reboot. Do you get pointers back when restarting? If not a possible culprit may be you're not seeing the bar since it's the same color as your background?

edit: config file may not be updating upon closing?

@videosmith
Copy link

Tried to Disable background transparency, upon application restart, setting ignored.

@Cuperino
Copy link
Owner

Cuperino commented Mar 4, 2023

@vibperson, when you restarted the QPrompt and the Indicator Bar setting was ignored, were you restarting through a prompt to restart that QPrompt gave you, or did you restart by manually exiting the app and re-opening it?

@Cuperino Cuperino added bug Something isn't working and removed unconfirmed bug labels Mar 4, 2023
@Cuperino Cuperino added this to the 1.1.6 milestone Mar 4, 2023
@videosmith
Copy link

Even when uninstalling the .deb app and executing the AppImage file instead, the issue persists.

Which files survive an 'apt purge qprompt' command?

@Cuperino
Copy link
Owner

Cuperino commented Mar 4, 2023

Nothing should survive 'apt purge qprompt'.

@videosmith
Copy link

Tried another reboot, apt autoremove... no change. Where are configuration files installed? Just in case something is stuck?

@Cuperino
Copy link
Owner

Cuperino commented Mar 4, 2023

On Linux they install to ~/.config/Cuperino/qprompt.conf.

Before deleting, please make a copy, and if QPrompt restarts correctly after deleting the original, upload the copy here for me to inspect.

A few betas also saved to ~/.config/Cuperino/QPrompt.conf, but those settings are ignored and the file can be safely deleted.

@videosmith
Copy link

videosmith commented Mar 4, 2023

That fixed it. File was not updating. Removing it caused QPrompt to generate a new one. Somehow not deleting when purge command issued.
screen1

conf_archive.zip

@Cuperino Cuperino added upstream and removed more info needed More information is required to work on this issue labels Mar 4, 2023
@Cuperino
Copy link
Owner

Cuperino commented Mar 4, 2023

The config is fine, it loads and saves correctly on my system. Maybe there's another file involved with saving that I don't know of. Either way, this seems like a problem with the Qt library itself now.

@vibperson, when you can, please answer the question that I asked earlier in this thread:

when you restarted the QPrompt and the Indicator Bar setting was ignored, were you restarting through a prompt to restart that QPrompt gave you, or did you restart by manually exiting the app and re-opening it?

Your answer could bring more certainty to my suspicions of what might be causing the bug; which might help prevent it from happening.

Also, deleting QPrompt's settings like @videosmith did will probably solve the problem for you as well. On Windows, QPrompt settings are saved to the Windows registry:

  1. Open Window's Registry Editor
  2. Navigate to Computer > HKEY_CURRENT_USER > Software > Cuperino > qprompt
  3. Right click on readRegion, then click on Delete

If the problem persists, report it here. Then delete the entire qprompt folder from the registry and restart QPrompt.

@Cuperino Cuperino modified the milestones: 1.1.6, 1.2 Mar 5, 2023
@vibperson
Copy link
Author

I've opened QPrompt put Bar and then i close it or press Quit in QPrompt.

@vibperson
Copy link
Author

vibperson commented Mar 5, 2023

It dont ask me to restart if i change pointers to bar.
Reg manipulations didn't helped

@Cuperino
Copy link
Owner

Cuperino commented Mar 5, 2023

Hello @vibperson,

I have good news.I managed to re-create the bug you were experiencing, and in the processed uncovered a different bug and fixed both.

Although I was able to fix the bug you are experiencing, I have to say that this bug is in part a bug in Windows 10 22H2 specifically. Windows 10 21H1, Windows 10 21H2 and Windows 11 22H2 are not subject to your issue and neither are KDE neon 22.04 Linux or MacOS 13.0.1 Ventura.

The issue was the following: When QPrompt starts for the first time, the problematic setting, which is called "styleState", should've been set to a default numeric value, but instead a text representation of the same value was being saved. This issue affected all systems and meant no setting was being loaded on when the app first launched, which made it default to a different setting. Fixing this bug did not solve the Windows 10 22H2 bug.

On Windows 10 22H2, QPrompt to save the default value to the registry every time the program is run, even when there's already a value present. This meant Windows 10 22H2 was not reporting QPrompt with the correct value for "styleState" when the application started. This issue occurred because QPrompt did some additional processing to choose an arrow based on the system's language. Since users never got to experience the true initial value, because of the first bug, what I did was turn that unintentional default into the deliberate default choice, allowing me to eliminate the step that the bug that can only be experience in Windows 10 22H2.

The bug is fixed for both the v1.1 and v1.2 branches. It may take a few weeks before I make new builds. I'm out of time for this weekend, the next 3 weekends are committed, and these builds can take 2 or 3 days to make, due to issues that arise because of upstream changes, and the amount of testing involved.

@Cuperino Cuperino closed this as completed Mar 5, 2023
@Cuperino Cuperino changed the title options bar and screen save "Indicator" settings failed to save on Windows 10 22H2 Mar 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working upstream
Projects
Status: Done
Development

No branches or pull requests

3 participants