UI fixes
Description
I added a few UI fixes/enhancements
- Added hover highlighting Buttons already got fully highlighted when checked or pressed. Now they get partially highlighted when hovered
Samples of before (no visual change when mouse hovers the buttons shown)
samples of after - mouse hover highlights slightly (dark mode)
samples of after - mouse hover highlights slightly (light mode)
- Make some fixes to the "Palette Test" page the light/dark toggle button was broken and some of the QGCComponent examples lookes messed up
before
after
- Fix QGC icon The QGC icon shown in the top left corner of the app did not match the typical QGC icon seen everywhere else (icon of QGC shortcut made after an install, icon branding on QGC website, etc). I replaced the instances of the extra-bloated Q with the more common QGC icon
before
after
- fix padding on launch card
before
after
Thanks for all the nice cleanup. I'll take a look.
If you look at QGCButton you'll see that hoverEnabled is disabled on mobile builds. This works around a problem where the hover would get stuck on for android. Since there really isn't "hover" on touch systems this fixes the problem without really changing any ui behavior.
So for any control which uses a raw Button as base and has had hover support added to it this same hoverEnabled things needs to be there.
These changes affect all button controls except QGCButton. Seems like that should be changed as well to match don't you think?
If you look at QGCButton you'll see that
hoverEnabledis disabled on mobile builds. This works around a problem where the hover would get stuck on for android. Since there really isn't "hover" on touch systems this fixes the problem without really changing any ui behavior.So for any control which uses a raw
Buttonas base and has had hover support added to it this samehoverEnabledthings needs to be there.These changes affect all button controls except QGCButton. Seems like that should be changed as well to match don't you think?
@DonLakeFlyer, I made the changes you suggested to set hoverEnabled and make QGCButton have the same hover behavior. Should be good to go now!
@DonLakeFlyer, this is ready for a second round of review! I made the changes you suggested but haven't added anything else. After I added my fixes I re-did the commits so the fixes are squashed into the commit they originated from.
Let me know what your preferred modify-current-PR strategy is. If you'd prefer me to not squash and keep the fix commits separate for easy second round review I could do that next time
As discussed on the PX4 dev call @gillamkid please split this into atomic changes so we can get reviewed, thank you for your contribution!
This pull request has been mentioned on Discussion Forum for PX4, Pixhawk, QGroundControl, MAVSDK, MAVLink. There might be relevant details there:
https://discuss.px4.io/t/px4-sync-q-a-sep-4-2024/40534/1
@gillamkid Closed because you are going to make new pull?
@DonLakeFlyer yes, making some smaller PRs with the individual pieces. It will make reviewing them much easier.