ssastry (Subramanya Sastry)
User

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Sunday

  • Clear sailing ahead.

User Details

User Since
Oct 7 2014, 5:34 AM (163 w, 3 d)
Availability
Available
LDAP User
Subramanya Sastry
MediaWiki User
SSastry (WMF)

Recent Activity

Yesterday

ssastry added a comment to T178838: Followups needed with wikis for Tidy replacement.

For the 300+ small wikis that may be already good to go: just generate a list, please, I drafted the message for your review.

Thu, Nov 23, 6:12 PM · Community-Liaisons (Oct-Dec 2017), Tidy
ssastry added a comment to T178838: Followups needed with wikis for Tidy replacement.

there are a number of false positives [...] which will go away as the linter script i started y'day processes articles through all wikis

Could you tell me when/if this script is done?

I think that one is. Not sure about the status of the one that was launched after that, but we'll certainly know more next week.

Thu, Nov 23, 5:59 PM · Community-Liaisons (Oct-Dec 2017), Tidy

Wed, Nov 22

ssastry assigned T45094: Parsoid: References should be wrapped in a <sup>, not a <span> to Sbailey.
Wed, Nov 22, 7:08 PM · Parsoid
ssastry triaged T181190: Enable RemexHTML on itwiki as High priority.
Wed, Nov 22, 7:00 PM · Parsing-Team, Tidy, RemexHtml
ssastry triaged T181188: Enable RemexHTML on dewiki as High priority.
Wed, Nov 22, 7:00 PM · Parsing-Team, Tidy, RemexHtml
ssastry added a subtask for T175706: Progressively switch Wikimedia wikis from Tidy to RemexHTML: T181190: Enable RemexHTML on itwiki.
Wed, Nov 22, 6:59 PM · Wikimedia-Site-requests, Tidy, Parsing-Team
ssastry added a parent task for T181190: Enable RemexHTML on itwiki: T175706: Progressively switch Wikimedia wikis from Tidy to RemexHTML.
Wed, Nov 22, 6:59 PM · Parsing-Team, Tidy, RemexHtml
ssastry created T181190: Enable RemexHTML on itwiki.
Wed, Nov 22, 6:59 PM · Parsing-Team, Tidy, RemexHtml
ssastry added a parent task for T181188: Enable RemexHTML on dewiki: T175706: Progressively switch Wikimedia wikis from Tidy to RemexHTML.
Wed, Nov 22, 6:56 PM · Parsing-Team, Tidy, RemexHtml
ssastry added a subtask for T175706: Progressively switch Wikimedia wikis from Tidy to RemexHTML: T181188: Enable RemexHTML on dewiki.
Wed, Nov 22, 6:56 PM · Wikimedia-Site-requests, Tidy, Parsing-Team
ssastry created T181188: Enable RemexHTML on dewiki.
Wed, Nov 22, 6:56 PM · Parsing-Team, Tidy, RemexHtml

Tue, Nov 21

ssastry triaged T181095: Audit Parsoid's handling of wikitext and html strings in HTML attributes as Normal priority.
Tue, Nov 21, 9:14 PM · MediaWiki-Parser, Parsoid
ssastry created T181095: Audit Parsoid's handling of wikitext and html strings in HTML attributes.
Tue, Nov 21, 9:14 PM · MediaWiki-Parser, Parsoid
ssastry added a comment to T181086: Quote transformation bug.

This manifests as three lint errors on https://it.wikipedia.org/wiki/The_Amazing_Race_(sedicesima_edizione)

Tue, Nov 21, 8:05 PM · Parsoid
ssastry triaged T181086: Quote transformation bug as Normal priority.
Tue, Nov 21, 8:03 PM · Parsoid
ssastry created T181086: Quote transformation bug.
Tue, Nov 21, 8:03 PM · Parsoid

Mon, Nov 20

ssastry added a comment to T161556: Implement a way to have linter reprocess all pages .

@Elitre FYI about T181007 since that is causing some of the delays in linter updates in the UI. But, once the script I restarted ends tomrorow OR services fixes that issue (whichever happens first), the delay should go away.

Mon, Nov 20, 10:58 PM · Services (watching), Parsoid, MediaWiki-extensions-Linter
ssastry added a comment to T161556: Implement a way to have linter reprocess all pages .

I fixed some problems with the script that legoktm created and retarted the script with a high concurrency level on Tuesday, May 14th. It finished in about ~5 days processing pages on all active wikis (except commons and wikidata). A total of about ~80M pages were processed. However, I noticed that talk namespaces haven't been processed properly (and maybe other non-article namespaces .. I need to investigate that). So, for now, I restarted to process all talk namespace pages on all those wikis. It has been running for about 3 hours now and I expect it to finish in < 24 hours.

Mon, Nov 20, 10:46 PM · Services (watching), Parsoid, MediaWiki-extensions-Linter
ssastry triaged T180995: Batch API request error as Normal priority.
Mon, Nov 20, 8:12 PM · Parsoid
ssastry created T180995: Batch API request error.
Mon, Nov 20, 8:12 PM · Parsoid
ssastry added a comment to T161306: Investigate P-wrapping oddity that introduces long horizontal no-wrap lines on many navboxes on shwiki.

The Izer one doesn't do that: the Departmani Francuske does.

@ssastry is this just me? (Specifically, I see the issue in the top template only if I expand it after the bottom one is already expanded.)

Mon, Nov 20, 6:52 PM · Patch-For-Review, Tidy, Parsoid
ssastry added a comment to T180824: Parsoid should use <br/> to handle paragraph tags in single-line-mode wikitext constructs like list items.

Right now, Parsoid does this for <li>, <dd>, <dt>, and <h*> tags. We could use the <br/> solution for li, dd, dt tags. What about h* tags? Should we retain current behavior? Or is <br/> the expectation there?

Mon, Nov 20, 4:20 PM · Parsoid
ssastry renamed T180824: Parsoid should use <br/> to handle paragraph tags in single-line-mode wikitext constructs like list items from Hard line breaks (not paragraphs) render correctly but do not save properly to Parsoid should use <br/> to handle paragraph tags in single-line-mode wikitext constructs like list items.
Mon, Nov 20, 4:18 PM · Parsoid
ssastry triaged T180824: Parsoid should use <br/> to handle paragraph tags in single-line-mode wikitext constructs like list items as Normal priority.
Mon, Nov 20, 4:17 PM · Parsoid
ssastry added a comment to T180824: Parsoid should use <br/> to handle paragraph tags in single-line-mode wikitext constructs like list items.

Yes, this is deliberate and known behavior in Parsoid. For single-line mode constructs (like list items), we force conversion to single-line context. But, we could convert those to add <br/> instead, I suppose.

Mon, Nov 20, 4:16 PM · Parsoid
ssastry updated subscribers of T180938: ParserMigration extension is not correctly joined into Phabricator.

@tstarling @Legoktm @Jdforrester-WMF @Aklapper Anyone know why this isn't showing up on phabricator? (I see that linter repo is set up in phab).

Mon, Nov 20, 4:02 PM · MediaWiki-extensions-Other, Repository-Admins, Diffusion, Parsing-Team
ssastry triaged T180963: Investigate http 404 errors returned by Parsoid as Normal priority.
Mon, Nov 20, 3:43 PM · Parsoid
ssastry created T180963: Investigate http 404 errors returned by Parsoid.
Mon, Nov 20, 3:43 PM · Parsoid

Fri, Nov 17

ssastry added a member for RemexHtml: tstarling.
Fri, Nov 17, 2:08 PM
ssastry added a member for RemexHtml: ssastry.
Fri, Nov 17, 2:08 PM

Wed, Nov 15

ssastry added a comment to T178838: Followups needed with wikis for Tidy replacement.

At this time, among large wikis, I am concerned about enwiki -- there has not been much work / progress on the high priority categories.

