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(WEB-7276): avoid sliver of content being viewable below the StickyFooter #3963

Closed
wants to merge 1 commit into from

Conversation

jkatofskyCoveo
Copy link
Contributor

Proposed Changes

Put down the sticky footer down by one pixel to avoid the visual issue reported here

@jkatofskyCoveo jkatofskyCoveo requested a review from a team as a code owner November 22, 2024 20:18
@jkatofskyCoveo jkatofskyCoveo requested review from GermainBergeron and FelixBlaisThon and removed request for a team November 22, 2024 20:18
Copy link

@FelixBlaisThon
Copy link
Contributor

I couldn't see the sliver in your previous PR. was it on a 4k screen ?

@jkatofskyCoveo
Copy link
Contributor Author

@FelixBlaisThon, hmm, it's finnicky to replicate, but I don't think it has to do with 4K, rather I think it's a rounding error with how CSS calculates the bottom of a sticky element or something. Here is the modal in dev on my laptop screen (1920 X 1200). You can see the sliver
image

@FelixBlaisThon
Copy link
Contributor

I'm sorry to be annoying, but I am not able to replicate even using the devTools of my browser (chrome in my case) and explicitly setting my resolution to 1920x1200 (see screenshot). Also, when I add bottom: -1px to the stickyfooter, I have a flickering footer (see recording below)

image
Screen.Recording.2024-11-27.at.9.36.25.AM.mov

What are your browser and OS ?

@jkatofskyCoveo
Copy link
Contributor Author

Haha no you're not annoying, it's this bug that is annoying @FelixBlaisThon 😅

Windows 11, Chrome 131.

The flickering footer is indeed another issue, but for those of us who can replicate the sliver of content underneath, the flickering happens with or without the bottom: -1px 😕

@FelixBlaisThon
Copy link
Contributor

Haha no you're not annoying, it's this bug that is annoying @FelixBlaisThon 😅

Windows 11, Chrome 131.

The flickering footer is indeed another issue, but for those of us who can replicate the sliver of content underneath, the flickering happens with or without the bottom: -1px 😕

Ok. I think we have to maybe check that before merging any fix (at least remove any flickering, with or without the bottom: -1px.. I will create a JIRA so that we can investigate it. From the time being, I would suggest to close this PR and link the JIRA to this one https://coveord.atlassian.net/browse/ADUI-10401

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