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

Removed "final" on StyledLabel #8

Merged
merged 1 commit into from Jul 10, 2016
Merged

Conversation

mjrehder
Copy link

I am creating a new component, which depends on this control and I would like to use the same styling elements in order to have matching styles. The new component needs to derive from the StyledLabel (extension is not adequate), so it would be nice if the "final" declaration is removed from StyledLabel.

@codecov-io
Copy link

Current coverage is 58.26%

Merging #8 into master will decrease coverage by 4.56%

@@             master         #8   diff @@
==========================================
  Files             8          8          
  Lines           460        460          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits            289        268    -21   
- Misses          171        192    +21   
  Partials          0          0          

Powered by Codecov. Last updated by 88f7b07...957b434

@yannickl
Copy link
Owner

Ok, this is a good use case. Usually I make all my classes final by default, I remove the keyword only when a use case seems relevant like here. :)

@yannickl yannickl merged commit 74c4cc1 into yannickl:master Jul 10, 2016
@mjrehder
Copy link
Author

Nothing against the final by default, it is a good principle. I need a couple of days for the first iteration of my new styled components. Maybe it will make sense at some point in time to refactor the StyledLabel out into its own repo.

@mjrehder
Copy link
Author

I have created a new pod with the mentioned project: https://github.com/mjrehder/MJRFlexStyleComponents
However, it seems that I cannot depend on the master branch of SnappingStepper when publishing to cocoapods. Version 2.3.0 still has the "final" class for StyledLabel.
Any chance of making a new version? Please :-)

yannickl added a commit that referenced this pull request Jul 28, 2016
@yannickl
Copy link
Owner

Sorry, I was waiting for any new feature or other, but nothing has changed.

I just published a new version (2.3.1) with the last changes.

Best

@mjrehder
Copy link
Author

Ok, thanks. Just released MJRFlexStyleComponents to cocoapods sucessfully.

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.

None yet

3 participants