Skip to content

fix(item): label does not expand indefinitely #28700

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

Merged
merged 2 commits into from
Dec 13, 2023
Merged

fix(item): label does not expand indefinitely #28700

merged 2 commits into from
Dec 13, 2023

Conversation

liamdebeasi
Copy link
Contributor

@liamdebeasi liamdebeasi commented Dec 13, 2023

Issue number: N/A


What is the current behavior?

While working on an unrelated task, I noticed the list items in the list starter app run off the page. This is due to the text wrapping behavior in v7.6 where we allows the labels to grow/shrink. However, we do not set a limit on how much they can grow. This means that child elements inside of ion-label with long text can cause the label container to grow indefinitely.

What is the new behavior?

  • Labels in item have a max-width of 100%

Does this introduce a breaking change?

  • Yes
  • No

Other information

Dev build: 7.6.1-dev.11702498225.1ad6eda6

before after
localhost_8102_home(Samsung Galaxy S8+) localhost_8102_home(Samsung Galaxy S8+) (1)

@github-actions github-actions bot added the package: core @ionic/core package label Dec 13, 2023
@liamdebeasi liamdebeasi marked this pull request as ready for review December 13, 2023 20:25
@liamdebeasi liamdebeasi requested review from brandyscarney and a team as code owners December 13, 2023 20:25
Copy link
Member

@brandyscarney brandyscarney left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good but should we put this in the item test since it only applies when a label is inside of an item? I am good with either way though!

@liamdebeasi liamdebeasi added this pull request to the merge queue Dec 13, 2023
Merged via the queue into main with commit bc51dd0 Dec 13, 2023
@liamdebeasi liamdebeasi deleted the ld/wrap-fix branch December 13, 2023 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: core @ionic/core package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants