Skip to content

Solution (#34526): Fix truncated label in microsuvery alert#34701

Open
TFGSUMIT wants to merge 1 commit into
mozilla-mobile:mainfrom
TFGSUMIT:fix/issue-34526
Open

Solution (#34526): Fix truncated label in microsuvery alert#34701
TFGSUMIT wants to merge 1 commit into
mozilla-mobile:mainfrom
TFGSUMIT:fix/issue-34526

Conversation

@TFGSUMIT

Copy link
Copy Markdown

This PR fixes the truncated label issue in the microsurvey alert by implementing a text wrapping mechanism. The changes include:

  • Adding a wrapText function to split input text into multiple lines based on a maximum character count.
  • Updating the multiLineTextView function to use the wrapped text.
  • Modifying the microsurveyAlert function to use the multiLineTextView function to display the wrapped text.

To test this PR, simply run the app and verify that the microsurvey alert text is properly wrapped and fits within the available space.

@TFGSUMIT
TFGSUMIT requested a review from a team as a code owner July 17, 2026 11:35
@TFGSUMIT
TFGSUMIT requested a review from Foxbolts July 17, 2026 11:35
@Foxbolts

Copy link
Copy Markdown
Contributor

Hey @TFGSUMIT, thanks for your contribution. Would you mind aligning your PR with the Pull Request Naming Guide as well as utilizing the PR template?

@Foxbolts

Copy link
Copy Markdown
Contributor

Just a heads up, another contributor opened #34652 which resolves the same issue. Will keep this open in case that one doesn't merge.

@janbrasna janbrasna left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm not sure this PR is even submitted in a working shape:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm pretty sure there's nothing that would work under this path.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Heh, would ya look at that, I didn't even review the diff 😂

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.

3 participants