Skip to content

fix(frontend - confirm-dialog): fixes #811 : prevent overflow of desc…#812

Open
wizard-knight wants to merge 1 commit into
srbhr:mainfrom
wizard-knight:bug/811-error_msg_overflow_in_modal
Open

fix(frontend - confirm-dialog): fixes #811 : prevent overflow of desc…#812
wizard-knight wants to merge 1 commit into
srbhr:mainfrom
wizard-knight:bug/811-error_msg_overflow_in_modal

Conversation

@wizard-knight

@wizard-knight wizard-knight commented May 28, 2026

Copy link
Copy Markdown

fix(frontend - confirm-dialog): fixes #811 : prevent overflow of description text in ConfirmDialog component

Pull Request Title

FIX #811 Prevent Overflow of error message in modal

Related Issue

#811

Description

copilot:summary
prevent overflow of description text in ConfirmDialog component by adding max-h-38, overflow-y-auto, and break-all tw classes

Type

  • Bug Fix
  • Feature Enhancement
  • Documentation Update
  • Code Refactoring
  • Other (please specify):

Proposed Changes

Added some tailwind classes; max-h-38, overflow-y-auto, and break-all, in confirm dialog dialog description

Screenshots / Code Snippets (if applicable)

Resume_Matcher__Issue__FIXED__error_overflow_in_modal

How to Test

  1. Create a resume
  2. Go to edit resume
  3. Click download such that an error is caused (as mentioned in issue)

Checklist

  • The code compiles successfully without any errors or warnings
  • The changes have been tested and verified
  • The documentation has been updated (if applicable)
  • The changes follow the project's coding guidelines and best practices
  • The commit messages are descriptive and follow the project's guidelines
  • All tests (if applicable) pass successfully
  • This pull request has been linked to the related issue (if applicable)

Additional Information

copilot:walkthrough


Summary by cubic

Prevents long error messages from overflowing in the ConfirmDialog modal by limiting height, enabling scroll, and breaking long strings. Fixes #811.

  • Bug Fixes
    • Added max-h-38, overflow-y-auto, and break-all to the dialog description so long text wraps and scrolls inside the modal.

Written for commit 9976714. Summary will update on new commits.

Review in cubic

… description text in ConfirmDialog component

@cubic-dev-ai cubic-dev-ai Bot 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.

No issues found across 1 file

Re-trigger cubic

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]: Error message inside modal during download is overflowing

1 participant