-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Tweaked the side bar of the settings window #4292
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
Conversation
- Make the leading space of the icon to 0, so that the icon can align with the search icon - Side bar width 220 -> 200
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I haven't upgraded my MacBook Air from Catalina as I am still waiting for Apple to fix some macOS regressions. Without adding new ones. Still waiting. This gives me the ability to test IINA under macOS 10.15.7.
Apple made a massive amount of incompatible stylistic changes in Big Sur. The Free Pascal Wiki page macOS Big Sur changes for developers details the changes. I've been working with @svobs and checking to make sure UI PRs look good in both Ventura and Catalina. This PR looks nice in Ventura, but this is what I see in Catalina:
Can you attach a picture of what our preference looked like on Catalina on the dev branch? |
I revert back the changes for 10.15 and above, please check how it looks on Catalina |
Worked under macOS Ventura, crashed under macOS Catalina. With the latest commit I'm getting this warning from Xcode:
I'm guessing it is crashing on this line in
This is the relevant portion of the crash report:
|
@uiryuu you are getting a null value for This is a constraint which you are attaching to a single It doesn't really make sense to try to inject the constraint into Unfortunately, since this table isn't using a |
Thanks for the great info! I also did some research a little bit, and found I might have to change the table view to view based, which I would not bother doing right now. I'll convert this to draft until I find a good way to solve the issue. |
Description:
This commit:
Before:

After:
