Skip to content

gui: Set Style to Fusion for all systems#125

Merged
joshanne merged 1 commit intodronecan:masterfrom
joshanne:pr/set-style-of-application
Apr 1, 2026
Merged

gui: Set Style to Fusion for all systems#125
joshanne merged 1 commit intodronecan:masterfrom
joshanne:pr/set-style-of-application

Conversation

@joshanne
Copy link
Copy Markdown
Collaborator

@joshanne joshanne commented Mar 31, 2026

There was something uncanny with the windows build.

You'll notice the windows before image looks different to the Linux before (I haven't added Linux before since it looks the same as Linux after).

The fusion style is platform agnostic, and should be same across systems. See: https://doc.qt.io/qt-6/qtquickcontrols-fusion.html

Windows before:
image

Windows after:
image

Linux after:
image

@joshanne
Copy link
Copy Markdown
Collaborator Author

joshanne commented Mar 31, 2026

@Huibean if you have an opportunity, are you able to test the Mac build to ensure the style looks consistent between your Mac build and the Linux/windows images above. Would be very appreciated.

Copy link
Copy Markdown
Member

@Huibean Huibean left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Image left side is with "Fusion", I have no perference on which style is better, but since it did fix the slider style, so LGTM

@joshanne
Copy link
Copy Markdown
Collaborator Author

joshanne commented Mar 31, 2026

Personally I found the new Qt6 theme a little jarring in windows. Linux was largely unchanged.

I think by setting the theme, we're being consistent across all operating systems now.

@joshanne
Copy link
Copy Markdown
Collaborator Author

joshanne commented Apr 1, 2026

Merging this to keep consistency across the OS'

@joshanne joshanne merged commit 22f2ac8 into dronecan:master Apr 1, 2026
5 checks passed
@joshanne joshanne deleted the pr/set-style-of-application branch April 1, 2026 08:18
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.

2 participants