Header of diff in the editor is missing when using live preview.
Live preview disabled:
Live preview enabled:
The text "Latest revision / Your text" in missing in the second screenshot.
Description
Description
Live preview enabled:
Details
Details
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| mediawiki.action.edit.preview: Use same UX as in normal diff | mediawiki/core | master | +37 -3 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T41272 Bugs in "Live preview" feature (tracking) | |||
| Resolved | Fomafix | T293307 Header of diff in the editor missing when using live preview |
Event Timeline
Comment Actions
Right, I noticed this while working on the live preview code for T109166, but forgot to create a task. I think partly it might be because I wasn't sure about the value of having these headers. On second thought though, I think we should include them.
Note: this was not caused by the recent work on the live preview, the headers were never there.
It's also worth noting that:
- The HTML for the headers is generated in DifferenceEngine::addHeader(), called by ::getDiff(), but ApiCompare doesn't use that (it only gets the diff body)
- The specific messages used there are chosen in EditPage::showDiff; in particular, the "Latest revision" one may vary depending on the page.
Perhaps the logic for choosing the messages might be moved out of EditPage into some diff-related class, and ApiCompare could have an option to use that. I'm not sure where the concept of "diffing the latest revision and some new text" should live.
Comment Actions
Change 732029 had a related patch set uploaded (by Fomafix; author: Fomafix):
[mediawiki/core@master] mediawiki.action.edit.preview: Use same UI as in normal diff
Comment Actions
Change 732029 merged by jenkins-bot:
[mediawiki/core@master] mediawiki.action.edit.preview: Use same UX as in normal diff