Wed, Nov 15, 4:59 PM · Community-Liaisons (Oct-Dec 2017), Tidy
ssastry added a comment to T178838: Followups needed with wikis for Tidy replacement.

cebwiki will also be ready once the bot is done through its work since the biggest problem there is the one that is being fixed by the bot ( https://ceb.wikipedia.org/wiki/Espesyal:LintErrors )

Wed, Nov 15, 4:56 PM · Community-Liaisons (Oct-Dec 2017), Tidy
ssastry added a comment to T178838: Followups needed with wikis for Tidy replacement.

Sv.wp looking close, I hear?

Wed, Nov 15, 4:16 PM · Community-Liaisons (Oct-Dec 2017), Tidy
ssastry added a comment to T161306: Investigate P-wrapping oddity that introduces long horizontal no-wrap lines on many navboxes on shwiki.

https://parsoid-vd-tests.wikimedia.org/visualdiff/pngs/shwiki/Gonfaron.parsoid.png is a non working link. I wanted to show people what the problem is, when contacting them about it.

I refreshed the db since then with new titles.

But, https://parsoid-vd-tests.wikimedia.org/diff/shwiki/Balbins has links to different versions. You will see the same problem there.

Wed, Nov 15, 3:13 PM · Patch-For-Review, Tidy, Parsoid
ssastry added a comment to T161306: Investigate P-wrapping oddity that introduces long horizontal no-wrap lines on many navboxes on shwiki.

https://parsoid-vd-tests.wikimedia.org/visualdiff/pngs/shwiki/Gonfaron.parsoid.png is a non working link. I wanted to show people what the problem is, when contacting them about it.

Wed, Nov 15, 3:10 PM · Patch-For-Review, Tidy, Parsoid

Tue, Nov 14

ssastry triaged T180534: Failed template encapsulation as Normal priority.
Tue, Nov 14, 10:02 PM · Parsoid
ssastry added a comment to T180534: Failed template encapsulation.

Other titles: الحميدي_الحربيand روبيرتو_ماروني

Tue, Nov 14, 10:01 PM · Parsoid
ssastry created T180534: Failed template encapsulation.
Tue, Nov 14, 9:57 PM · Parsoid
ssastry added a comment to T180366: All Reading Infrastructure engineers should have deploy rights for all services Readers engineering maintains.

Thnx @Mholloway, +1 from me. This will also need approval from Gergo's direct manager. Could you add them and have them approve the request?

Tue, Nov 14, 2:27 PM · Ops-Access-Requests, Operations, Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban)

Thu, Nov 9

ssastry added a comment to T179259: Visual Editor removing whitespace from infoboxes by default.

Another middle ground might be to always respect the original argument order (ignoring templatedata), while still honoring the per-argument formatting specs. This would alleviate some of the issues resulting from our current line-oriented diff mechanisms, although we'd probably still have whitespace-related complaints.

If people are intentionally using parameters in a specific order for whatever reason, then it would be good if Parsoid could respect that.

Thu, Nov 9, 3:16 PM · Parsoid, VisualEditor

Wed, Nov 8

ssastry added a comment to T180010: Parsoid creates broken wikitext for link inside square brackets.

This is a pure Parsoid bug. The HTML>WT>HTML round trip is:

[<a rel="mw:WikiLink" href="./Foo">Foo</a>] -> [[[foo]]] -> [[[foo]]]

Wed, Nov 8, 3:21 PM · Patch-For-Review, Parsoid

Tue, Nov 7

ssastry added a comment to T103675: Android app fails to render formulas.

This is also peculiar because, as far as I know, Parsoid HTML always displays math formulas as Mathoid-rendered SVGs. Documentation is pretty thin in this area, though, so I'm not sure if this is always so, or depends on the user's logged-in state and (if logged in) math display preference. (+@bearND, @ssastry: any ideas about what could be going on? )

Tue, Nov 7, 8:40 PM · Patch-For-Review, Android-app-Bugs, WorkType-Maintenance, Wikipedia-Android-App-Backlog
ssastry added a comment to T178879: Automatically (or manually, server-side) fix user signatures containing Lint errors.

