Page MenuHomePhabricator

Footer should not have top shadow if it is not floating
Closed, ResolvedPublicBUG REPORT

Description

Steps to replicate the issue (include links if applicable):

What happens?:

  • Footer has top shadow

image.png (281×2 px, 39 KB)

What should have happened instead?:

  • Footer does not have top shadow

image.png (240×2 px, 17 KB)

Software version (skip for WMF-hosted wikis like Wikipedia):

Other information (browser name/version, screenshots, etc.):

Event Timeline

Scardenasmolinar changed the task status from Open to In Progress.Aug 11 2023, 11:59 PM
Scardenasmolinar claimed this task.
Scardenasmolinar moved this task from Ready to In Progress on the Moderator-Tools-Team (Kanban) board.

Change 949559 had a related patch set uploaded (by Scardenasmolinar; author: Scardenasmolinar):

[mediawiki/extensions/PageTriage@master] [WIP]Fix footer shadow

https://gerrit.wikimedia.org/r/949559

Change 950001 had a related patch set uploaded (by Jsn.sherman; author: Jsn.sherman):

[mediawiki/extensions/PageTriage@master] Example followup to: Fix footer shadow

https://gerrit.wikimedia.org/r/950001

Change 950001 abandoned by Jsn.sherman:

[mediawiki/extensions/PageTriage@master] Example followup to: Fix footer shadow

Reason:

https://gerrit.wikimedia.org/r/950001

Change 949559 merged by jenkins-bot:

[mediawiki/extensions/PageTriage@master] Fix footer shadow

https://gerrit.wikimedia.org/r/949559

May need more work. The shadow correctly disappears when you switch filters from a list with a lot of items to a list with zero items, but when you scroll up, the shadow prematurely comes back.

image.png (355×1 px, 60 KB)

Change 954978 had a related patch set uploaded (by Scardenasmolinar; author: Scardenasmolinar):

[mediawiki/extensions/PageTriage@master] Add guard clause for shadow footer edge case

https://gerrit.wikimedia.org/r/954978

Change 955348 had a related patch set uploaded (by Jsn.sherman; author: Jsn.sherman):

[mediawiki/extensions/PageTriage@master] [Example] Observe results list instead of mw footer for list stats nav shadow

https://gerrit.wikimedia.org/r/955348

Change 955348 abandoned by Jsn.sherman:

[mediawiki/extensions/PageTriage@master] [Example] Observe results list instead of mw footer for list stats nav shadow

Reason:

https://gerrit.wikimedia.org/r/955348

Change 954978 merged by jenkins-bot:

[mediawiki/extensions/PageTriage@master] Refactor IntersectionObserver for footer shadow

https://gerrit.wikimedia.org/r/954978