Page MenuHomePhabricator

Unexpected textarea left padding when editing code pages
Closed, ResolvedPublicBUG REPORT

Description

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

  • Enable WikiEditor 2010
  • Disable CodeMirror
  • Edit any code pages (JS/CSS/JSON/Lua etc.)

What happens?:

The textarea has a large left padding.

What should have happened instead?:

The textarea should only have a small padding.

Software version (on Special:Version page; skip for WMF-hosted wikis like Wikipedia):

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

The padding was added to prevent CodeMirror-related FOUC, but it should only be applied to users with CodeMirror enabled.

Event Timeline

Bhsd added a project: Regression.

Tagging Regression because it only happens after this week's deployment.

Change #1309553 had a related patch set uploaded (by Neelima25; author: Neelima25):

[mediawiki/extensions/CodeMirror@master] Fix: Unexpected textarea left padding when editing code pages

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

Hi @MusikAnimal, this regression is caused by this patchset in response to this comment. May I know your thoughts?

Sorry for the delayed response. I've been quite busy, but will resume focus on CodeMirror starting this weekend. This doesn't seem like an urgent issue but I will get it fixed (unless Neelimaa1 beats me to it :)

Change #1311605 had a related patch set uploaded (by Bhsd; author: Bhsd):

[mediawiki/extensions/CodeMirror@master] Hooks: load style module only if CodeMirror is enabled

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

This doesn't seem like an urgent issue but I will get it fixed (unless Neelimaa1 beats me to it :)

Or I can make a patch for it.

Change #1309553 abandoned by MusikAnimal:

[mediawiki/extensions/CodeMirror@master] Fix: Unexpected textarea left padding when editing code pages

Reason:

superseded by If19386da26. Thank you nonetheless for your contributions!

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

Change #1311605 merged by jenkins-bot:

[mediawiki/extensions/CodeMirror@master] Hooks: load style module only if CodeMirror is enabled

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

Bhsd claimed this task.
Bhsd moved this task from Bugs to Done on the MediaWiki-extensions-CodeMirror board.
Chuiimuii_ofc subscribed.

RoadToWiki board hygiene: moved this task to Completed so the board matches Phabricator status (resolved/closed). No change to task status or assignee.