Skip to content

Fix UI issues with event import and display#80

Merged
travisdock merged 1 commit intomainfrom
ui-issues
Feb 2, 2026
Merged

Fix UI issues with event import and display#80
travisdock merged 1 commit intomainfrom
ui-issues

Conversation

@travisdock
Copy link
Owner

  • Fix import preview modal not visible on long pages by using min-h-screen instead of min-h-full for proper viewport centering
  • Fix long URLs overflowing in event show page by adding break-words to location and description fields
  • Allow vertical resizing of description textarea in event form

  - Fix import preview modal not visible on long pages by using min-h-screen
    instead of min-h-full for proper viewport centering
  - Fix long URLs overflowing in event show page by adding break-words to
    location and description fields
  - Allow vertical resizing of description textarea in event form
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Improves Events UI behavior for long content and long pages by adjusting Tailwind utility classes to prevent overflow and ensure modals are properly centered in the viewport.

Changes:

  • Prevent long location/description strings (e.g., URLs) from overflowing on the event show page by enabling word breaking.
  • Ensure the import preview modal centers correctly on long pages by using viewport-based minimum height.
  • Allow vertical resizing of the event description textarea in the event form.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
app/views/events/show.html.erb Adds break-words to location/description display to prevent overflow from long strings.
app/views/events/_import_form.html.erb Switches modal container from min-h-full to min-h-screen for reliable viewport centering.
app/views/events/_form.html.erb Changes textarea from resize-none to resize-y to allow vertical resizing.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@travisdock travisdock merged commit 66f154c into main Feb 2, 2026
11 checks passed
@travisdock travisdock deleted the ui-issues branch February 2, 2026 13:08
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.

1 participant