-
Notifications
You must be signed in to change notification settings - Fork 218
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
Use Qt 6.5 setBadgeNumber on Server (macOS) #3062
Comments
I think this issue needs further clarification: "Once we've moved to Qt6.5" implies there's a roadmap to move to Qt6.5. Is there an issue for that? If so, it should be referenced here. The choices offered sound like "do it" and "don't do it", where "do it" would be part of the move to Qt6.5 anyway. So I'm not sure how this is a separate issue. Maybe more context is needed? |
Qt 6.5 fails at the moment so we can't move forward. The current macOS only code for the badges is a bit like a workaround and as far as I understand just a macOS only feature. If we use the Qt native function we'd have more support on other OS. |
Could you also please point out where in Jamulus the issue is, so it's easier to find. Thanks. |
Sure. It's this file: https://github.com/jamulussoftware/jamulus/blob/main/src/mac/badgelabel.mm As far as I understand it's basically injecting Objective C into Jamulus on macOS. |
OK, so as a minimum we'd want something like https://github.com/jamulussoftware/jamulus/blob/main/src/mac/badgelabel.h not just for macOS, implemented as appropriate, if it's not supported in Qt directly. (There's a Qt Windows one - https://doc.qt.io/qt-5/qwintaskbarbutton.html - for Qt 5.15. My googling didn't find much on the topic...) |
I think the Windows version displays another icon if people are connected. So I'm not sure if that's needed on Windows too. I don't run a Windows server so I'm not qualified. |
What is the current behaviour and why should it be changed?
Currently we're not using the Qt 6.5 but the native Objective C code to set the Badge on macOS. Once we've moved to Qt6.5 we should add a conditional compilation step to use setBadgeNumber: https://doc-snapshots.qt.io/qt6-dev/qguiapplication.html#setBadgeNumber
Describe possible approaches
Has this feature been discussed and generally agreed?
No
The text was updated successfully, but these errors were encountered: