[wp-trac] [WordPress Trac] #49540: Regression: Copy action now scrolls up the page on the privacy policy guide.
WordPress Trac
noreply at wordpress.org
Mon Mar 2 17:30:09 UTC 2020
#49540: Regression: Copy action now scrolls up the page on the privacy policy
guide.
-------------------------------------------------+-------------------------
Reporter: garrett-eclipse | Owner: garrett-
| eclipse
Type: defect (bug) | Status: accepted
Priority: normal | Milestone: 5.4
Component: Privacy | Version: trunk
Severity: normal | Resolution:
Keywords: has-patch has-screenshots needs- | Focuses: ui
testing |
-------------------------------------------------+-------------------------
Comment (by garrett-eclipse):
Hi @birgire that matches my install. It's the change to using `display:
none` that causes the issue of the jump and is what the latest patch
addresses. Previously it was just changing visibility but that left large
gaps in the copied text so the display none ensured that space was removed
but causes a jump in the page on certain installs.
Are you on Windows per-chance? I noticed Firefox wasn't affected by this
and some other browsers might not have the issue either so I ensured the
patch didn't negatively affect those browsers while addressing the ones of
issue.
If you inspect and put a breakpoint just after that addClass javascript
line it should pause what's happening to the point you should see your
page has gotten much shorter. And then when that class is removed the page
will become longer again but your scroll position may be left half-way up
the page. If it returns your view back then maybe try to reproduce on
another browser. The ones i found of issue were all on mac; Chrome, Opera,
Brave, Safari.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/49540#comment:11>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list