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

"Appear disabled while paused". #46

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

ErezVolk
Copy link

Thanks for a wonderful little app! Here's something I find useful. I like to be able to tell at a glance whether the timer is paused or not. I've added an option to gray out the time.

This is my first time ever writing anything for OSX, using Xcode, etc, so everything may be horribly wrong.

bildschirmfoto 2017-12-23 um 12 34 54

if ([self.stopwatch isStopped]) {
[statusItem setLength:26.0];
Copy link
Author

Choose a reason for hiding this comment

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

The docs said setTitle etc. were deprecated and you should now use the button. If this is breaking anything, it can go, of course.

Copy link

@smokris smokris left a comment

Choose a reason for hiding this comment

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

.

Repository owner deleted a comment from ErezVolk May 12, 2019
Copy link

@smokris smokris left a comment

Choose a reason for hiding this comment

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

Oh, I should've explained — when I built your PR and ran it and opened the Preferences window, I got an error message saying that it couldn't connect appearDisabledWhilePausedButton to its outlet in the class PreferencesWindowController (whose property name lacks the Button suffix).

Going the other direction (adding the Button suffix to the property) sounds good to me. I've added those change suggestions.

PreferencesWindowController.h Outdated Show resolved Hide resolved
PreferencesWindowController.h Outdated Show resolved Hide resolved
PreferencesWindowController.m Outdated Show resolved Hide resolved
PreferencesWindowController.m Outdated Show resolved Hide resolved
ErezVolk and others added 5 commits May 12, 2019 17:50
Fix by @smokris for `Failed to connect (appearDisabledWhilePausedButton) outlet from (PreferencesWindowController) to (NSButton)`

Co-Authored-By: Steve Mokris <smokris@softpixel.com>
Fix by @smokris for `Failed to connect (appearDisabledWhilePausedButton) outlet from (PreferencesWindowController) to (NSButton)`

Co-Authored-By: Steve Mokris <smokris@softpixel.com>
Fix by @smokris for `Failed to connect (appearDisabledWhilePausedButton) outlet from (PreferencesWindowController) to (NSButton)`

Co-Authored-By: Steve Mokris <smokris@softpixel.com>
Fix by @smokris for `Failed to connect (appearDisabledWhilePausedButton) outlet from (PreferencesWindowController) to (NSButton)`

Co-Authored-By: Steve Mokris <smokris@softpixel.com>
@ErezVolk
Copy link
Author

Update: I upgraded DDHotKey and fixed its usage (looks like you're not supposed to create your own DDHotKeyCenter anymore, and use [DDHotKeyCenter sharedHotKeyCenter] instead).

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