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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

10.13.6 checkbox issues #716

Closed
jonsummers opened this issue Nov 23, 2021 · 7 comments
Closed

10.13.6 checkbox issues #716

jonsummers opened this issue Nov 23, 2021 · 7 comments
Labels
bug Something isn't working
Projects

Comments

@jonsummers
Copy link

Describe the bug
On 10.13.6 settings and app checkboxes are not shown.

Details:

  • Device: Macbook Pro 13" 2015
  • macOS: 10.13.6
  • Application version: 2.6.20

The checkboxes are the setting panel at top right, all settings.
And the app panel checkboxes "Show icon in dock" and "Start at login"

Correct behaviour (checkboxes show) on 10.14.4 and (as sliders) on 11.6

Fix:
I believe the fix is as follows:
button.isTransparent = false // was true ; as in func toggleSettingRow#
button.wantsLayer = true // added (maybe not required) copied per func toggleSettingRow##

in Kit/module/settings.swift: line 121 in func header
in Stats/Views/AppSettings.swift: line 244 in func toggleView

Fix does not change 10.14.4 behaviour and cannot affect 10.15+
#, ## reference is Kit/extension.swift func toggleSettingRow #line 256 and ##line 258

@jonsummers
Copy link
Author

Further, I believe the NSButton ctr in in Stats/Views/AppSettings.swift: line 239 in func toggleView
let button: NSButton = NSButton(frame: NSRect(x: 0, y: 0, width: 30, height: 20))
can be safely replaced with
let button: NSButton = NSButton()

@exelban
Copy link
Owner

exelban commented Nov 23, 2021

Hi. Thanks for the feedback.
Does I understand correctly, the checkbox does not shown in the settings?

@jonsummers
Copy link
Author

The 'master' checkbox that controls the enabling of the panel at the top right of each settings panel does not show.

@exelban
Copy link
Owner

exelban commented Nov 24, 2021

Could you please share the screenshot of the bug?
I don't have 10.13 to check this.

@jonsummers
Copy link
Author

Here are Settings Panel and AppSettings images from v2.6.20 on macOS 10.13.6

Settings10-13-6

AppSettings10-13-6

@jonsummers
Copy link
Author

And here are the same panels on macOS 10.13.6 with fix "button.isTransparent = false" applied
(The "642" version number is my own for tracking my internal development changes)

SettingsFIX10-13-6

AppSettingsFIX10-13-6

@exelban
Copy link
Owner

exelban commented Nov 26, 2021

Thanks, will be fixed.

@exelban exelban added the bug Something isn't working label Nov 26, 2021
@exelban exelban added this to To do in Roadmap Nov 26, 2021
@exelban exelban moved this from To do to In progress in Roadmap Nov 27, 2021
@exelban exelban moved this from In progress to Waiting for the next release in Roadmap Nov 27, 2021
@exelban exelban closed this as completed Nov 28, 2021
@exelban exelban moved this from Waiting for the next release to Done in Roadmap Nov 28, 2021
gmcinalli pushed a commit to gmcinalli/stats that referenced this issue Feb 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
No open projects
Roadmap
  
Done
Development

No branches or pull requests

2 participants