Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix dropdown overflow & add border to reactionpick #3520

Merged
merged 1 commit into from Feb 5, 2023
Merged

Conversation

LudvigHz
Copy link
Member

@LudvigHz LudvigHz commented Feb 5, 2023

Description

Fix dropdown overflow.
Add border to reactionpicker popover to get more depth and contrast.

Result

before after
image image
image image

Testing

  • I have thoroughly tested my changes.

Resolves ABA-242

@linear
Copy link

linear bot commented Feb 5, 2023

ABA-242 Overflow issue on Dropdown and Popover

Now that they have a bigger border-radius, it is more noticeable that its content overflows. E.g., open header dropdown, and hover over the top or bottom row.

Because of the caret (arrow at the top), we cannot use overflow: hidden.

@github-actions github-actions bot added the review-needed Pull requests that need review label Feb 5, 2023
Copy link
Member

@ivarnakken ivarnakken left a comment

Choose a reason for hiding this comment

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

💯

app/styles/utilities.css Outdated Show resolved Hide resolved
@ivarnakken ivarnakken added bug-fix Pull requests that fix a bug approved Pull requests that have been approved changes-requested Pull requests with requested changes and removed review-needed Pull requests that need review labels Feb 5, 2023
@github-actions github-actions bot added the review-needed Pull requests that need review label Feb 5, 2023
@ivarnakken ivarnakken removed review-needed Pull requests that need review changes-requested Pull requests with requested changes labels Feb 5, 2023
@github-actions github-actions bot added the review-needed Pull requests that need review label Feb 5, 2023
@LudvigHz LudvigHz merged commit bb3f7e8 into master Feb 5, 2023
@LudvigHz LudvigHz deleted the popover-fixes branch February 5, 2023 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Pull requests that have been approved bug-fix Pull requests that fix a bug review-needed Pull requests that need review
Projects
None yet
2 participants