| bwang | |
| Jan 5 2024, 5:27 PM |
| F42398480: screenshot 621.png | |
| Mar 5 2024, 2:29 PM |
| F41648906: Screenshot 2024-01-02 at 1.23.32 PM.png | |
| Jan 9 2024, 5:52 PM |
| F41648903: Screenshot 2024-01-02 at 1.20.10 PM.png | |
| Jan 9 2024, 5:52 PM |
| F41648904: Screenshot 2024-01-02 at 1.23.45 PM.png | |
| Jan 9 2024, 5:52 PM |
| F41648902: Screenshot 2024-01-02 at 1.19.53 PM.png | |
| Jan 9 2024, 5:52 PM |
Due to the way the spacing on the pinnable element is implemented, it looks like this when the scrollbar is present
We could move the spacing from the pinnable element to its container to get something like this:
| AC | Status | Details |
|---|---|---|
| 1 | ✅ | T354431#9601140 |
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Move pinnable element spacing from the pinnable element to the container | mediawiki/skins/Vector | master | +6 -8 |
Change 979392 had a related patch set uploaded (by Bernard Wang; author: Bernard Wang):
[mediawiki/skins/Vector@master] Move pinnable element spacing from the pinnable element to the container
Change 979392 merged by jenkins-bot:
[mediawiki/skins/Vector@master] Move pinnable element spacing from the pinnable element to the container
Status: ✅ PASS
Environment: enwiki
OS: macOS Sonoma
Browser: Chrome
Device: MBA
Emulated Device:NA
Test Artifact(s):
✅ AC1: Verify the the spacing from the pinnable element is moved to its container to prevent the scrollbar from appearing above the pinnable element title
This was verified only in Prod in the previous deployment, it just needs someone other than me to sign it off.