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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

macOS Sonoma Fixes #166

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

furby-tm
Copy link

@furby-tm furby-tm commented Jul 17, 2023

Bit of a noisy commit, sorry about that. This repo should use a formatter, SwiftFormat is a good one.

It appears to be a SwiftUI bug, however applying a .buttonStyle() modifier, even with a PlainButtonStyle() appears to break the ability to click on any of the Server (AKA Guild) buttons -- at least on macOS 14 Sonoma. Not sure if it's functioning correctly on macOS 13 Ventura or below.

In this revision I've removed the .buttonStyle() which introduces the default style -- slightly ugly rounded rect button backgrounds to the buttons, however; it restores the functionality of these buttons in the meantime.

The work for that can be summarized in the changes within the following files:

There was also a fix for the settings window, as it does not open on versions macOS 14 Sonoma and above without being opened from within a SettingsLink closure.

If server buttons are working as expected for older versions of macOS, I can clean these changes up and work out of a macOS 14 Sonoma branch which you can merge at some point down the line.

Thanks! I appreciate your work so far on Swiftcord, the Swift community keeps getting more interesting by the day 馃槃.

* TODO: The buttons need PlainButtonStyle() or a similiar custom style, however,
  that breaks the ability to click on them, SwiftUI bug?
@Candygoblen123
Copy link
Collaborator

Hi,
Thanks for the suggestion, but we already have a linter.

Could you please revert the team ID in the Xcode project back to what it was originally?

@cryptoAlgorithm cryptoAlgorithm changed the title Working Server (aka Guild) buttons. macOS Sonoma Fixes Jul 25, 2023
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.

None yet

2 participants