The "tidy font bug" category was added after @ssastry's comment, I think. Previously they only showed up in the low-priority category for deprecated tags.

Nope! 😃 T176363#3658974

ssastry closed this task as Resolved. Oct 4 2017, 17:13

Tue, Nov 7, 5:13 AM · MediaWiki-extensions-Linter
ssastry added a comment to T124398: Newline is removed after opening <ref> even when it changes semantics.

T137751 is related. Anyway, it looks like we need to look at all tag extensions that process wikitext and see if we can adopt semantics that are not dependent on parent start-of-line context.

Tue, Nov 7, 12:37 AM · Parsoid
ssastry added a comment to T137751: Parsoid's translation extension support needs to handle SOL context better.

Based on testing with <translate> and <ref>, it looks like individual extensions can dictate the parsing start-of-line context (translate borrows from parent context, ref is always non-sol.

Tue, Nov 7, 12:32 AM · Parsoid
ssastry lowered the priority of T127329: Using Parsoid as a wikitext bridge for importing content into wikitext format from Normal to Low.
Tue, Nov 7, 12:09 AM · Possible-Tech-Projects, Parsoid
ssastry removed a project from T131444: Edited transclusion with multiple paragraphs/block elements in a parameter serialised onto one line, breaking semantics when TemplateData missing: Parsoid.

Not sure if there is anything to be done here for Parsoid or VE, but will let @Deskana take a look. Removing the Parsoid tag.

Tue, Nov 7, 12:08 AM · VisualEditor

Mon, Nov 6

ssastry added a comment to T119883: Investigate inefficiencies in DOM construction and passes for large wikitext pages.

Enough work has been done on this generic ticket. There are other specific tickets tackling other aspects of DOM performance problems.

Mon, Nov 6, 11:58 PM · Performance, Parsoid-DOM, Parsoid
ssastry closed T119883: Investigate inefficiencies in DOM construction and passes for large wikitext pages as Resolved.
Mon, Nov 6, 11:58 PM · Performance, Parsoid-DOM, Parsoid
ssastry moved T114072: <section> tags for MediaWiki sections from Backlog to In Progress on the Parsoid board.
Mon, Nov 6, 11:57 PM · Reading Epics (Platform JS CSS and HTML consolidation), Services (blocked), RfC, Parsing-Team, Accessibility, TechCom-RfC, RESTBase, Patch-For-Review, Parsoid, MediaWiki-Parser, MediaWiki-Interface
ssastry added a comment to T152806: Wikitext representing an empty <tr> produces technically-invalid output.

I was about to work on this, but the HTML5 spec contradicts this statement from the description:

Mon, Nov 6, 11:53 PM · MediaWiki-extensions-Linter, Parsoid, Parsing-Team
ssastry updated the task description for T137751: Parsoid's translation extension support needs to handle SOL context better.
Mon, Nov 6, 11:42 PM · Parsoid
ssastry renamed T137751: Parsoid's translation extension support needs to handle SOL context better from SOL # in heading gets incorrectly parsed as list to Parsoid's translation extension support needs to handle SOL context better.
Mon, Nov 6, 11:32 PM · Parsoid
ssastry added a comment to T137751: Parsoid's translation extension support needs to handle SOL context better.

Also check PHP parser output and Parsoid output on "<h2>\n*a\n*b\n</h2>". That is another edge case where the output differs.

Mon, Nov 6, 11:23 PM · Parsoid
ssastry added a comment to T143378: Heartbeat timeout on User:Pdebee.

Looks like parse of the serialize wt does lock up the parse.

[subbu@earth parsoid] node bin/roundtrip-test.js --prefix enwiki "User:Pdebee"
{"name":"parsoid","hostname":"earth","pid":32620,"level":30,"levelPath":"info/service-runner","msg":"master(32620) initializing 1 workers","time":"2017-11-06T19:29:53.600Z","v":0}
{"name":"/home/subbu/work/wmf/parsoid/lib/index.js","hostname":"earth","pid":32630,"level":30,"levelPath":"info","msg":"loading ...","time":"2017-11-06T19:29:54.544Z","v":0}
{"name":"/home/subbu/work/wmf/parsoid/lib/index.js","hostname":"earth","pid":32630,"level":30,"levelPath":"info","msg":"ready on localhost:39725","time":"2017-11-06T19:29:54.574Z","v":0}
{"name":"parsoid","hostname":"earth","pid":32620,"level":40,"levelPath":"warn/service-runner","msg":"Startup finished","time":"2017-11-06T19:29:54.575Z","v":0}
{"name":"/home/subbu/work/wmf/parsoid/lib/index.js","hostname":"earth","pid":32630,"level":30,"logType":"info","wiki":"enwiki","title":"User:Pdebee","oldId":808980326,"reqId":null,"userAgent":null,"msg":"redirecting to revision 808980326 for wikitext","longMsg":"redirecting to revision\n808980326\nfor\nwikitext","levelPath":"info","time":"2017-11-06T19:29:56.133Z","v":0}
{"name":"/home/subbu/work/wmf/parsoid/lib/index.js","hostname":"earth","pid":32630,"level":30,"logType":"info","wiki":"enwiki","title":"User:Pdebee","oldId":null,"reqId":null,"userAgent":null,"msg":"started parsing","longMsg":"started parsing","levelPath":"info","time":"2017-11-06T19:29:56.661Z","v":0}
{"name":"/home/subbu/work/wmf/parsoid/lib/index.js","hostname":"earth","pid":32630,"level":30,"logType":"info","wiki":"enwiki","title":"User:Pdebee","oldId":null,"reqId":null,"userAgent":null,"msg":"completed parsing in 37381 ms","longMsg":"completed parsing in 37381 ms","levelPath":"info","time":"2017-11-06T19:30:34.035Z","v":0}
{"name":"/home/subbu/work/wmf/parsoid/lib/index.js","hostname":"earth","pid":32630,"level":30,"logType":"info","wiki":"enwiki","title":"User:Pdebee","oldId":null,"reqId":null,"userAgent":null,"msg":"started serializing","longMsg":"started serializing","levelPath":"info","time":"2017-11-06T19:30:34.142Z","v":0}
{"name":"/home/subbu/work/wmf/parsoid/lib/index.js","hostname":"earth","pid":32630,"level":30,"logType":"info","wiki":"enwiki","title":"User:Pdebee","oldId":null,"reqId":null,"userAgent":null,"msg":"completed serializing in 1249 ms","longMsg":"completed serializing in 1249 ms","levelPath":"info","time":"2017-11-06T19:30:35.375Z","v":0}
{"name":"/home/subbu/work/wmf/parsoid/lib/index.js","hostname":"earth","pid":32630,"level":30,"logType":"info","wiki":"enwiki","title":"User:Pdebee","oldId":null,"reqId":null,"userAgent":null,"msg":"started parsing","longMsg":"started parsing","levelPath":"info","time":"2017-11-06T19:30:35.883Z","v":0}
{"name":"parsoid","hostname":"earth","pid":32620,"level":50,"message":"worker 32630 stopped sending heartbeats, killing.","status":["enwiki/User:Pdebee"],"levelPath":"error/service-runner/master","msg":"worker 32630 stopped sending heartbeats, killing.","time":"2017-11-06T19:32:54.584Z","v":0}
Mon, Nov 6, 7:35 PM · Parsoid
ssastry added a comment to T143378: Heartbeat timeout on User:Pdebee.

This page parses just fine now. But, the output is not all great in terms of RT diffs. But, looks like most of the time spend parsing this page is in the PEG.

Mon, Nov 6, 7:30 PM · Parsoid
ssastry moved T148258: html2wt: Further relax link handler requirements around expecations of input HTML for a link from Backlog to Links on the Parsoid board.
Mon, Nov 6, 7:19 PM · Parsoid-Serializer, Parsoid
ssastry renamed T148258: html2wt: Further relax link handler requirements around expecations of input HTML for a link from Buggy html2wt serialization of old HTML to html2wt: Further relax link handler requirements around expecations of input HTML for a link.
Mon, Nov 6, 7:19 PM · Parsoid-Serializer, Parsoid
ssastry added a comment to T148258: html2wt: Further relax link handler requirements around expecations of input HTML for a link.

@Catrope, in case this lets you "hack around the problem" while we fix this issue. We have progressively relaxed constraints on how a link should look like, but looks like some things still trip it up. Looks like adding https: should do the trick in your case.

Mon, Nov 6, 7:17 PM · Parsoid-Serializer, Parsoid
ssastry moved T152540: Migrate to HTML5 section ids from Backlog to In Progress on the Parsoid board.
Mon, Nov 6, 6:51 PM · Readers-Web-Backlog (Tracking), User-notice, Mobile-Content-Service, Reading-Infrastructure-Team-Backlog, TechCom-RfC, Community-Wishlist-Survey-2016, Community-Tech, ContentTranslation, MediaWiki-Parser, Parsoid, Parsing-Team
ssastry edited projects for T155978: Table bug, rows getting merged, added: Parsoid-Serializer; removed VisualEditor-Tables, VisualEditor.
Mon, Nov 6, 6:47 PM · Parsoid-Serializer, Parsoid
ssastry removed a project from T156507: Issues with template parameters: Parsoid.

Not sure if Parsoid is involved here. But, please re-add Parsoid tag if a Parsoid fix is needed.

Mon, Nov 6, 6:44 PM · WMSE-Bug-Reporting-and-Translation-2017, VisualEditor-MediaWiki-2017WikitextEditor, VisualEditor
ssastry removed a project from T171616: Font size changes, and then the editor crashes: Parsoid.

I am not sure this is Parsoid related. So, removing the Parsoid tag. Please re-add if there is more info indicating that this requires a Parsoid fix.

Mon, Nov 6, 6:37 PM · VisualEditor-MediaWiki-2017WikitextEditor, VisualEditor
ssastry added a project to T178253: Figure handler rejects nested tables in figure captions: Parsoid-Tokenizer.
Mon, Nov 6, 6:35 PM · Patch-For-Review, Parsoid-Tokenizer, Parsoid
ssastry renamed T179847: Recently added DIV wrapper around references needs DSR info to satisfy rountrip-test.js from Bad dsr for DIV to Recently added DIV wrapper around references needs DSR info to satisfy rountrip-test.js.
Mon, Nov 6, 6:23 PM · Parsoid
ssastry moved T171381: Fix missing-end-tag linter issue generation from Linter to In Progress on the Parsoid board.
Mon, Nov 6, 6:10 PM · Patch-For-Review, MediaWiki-extensions-Linter, Parsoid
ssastry claimed T171381: Fix missing-end-tag linter issue generation.
Mon, Nov 6, 6:07 PM · Patch-For-Review, MediaWiki-extensions-Linter, Parsoid
ssastry closed T179847: Recently added DIV wrapper around references needs DSR info to satisfy rountrip-test.js as Declined.

This is related to roundtrip-test.js with the double diff business and not part of regular dsr computation which is at some level can only get so precise. At one point, i would go through these and tweak roundtrip-test.js. If we want to do another pass at them, the parsoid logs on ruthenium will have a whole bunch of them to look at. Not worth a ticket for individual titles.

Mon, Nov 6, 6:01 PM · Parsoid
ssastry closed T178217: DSR values for linter issues in extension content with wikitext content model are incorrect as Resolved.
Mon, Nov 6, 5:48 PM · MediaWiki-extensions-Linter, Parsoid
ssastry moved T176596: Re-evaluate Linter update mechanism from Backlog to Non-Parsoid Tasks on the Parsoid board.
Mon, Nov 6, 5:46 PM · Services (watching), Parsoid, MediaWiki-extensions-Linter
ssastry moved T178778: Parsoid, VisualEditor not working with SSL / HTTPS from Backlog to Non-Parsoid Tasks on the Parsoid board.
Mon, Nov 6, 5:39 PM · Operations, HTTPS, Parsoid, VisualEditor
ssastry moved T178037: Parsoid uses non-canonical URL encoding from Backlog to Next Up on the Parsoid board.
Mon, Nov 6, 5:01 PM · Parsoid, Services (later), RESTBase
ssastry renamed T178037: Parsoid uses non-canonical URL encoding from RESTBase uses non-canonical URL encoding to Parsoid uses non-canonical URL encoding.
Mon, Nov 6, 5:01 PM · Parsoid, Services (later), RESTBase
ssastry raised the priority of T178037: Parsoid uses non-canonical URL encoding from Normal to High.
Mon, Nov 6, 5:00 PM · Parsoid, Services (later), RESTBase
ssastry added a comment to T176596: Re-evaluate Linter update mechanism.

@Legoktm anything to do here? I inclined to close this as declined.

Mon, Nov 6, 4:51 PM · Services (watching), Parsoid, MediaWiki-extensions-Linter
ssastry triaged T178927: Another reparse table attribute scenario as Normal priority.
Mon, Nov 6, 4:50 PM · Parsoid
ssastry closed T178593: Potentially O(n^2) or worse bug in a DOM pass on a pathological page as Declined.

Closing this generic task since specific pieces are being addressed elsewhere (T178604, https://github.com/fgnass/domino/pull/104 )

Mon, Nov 6, 4:38 PM · Parsoid-DOM, Performance, Parsoid
ssastry closed T178604: Investigate perf problems with nextSibling (with / without DOM mutations) in domino as Resolved.

This has now been fixed in domino ( https://github.com/fgnass/domino/commit/056cb4ea5e50e68b65974e5f2f1e9e22ccd8a7ef ) and deployed to production on Oct 26.

Mon, Nov 6, 4:35 PM · Parsoid-DOM, Parsoid
ssastry moved T178705: Prepare MCS for Parsoid section tags from Backlog to Non-Parsoid Tasks on the Parsoid board.
Mon, Nov 6, 4:31 PM · Reading-Infrastructure-Team-Backlog (Kanban), Mobile-Content-Service, Parsoid
ssastry moved T179259: Visual Editor removing whitespace from infoboxes by default from Backlog to Non-Parsoid Tasks on the Parsoid board.
Mon, Nov 6, 4:31 PM · Parsoid, VisualEditor
ssastry triaged T178914: parse.js should have standard arguments as Low priority.
Mon, Nov 6, 4:30 PM · Parsoid
ssastry reassigned T178970: A left-floating template is broken while editing in an RTL wiki from ssastry to Esanders.
Mon, Nov 6, 4:30 PM · Parsoid
ssastry closed T178970: A left-floating template is broken while editing in an RTL wiki as Resolved.
Mon, Nov 6, 4:29 PM · Parsoid
ssastry moved T179605: LintError bogus-image-options triggers on "Thumbtime" from Backlog to Linter on the Parsoid board.
Mon, Nov 6, 4:23 PM · Parsoid, MediaWiki-extensions-Linter
ssastry moved T179757: Linter/tidy-font-bug triggers on empty font tags from Backlog to In Progress on the Parsoid board.
Mon, Nov 6, 4:23 PM · Parsoid, Patch-For-Review, MediaWiki-extensions-Linter

Sun, Nov 5

ssastry added a comment to T179259: Visual Editor removing whitespace from infoboxes by default.

Looks like an editor removed the offending block-formatting directive from this particular infobox -- so going forward, edits to transclusions of this template should leave things untouched.

Sun, Nov 5, 5:44 PM · Parsoid, VisualEditor

Sat, Nov 4

ssastry claimed T179757: Linter/tidy-font-bug triggers on empty font tags.
Sat, Nov 4, 3:47 PM · Parsoid, Patch-For-Review, MediaWiki-extensions-Linter

Fri, Nov 3

ssastry closed T75412: OCG Attribution request times out reguarly as Declined.

Since OCG has been turned off.

Fri, Nov 3, 10:06 PM · OCG-General, Performance
ssastry added a comment to T179715: Enable section collapsing for readers.

Note that in some scenarios, without additional work, this can break template wrapping semantics. See Example 2 in https://www.mediawiki.org/wiki/Parsing/Notes/Section_Wrapping#Examples and the commit message in my section wrapper patch ( https://gerrit.wikimedia.org/r/#/c/364933/32 ) for how even adding just the <section> tag can do this and how the section wrapping code handles this.

Fri, Nov 3, 6:27 PM · Reading-Infrastructure-Team-Backlog, Page Content Service
ssastry added a comment to T179605: LintError bogus-image-options triggers on "Thumbtime".

It seems the img_ options are "case-sensitive" but the timedmedia_ are not.

Fri, Nov 3, 4:45 PM · Parsoid, MediaWiki-extensions-Linter

Wed, Nov 1

ssastry committed rIVDIdb5b6b2adf91: Follow redirects to eliminate noise in tests (authored by ssastry).
Follow redirects to eliminate noise in tests
Wed, Nov 1, 10:29 PM
ssastry committed rIVDI5cac19277721: Merge branch 'master' into ruthenium (authored by ssastry).
Merge branch 'master' into ruthenium
Wed, Nov 1, 10:29 PM
ssastry added a comment to T176959: Display proper section collapsing within article namespace.

Thanks everybody. We'll not add the <div> wrapper then. I'll update my patch accordingly.

Wed, Nov 1, 2:24 PM · Marvin, Epic

Tue, Oct 31

ssastry added a comment to T176959: Display proper section collapsing within article namespace.

Anyway, why not get rid of <section> and wrap the section body in a div? That way the level of client-specific bloat stays the same, and it doesn't seem as if section tags would be useful to anyone; and they don't have any real semantic meaning anyway, given that the MediaWiki section structure is not DOM-based.

Tue, Oct 31, 10:58 PM · Marvin, Epic
ssastry added a comment to T176959: Display proper section collapsing within article namespace.
Tue, Oct 31, 8:39 PM · Marvin, Epic
ssastry added a comment to T176959: Display proper section collapsing within article namespace.
Tue, Oct 31, 8:37 PM · Marvin, Epic
ssastry added a comment to T176959: Display proper section collapsing within article namespace.

At the moment the amount of code to do the transformation either way is going to be similar, but as this is the first time we're upstreaming mobile-specific DOM transformations we need to think carefully about where we want to set our boundaries (cf cscott's escalation of markup list).

Tue, Oct 31, 8:19 PM · Marvin, Epic

Mon, Oct 30

ssastry added a comment to T176959: Display proper section collapsing within article namespace.

Backing up, let's describe a general range of markups:
... snip ...
One answer may be to go back and restructure our parsoid toolchain a little bit more (eventually) to separate out something closer to #1 as "storage format DOM", and then explicitly structure the path to #4 as a set of transformations, with something close to #4 as a "read format DOM" -- and then we already have separated out data-mw attributes to form an "edit format DOM", and data-parsoid attributes to form a further "round-trippable DOM".

Or else we'll keep muddling through with a compromise DOM format which tries to be directly viewable and not too inefficient for storage or data extraction.

Mon, Oct 30, 9:05 PM · Marvin, Epic
ssastry committed rIVDI87daa3733b5f: Update Tidy vs Remex adaptor to eliminate noise from race conditions (authored by ssastry).
Update Tidy vs Remex adaptor to eliminate noise from race conditions
Mon, Oct 30, 8:02 PM
ssastry added a comment to T176959: Display proper section collapsing within article namespace.

That is something for @Jdlrobson or @phuedx to answer based on product needs.

Well it affects all users of Parsoid. Having the body wrapper in the HTML will make life easier for clients that want to collapse sections, but puts a burden on other clients (bandwidth and extra code).

Mon, Oct 30, 3:21 PM · Marvin, Epic