Skip to content

Fixed a problem with selected title color for segment#25

Closed
petergam wants to merge 7 commits intonealyoung:masterfrom
petergam:master
Closed

Fixed a problem with selected title color for segment#25
petergam wants to merge 7 commits intonealyoung:masterfrom
petergam:master

Conversation

@petergam
Copy link
Contributor

A problem has added where the textColor of the NYSegmentLabel where changed, which caused a behaviour where the segment would keep to appear selected when moving the indicator. The textColor of the NYSegmentLabel should not be changed when selected since it is handled by the maskFrame instead.

@colasbd
Copy link

colasbd commented Sep 26, 2015

+1

This PR fixes the bug.

@neil-wu
Copy link

neil-wu commented Sep 29, 2015

well done

Peter Gammelgaard added 4 commits September 29, 2015 11:34
This is done to avoid a visual glitch when the selected title color is in a lighter color than the normal text color.
Further this commit adds support for a more smooth transition to a bold selected font
This reverts commit 49da75e, reversing
changes made to 66125d9.
Peter Gammelgaard added 2 commits September 30, 2015 09:10
This is done to avoid a visual glitch when the selected title color is in a lighter color than the normal text color.
Further this commit adds support for a more smooth transition to a bold selected font
@petergam
Copy link
Contributor Author

This PR was never merged. The changes in this PR is included in my new PR #35.

Closing this one.

@petergam petergam closed this Sep 30, 2015
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.

3 participants