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

Additional property that allows positioning of image relative to text #242

Merged
merged 8 commits into from Aug 31, 2017
Merged

Conversation

lordzsolt
Copy link
Contributor

Added imagePosition property. The image can be left, right, above, below or behind the text.
Both dynamic and fixed width is supported.

screen shot 2016-11-02 at 6 18 26 pm

screen shot 2016-11-02 at 6 19 29 pm

@HeshamMegid
Copy link
Owner

Thanks for the pull request and the awesome work! This version of HMSegmentedControl is currently in maintenance mode, so I'm only merging bug-fix pull requests, not new features. Feel free to contribute to the brand new Swift version.

@HeshamMegid HeshamMegid closed this Nov 4, 2016
@gautamjain987
Copy link

I don't get it. It's such a useful feature. I would love to see it get merged.

Also, there's a bug fix for text and image overlapping, which you've reverted. Any reason why ?
@HeshamMegid

@sara-shokry
Copy link

this is really useful feature , hope to be merged soon

@HeshamMegid HeshamMegid reopened this Aug 31, 2017
@HeshamMegid
Copy link
Owner

Okay, let's merge this!

I would be grateful if you update the branch to the latest version of master to be able to merge it.

Zsolt Kovacs added 2 commits August 31, 2017 10:51
# Conflicts:
#	HMSegmentedControl/HMSegmentedControl.m
@lordzsolt
Copy link
Contributor Author

Should be up to date now :)

@HeshamMegid HeshamMegid merged commit 16fad4a into HeshamMegid:master Aug 31, 2017
@HeshamMegid
Copy link
Owner

@lordzsolt awesome, thanks.

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

4 participants