client
client copied to clipboard
Add update information to the about dialog
Fixes: #11486
@TheOneRing I'm not sure if we should move everything from the settings dialog to the about dialog. I think we should move the update part in the settings up higher, because it's probably more used than bandwidth limits.
For the about dialog: the text is customisable, so I added a label about the update status. We should probably also add a button or link text to redirect to the settings when there is an update available.
Hmm the channel selection is only available in testpilotcloud so maybe we can move it to the about dialog?
Hmm the channel selection is only available in testpilotcloud so maybe we can move it to the about dialog?
We can. I think it would give theming/design problems if we put it in the "about" tab, so maybe make a new (second) tab called "Updates" and put the contents of the QGroupBox in there? Also, I introduced that label at the bottom to draw more attention to updates, should that label stay? (See screenshot below.)
I think only the text can be branded, so it should be possible to add the info into the main widget.
In the latest commit I moved all of the update stuff from the settings dialog to the about dialog. @TheOneRing How can I test this?
I'm not sure if I can get the version number out. I think it'd be useful to show that too.
@TheOneRing The OCUpdater could provide the version. Should I add that in a separate PR?
I'm not sure if I can get the version number out. I think it'd be useful to show that too.
So that was during testing. Meaning it passed the testing.