Skip to content
This repository has been archived by the owner on Nov 1, 2017. It is now read-only.

Oops! Fixed scroll knob visibility glitch. #98

Merged
merged 2 commits into from
Oct 2, 2012
Merged

Oops! Fixed scroll knob visibility glitch. #98

merged 2 commits into from
Oct 2, 2012

Conversation

avaidyam
Copy link

@avaidyam avaidyam commented Oct 2, 2012

It's an error I made, and I shall fix it! :D

@@ -71,7 +71,13 @@ - (void)dealloc {

- (void)refreshKnobTimer {
if([NSScroller preferredScrollerStyle] == NSScrollerStyleOverlay) {
if(scrollView.scrollIndicatorStyle != TUIScrollViewIndicatorVisibleNever) {
TUIScrollViewIndicatorVisibility visibility;
if(self.isVertical)

Choose a reason for hiding this comment

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

Dot syntax shouldn't be used with is methods – use brackets or self.vertical instead.

@jspahrsummers
Copy link

📮

@ghost ghost assigned jspahrsummers Oct 2, 2012
@avaidyam
Copy link
Author

avaidyam commented Oct 2, 2012

Fixed.

@jspahrsummers jspahrsummers merged commit aef4682 into github:master Oct 2, 2012
@jspahrsummers jspahrsummers removed their assignment May 22, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants