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

[ui] Versions page diff dropdown: z-index so you can click on labels #24068

Merged
merged 1 commit into from
Sep 26, 2024

Conversation

philrenaud
Copy link
Contributor

@philrenaud philrenaud commented Sep 26, 2024

Noticed that while the radio button itself was clickable, the labels in the dropdown fell beneath the .timeline text and so weren't clickable, and in some browsers even showed up behind the timeline itself.

image

Copy link

github-actions bot commented Sep 26, 2024

Ember Test Audit comparison

main e0db44c change
passes 1581 1581 0
failures 0 0 0
flaky 0 0 0
duration 11m 51s 852ms 11m 44s 245ms -07s 607ms

Copy link
Contributor

@pkazmierczak pkazmierczak left a comment

Choose a reason for hiding this comment

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

LGTM! Percy tests fail though?

@philrenaud
Copy link
Contributor Author

philrenaud commented Sep 26, 2024

Percy tests fail though?

Deliberately — it detects a visual diff that was intended (moved the stuff up in the z-index and over to the right). I've approved the changes in Percy now

@philrenaud philrenaud merged commit c07806e into main Sep 26, 2024
23 checks passed
@philrenaud philrenaud deleted the b-ui/fix-versions-compare-z-index branch September 26, 2024 16:34
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.

2 participants