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

#2406 - payment table UI issue #2410

Merged
merged 2 commits into from
Nov 25, 2024
Merged

Conversation

SebinSong
Copy link
Collaborator

@SebinSong SebinSong commented Nov 1, 2024

closes #2406
closes #2415

As mentioned in this Slack thread, horizontal scrollbar creates another critical issue where the menu dropdown is cropped or even not visible at all in some cases.
So decided to revert the previous fix and instead extended current css media-queries settings to make sure nothing overflows the table layout.

[case 1 - table has enough space]

image


[case 2- table does not have enough space]

image

Hope it looks good!

@SebinSong SebinSong self-assigned this Nov 1, 2024
Copy link

cypress bot commented Nov 1, 2024

group-income    Run #3370

Run Properties:  status check passed Passed #3370  •  git commit 99e89a1272 ℹ️: Merge 091647d879cfb5ceac40667e1cdf4e177efb0f54 into 3b142e94445de8ab49b435e8d4bd...
Project group-income
Branch Review sebin/task/#2406-payment-table-issue
Run status status check passed Passed #3370
Run duration 09m 04s
Commit git commit 99e89a1272 ℹ️: Merge 091647d879cfb5ceac40667e1cdf4e177efb0f54 into 3b142e94445de8ab49b435e8d4bd...
Committer Sebin Song
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 10
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 111
View all changes introduced in this branch ↗︎

@corrideat
Copy link
Member

LGTM

Copy link
Member

@taoeffect taoeffect left a comment

Choose a reason for hiding this comment

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

Nice!

@taoeffect taoeffect merged commit 05835cb into master Nov 25, 2024
4 checks passed
@taoeffect taoeffect deleted the sebin/task/#2406-payment-table-issue branch November 25, 2024 18:04
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.

Ensure menu expands in visible area CSS bug in Payments table row
3 participants