-
Notifications
You must be signed in to change notification settings - Fork 317
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
Long title text overflow everywhere #124
Comments
Well, what's your expected behaviour? I would say that should be truncated on your side with e.g. On the other hand - we could just set it up properly so there's native |
@grabbou I'll add |
Sorry I didn't set expectations, I think normally you want to have the text truncated with '...' At the end. It's not possible to use substring because you don't know the width available and the character size. I suggested to use numberOfLines and remove the absolute positioning. |
Sorry for the very late response, but can you compose a PR? |
@ex/navigator truncates, so I think we could also |
Any news on this? Running into the same issue with long titles |
You can set maxHeight in style for Text |
Has there been an update for this @ngankt2? I'm sure I tried |
Bump... |
IIRC, we using absolute positioning for the title, so no boundaries for the text, unless you specify them by yourself |
Yes. Ordinarily the title boundaries will expand to accommodate the
enclosed text, because the parent (styles.navBar) is flex with
justifyContent = space-between.
But when the space occupied by the text is dynamic, adjustsFontSizeToFit is
trying to fit the text into a variable-size container and it ends up
deciding to use a very small font size. I'm not sure how to fix this.
…On Sat, Dec 3, 2016 at 2:44 AM, Alexey ***@***.***> wrote:
IIRC, we using absolute positioning for the title, so no boundaries for
the text, unless you specify them by yourself
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#124 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AA7l1y2WcolcUKQf2Fu2vMVx5xRjWo2kks5rETnggaJpZM4G5VFc>
.
|
Yeah, it's a tricky one. I can refactor |
@Kureev Thanks. I tried several changes but finally got it to work by adding Thanks |
@breadadams |
Thanks, @ngankt2. Should we make it default for the title? Can somebody compose a PR? |
If you use a long title it doesn't truncate and instead overflow (because in your component style you use absolute positioning and doesn't set 'numberOfLines' on the Text component).
See:
The text was updated successfully, but these errors were encountered: