Page MenuHomePhabricator

Cursor pushed off screen when focussing first line in iOS
Closed, ResolvedPublic

Event Timeline

Safari scrolls to the wrong position. Here's a standalone example that exhibits the same behavior, and a recording:

Apparently the issue is easily solved by adding some padding to the contenteditable element. Uncomment the padding-top: 1em; line in the example, and you get:

Conveniently, we already have padding-top matching the height of the toolbar, but it's currently set on a different element.

Change 499590 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[mediawiki/extensions/VisualEditor@master] ve.init.mw.MobileArticleTarget: Improve iOS Safari by moving 'padding-top' for toolbar

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

Change 499590 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] ve.init.mw.MobileArticleTarget: Improve iOS Safari by moving 'padding-top' for toolbar

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

Change 499870 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[mediawiki/extensions/VisualEditor@master] ve.init.mw.MobileArticleTarget: Set 'padding-top' on the right node when section-editing

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

Change 499870 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] ve.init.mw.MobileArticleTarget: Set 'padding-top' on the correct node when section-editing

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

Change 500630 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[mediawiki/extensions/VisualEditor@master] ve.init.mw.MobileArticleTarget: Fix placeholder being hidden by the toolbar

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

Change 500630 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] ve.init.mw.MobileArticleTarget: Fix placeholder being hidden by the toolbar

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