Page MenuHomePhabricator

Draft email announcement about proposed change to output <figure> from PHP parser for images
Closed, ResolvedPublic

Description

https://gerrit.wikimedia.org/r/#/c/196532/ proposes changing output of images in PHP parser to use <figure>, thus moving parser output closer to Parsoid's semantic markup.

But, based on discussion, it seemed better to get word about this: why this is a good thing to do, what are the implications on tools and scripts out there, how to fix them, etc.

Please share a draft (link to etherpad / google docs, whatever) for others to comment on.

Event Timeline

ssastry assigned this task to cscott.
ssastry raised the priority of this task from to High.
ssastry updated the task description. (Show Details)
ssastry added subscribers: ssastry, Krinkle, GWicke.

So I remember:
(11:10:56 AM) gwicke: cscott: btw, <figure> will need a small tweak to work in IE 6-8
(11:11:10 AM) gwicke: as in "won't be dropped in the parser"
(11:12:10 AM) gwicke: since we'll style those figures it's important to preserve them, but it's 1-2 lines of JS to register those tags
(11:22:18 AM) cscott-free: document.createElement('figure'), I believe, right?

http://knockoutjs.com/documentation/component-custom-elements.html#note-custom-elements-and-internet-explorer-6-to-8

ssastry lowered the priority of this task from High to Medium.
ssastry added a subscriber: cscott.

@Arlolra I reassigned this since you've been working on this code. Notices about these changes have gone out as part of SoS but we should do a clear announcement once closer to when this is going to actually be done. Maybe the RFC process is it, but, one way or other, this change needs to be communicated.

Krinkle renamed this task from Draft email announcement about proposed changes to PHP parser image output in https://gerrit.wikimedia.org/r/#/c/196532/ to Draft email announcement about proposed change to output <figure> from PHP parser for images.Apr 11 2018, 2:37 AM
Krinkle updated the task description. (Show Details)
Aklapper added a subscriber: Arlolra.

Removing task assignee due to inactivity, as this open task has been assigned to the same person for more than two years (see the emails sent to the task assignee on Oct27 and Nov23). Please assign this task to yourself again if you still realistically [plan to] work on this task - it would be welcome.
(See https://www.mediawiki.org/wiki/Bug_management/Assignee_cleanup for tips how to best manage your individual work in Phabricator.)

Arlolra raised the priority of this task from Medium to High.

@Arlolra the mailing list message says "This will be included in Tech News", but I do not see it when searching the archives. Is this planned?

Edit: Sorry, disregard, looks like it *was* mentioned and I just missed it.