[web] fix leave channel modal for long thread names
ClosedPublic
Actions

Authored by ginsu on Jun 27 2024, 12:45 AM.

Details

Reviewers

inka
ashoat

Commits

rCOMM1e78a6e4f88a: [web] fix leave channel modal for long thread names

Summary

This fixes a bug where the modal would gorw really wide if the thread name has a really long name. This diff fixes this bug

Linear task: https://linear.app/comm/issue/ENG-8438/leave-channel-modal-on-web-grows-when-a-thread-has-a-really-long-name

Test Plan

Please see the screenshots

Before:

Screenshot 2024-06-27 at 3.42.59 AM.png (1×3 px, 619 KB)

After:

Screenshot 2024-06-27 at 3.43.06 AM.png (2×3 px, 862 KB)

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ginsu created this revision.Jun 27 2024, 12:45 AM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptJun 27 2024, 12:45 AM

ginsu edited the summary of this revision. (Show Details)Jun 27 2024, 12:46 AM

ginsu edited the test plan for this revision. (Show Details)

ginsu added a reviewer: inka.

ginsu attached a referenced file: F2125414: Screenshot 2024-06-27 at 3.43.06 AM.png. (Show Details)

ginsu attached a referenced file: F2125413: Screenshot 2024-06-27 at 3.42.59 AM.png. (Show Details)

Harbormaster completed remote builds in B29947: Diff 41744.Jun 27 2024, 1:01 AM

ginsu requested review of this revision.Jun 27 2024, 1:01 AM

Don't love hardcoding pixel values like this. Should it be a CSS variable?

This revision is now accepted and ready to land.Jul 1 2024, 11:50 AM

address comments + rebase before landing

Harbormaster failed remote builds in B30034: Diff 41861!Jul 1 2024, 1:35 PM

Harbormaster completed remote builds in B30034: Diff 41861.Jul 1 2024, 2:02 PM

Closed by commit rCOMM1e78a6e4f88a: [web] fix leave channel modal for long thread names (authored by ginsu). · Explain WhyJul 1 2024, 8:39 PM

This revision was automatically updated to reflect the committed changes.

ginsu added a commit: rCOMM1e78a6e4f88a: [web] fix leave channel modal for long thread names.

Revision Contents
Changeset List

Path

Size

web/

modals/

modal.css

9 lines

threads/

confirm-leave-thread-modal.css

2 lines

Diff 41869

View Options

web/modals/modal.css