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

[Bug] Widgets color reset after update (GPU widget) #432

Closed
lucasaf04 opened this issue Apr 20, 2021 · 25 comments
Closed

[Bug] Widgets color reset after update (GPU widget) #432

lucasaf04 opened this issue Apr 20, 2021 · 25 comments
Labels
bug Something isn't working
Projects

Comments

@lucasaf04
Copy link
Contributor

Describe the bug
Widgets colors have been reset after update to latest version. After setting them back, GPU % widget color isn't saved and reverts back to system accent if you close and reopen Stats.

Details:

  • Device: Macbook Pro M1
  • MacOS 11.2.3
  • Application version 2.5.10
@exelban
Copy link
Owner

exelban commented Apr 20, 2021

Hi.
It's was because I was added a translation to the colors. And must change the keys. Sorry for not include this information in the release notes.

@exelban exelban closed this as completed Apr 20, 2021
@lucasaf04
Copy link
Contributor Author

@exelban But GPU % widget shouldn’t save color value? As I mentioned it resets it to system accent, while the rest of the widgets work properly. Sorry if I didn’t explain well.

@exelban
Copy link
Owner

exelban commented Apr 20, 2021

Oh, sorry. Don't understand the problem with GPU.
I have rechecked it now and everything looks ok. What exactly widget have this problem?

@lucasaf04
Copy link
Contributor Author

@exelban This one:
Screenshot 2021-04-21 at 08 05 36

@exelban
Copy link
Owner

exelban commented Apr 21, 2021

Hmm. It's not saving the color when you change to something else?

@lucasaf04
Copy link
Contributor Author

@exelban Yes.

@exelban
Copy link
Owner

exelban commented Apr 21, 2021

Could you please run defaults delete eu.exelban.Stats and check if it helps?

@lucasaf04
Copy link
Contributor Author

@exelban It doesn’t help.😢 After setting up everything again, now gpu % widget defaults to monochrome accent, the rest works fine.

@exelban
Copy link
Owner

exelban commented Apr 21, 2021

I check this behavior on 2 macs. And it works.
Does the color not change when you change it in the settings? Or it resets after restart?

@lucasaf04
Copy link
Contributor Author

It resets after restart. I have also checked it on my intel Mac and the behavior is the same.

@exelban
Copy link
Owner

exelban commented Apr 21, 2021

What happens when you select red color?

@lucasaf04
Copy link
Contributor Author

Same.

@exelban
Copy link
Owner

exelban commented Apr 21, 2021

What do you mean by same?

@exelban
Copy link
Owner

exelban commented Apr 21, 2021

The color of the value in the widget does not change from black to red?

@lucasaf04
Copy link
Contributor Author

It changes, I close and reopen stats and it’s on monochrome accent

@exelban
Copy link
Owner

exelban commented Apr 21, 2021

Are you sure that it happens only with GPU?

@lucasaf04
Copy link
Contributor Author

Yes

@exelban
Copy link
Owner

exelban commented Apr 21, 2021

Very strange. Because there is one "sharable" widget.
Could you make next steps:

  • run defaults read eu.exelban.Stats GPU_mini_color and check the output
  • change the color and run defaults read eu.exelban.Stats GPU_mini_color again
  • restart the app and run defaults read eu.exelban.Stats GPU_mini_color

Post please the output of each run.

@lucasaf04
Copy link
Contributor Author

lucasaf04 commented Apr 21, 2021

defaults read eu.exelban.Stats GPU_mini_color
2021-04-21 21:37:23.189 defaults[56248:208517]
The domain/default pair of (eu.exelban.Stats, GPU_mini_color) does not exist

If this helps, this are all the saved keys:
Screenshot 2021-04-21 at 21 39 33

@lucasaf04
Copy link
Contributor Author

If I check iGPU_mini_color:
First run: app just opened, showing monochrome accent.
Second run: color changed to green.
Third run: app restarted and showing monochrome accent.
Screenshot 2021-04-21 at 21 42 32

@exelban exelban reopened this Apr 21, 2021
@exelban
Copy link
Owner

exelban commented Apr 21, 2021

Ok, I found the problem. If you disable the Show GPU type options, everything will work fine.

@exelban exelban added the bug Something isn't working label Apr 21, 2021
@exelban exelban added this to To do in Roadmap Apr 21, 2021
@exelban exelban changed the title [Bug] Widgets color reset after update. [Bug] Widgets color reset after update (GPU widget) Apr 21, 2021
@lucasaf04
Copy link
Contributor Author

Thanks. 😊

@exelban
Copy link
Owner

exelban commented Apr 21, 2021

It will be fixed in the next release.
Thanks for the feedback and help with debugging the problem)

@arisudesu
Copy link
Contributor

arisudesu commented Apr 22, 2021

Also, the "second purple" color suddenly became black.

Not the single one, but multiple colors were affected:
05091a9#diff-f40890c39c072dadc03386db07b0e3d96fecd408b3fe9c21ea0853b18f502a6cR122-R128

@exelban
Copy link
Owner

exelban commented Apr 23, 2021

@arisudesu Hi. Thanks for reporting. I have noticed this, and fix will be today.

@exelban exelban moved this from To do to Waiting for the next release in Roadmap Apr 24, 2021
@exelban exelban closed this as completed Apr 25, 2021
@exelban exelban moved this from Waiting for the next release to Done in Roadmap Apr 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
No open projects
Roadmap
  
Done
Development

No branches or pull requests

3 participants