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

The label gets out of bounds in example #19

Closed
gerchicov-bp opened this issue May 29, 2018 · 5 comments
Closed

The label gets out of bounds in example #19

gerchicov-bp opened this issue May 29, 2018 · 5 comments
Assignees
Labels
bug Something isn't working

Comments

@gerchicov-bp
Copy link

iPhone 5s simulator:
simulator screen shot - iphone 5s - 2018-05-29 at 15 13 49

@gerchicov-bp gerchicov-bp changed the title The label gets out of bounds The label gets out of bounds in example May 29, 2018
@richardtop
Copy link

I can confirm the issue on iPhone SE.

@croossin croossin added the bug Something isn't working label May 29, 2018
@lwdupont
Copy link
Contributor

@croossin Is this an issue in the CardParts code, or just the example? Can I take a poke around at it?

@croossin
Copy link
Contributor

@lwdupont That would be awesome if you could!

@lwdupont
Copy link
Contributor

@croossin So I looked at it.

Thinking CardParts should handle this, I traced into where it's setting up the CardPartsTitleView. I tweaked updateContraints() slightly to pin the horizontal to both edges. I'll open up a pull request soonish, maybe tomorrow.

Results:

iphone 8 plus
iphone se
iphone x

lwdupont pushed a commit that referenced this issue May 31, 2018
…rtTitleView. The CardPartTitleView was setting the horizontal constraint to pin left, but nothing on the right, so the UILabel just floated off of the containing CardPartVIew.

Figured this fix was good because then the developer can decide if they want to go further in fixing up their label (vs CardParts deciding for them).
croossin added a commit that referenced this issue May 31, 2018
…t-part

Tweak for issue #19 to fix the going off the right edge of the CardPartTitleView
@croossin
Copy link
Contributor

This was fixed in PR #23 and released in CardParts 2.3.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants