Skip to content

Fix long unbroken text overflow in resume preview#21

Open
Jiya3177 wants to merge 1 commit into
sunilkumar2170:mainfrom
Jiya3177:fix-long-text-overflow
Open

Fix long unbroken text overflow in resume preview#21
Jiya3177 wants to merge 1 commit into
sunilkumar2170:mainfrom
Jiya3177:fix-long-text-overflow

Conversation

@Jiya3177
Copy link
Copy Markdown

Related Issue

Closes #19

What changed?

Fixed long unbroken text overflowing outside the resume preview in:

  • Profile website/URL field
  • Project description bullet text

Why?

Long unbroken strings such as long URLs or single-token descriptions had no natural wrap points, so they could overflow outside the resume preview layout.

How I fixed it

  • Added a helper to insert safe zero-width soft break opportunities into long unbroken text.
  • Applied it to shared bullet list rendering for project descriptions.
  • Applied it to profile contact text rendering for long website/URL values.
  • Added minimal flex constraints to the profile contact row so text can shrink and wrap inside the preview width.

How to test?

  1. Run npm install
  2. Run npm run dev
  3. Open /resume-builder
  4. Add a long unbroken website URL in the Profile section
  5. Add a long unbroken Project description
  6. Confirm both wrap inside the resume preview instead of overflowing

Screenshots

Before:
Screenshot 2026-05-14 at 1 45 37 PM

After:

Screenshot 2026-05-14 at 1 46 50 PM

Validation

  • Tested locally with long unbroken URL text
  • Tested locally with long unbroken project description text
  • Checked normal content still displays correctly
  • Verified main app pages still load
  • Ran npm run lint

Note: npm run lint shows a pre-existing warning in src/app/lib/redux/hooks.tsx about a missing dispatch dependency, unrelated to this PR.

@Jiya3177
Copy link
Copy Markdown
Author

Hi @sunilkumar2170,

I have submitted the PR for issue #19 and included:

  • Before/after screenshots
  • Local testing steps
  • npm run lint validation
  • Note about the pre-existing unrelated lint warning

The PR has no merge conflicts and is ready for review. Please review it when you get time.

Thank you!

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.

Bug: Long unbroken text overflows outside resume preview in URL and project description fields

1 participant