[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #65 from Erwin Dokter er...@darcoury.nl --- This is also caused by (the absence of) Tidy. Unless the HTML parser (and Parsoid) strip leading and trailing spaces, they will be visible. You may want to ping the parsoid team to see if they want to fix this, but at least you have found a workaround. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 Mark A. Hershberger m...@everybody.org changed: What|Removed |Added CC||m...@everybody.org Target Milestone|1.22.0 release |--- --- Comment #62 from Mark A. Hershberger m...@everybody.org --- Removing target milestone that was in the past. If you want this in a specific release, have a good reason AND you are willing to find resources to fix this bug, feel free to change it to something appropriate. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 Erwin Dokter er...@darcoury.nl changed: What|Removed |Added Status|PATCH_TO_REVIEW |RESOLVED Resolution|--- |FIXED --- Comment #63 from Erwin Dokter er...@darcoury.nl --- The issue has been resolved with https://gerrit.wikimedia.org/r/134380. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #64 from Helder he7...@gmail.com --- Thanks! However, after I updated the code in https://pt.wikipedia.org/w/index.php?diff=39195928oldid=37160137 I still noticed differences between [[pt:MediaWiki:Recentchangestext]] and [[pt:Special:RecentChanges]]. Specifically, there was a space between the first item of each sublist and the ( just before it. I had to make this change to fix the issue: https://pt.wikipedia.org/w/index.php?diff=39195952 I don't know if that is is relevant enough to open a new bug. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #61 from Gerrit Notification Bot gerritad...@wikimedia.org --- Change 94443 abandoned by Cscott: Revert Have list items occupy their own line. Reason: Abandoned in favor of Ib7aa9449bbd994cb23b83b3f23cff944b1cddadf https://gerrit.wikimedia.org/r/94443 -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #60 from Gerrit Notification Bot gerritad...@wikimedia.org --- Change 134380 merged by jenkins-bot: Update list item newline handling to follow Parsoid's model https://gerrit.wikimedia.org/r/134380 -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 Gerrit Notification Bot gerritad...@wikimedia.org changed: What|Removed |Added Status|RESOLVED|PATCH_TO_REVIEW Resolution|FIXED |--- -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #59 from Gerrit Notification Bot gerritad...@wikimedia.org --- Change 134380 had a related patch set uploaded by GWicke: Update list item newline handling to follow Parsoid's model https://gerrit.wikimedia.org/r/134380 -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #48 from Erwin Dokter er...@darcoury.nl --- Regarding hlist, the wrapping problem has been resolved by means of disabling nowrap alltogether. But it still has the potential of malfunctioning if nowrap is applied locally (ie. in a template). With that in mind, I recommend that *this* bug be closed and discussion regarding parser/parsoid to continue in bug 56809. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #49 from Gabriel Wicke gwi...@wikimedia.org --- Sorry for bringing this up this late, but can you describe the use case for rendering nested lists as a flat structure? Can those lists be a flat DOM structure instead? -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #50 from Erwin Dokter er...@darcoury.nl --- No, hlist must work just like regular HTML lists because they must be sematically the same. Anything breaking that behaviour negates the entire purpose of hlist. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #51 from Gabriel Wicke gwi...@wikimedia.org --- I don't think that this answers my question though. Why can't those lists just be flat if they are intended to render as a flat structure? -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #52 from Erwin Dokter er...@darcoury.nl --- Accessibility. They only render flat visually, but its structure must be maintained to ensure they are accessable as regular HTML lists for screenreaders. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #53 from Gabriel Wicke gwi...@wikimedia.org --- That points out why they should be a list, which we don't disagree on. Can you describe why they have to be *nested* lists? -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #54 from Erwin Dokter er...@darcoury.nl --- That is entirely up to the user constructing the list. hlist only allows you to have nested lists, because HTML/wikimarkup allows you to have nested lists. I couldn't 'unnest' the lists even if I wanted to. I don't see how this has become an issue. It was the nowrap issue that caused hlists to misbehave. The nowrap has been remove. What is the problem? -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #55 from Gabriel Wicke gwi...@wikimedia.org --- I'm still trying to figure out why you have the requirement to render *nested* lists as a flat structure. So far it seems that the only reason to support this is that lists *can* generally be nested, not that they have to be for some reason. So right now it looks like we are actually trying to solve a problem that doesn't necessarily need to be solved. The current solution (newline insertion) causes issues with round-tripping in Parsoid and restricts the way we can render things as described in comment 18. We could perhaps find a way to work around the issues in Parsoid, but this involves considerable effort and comes at a cost of an irregularity in the way we convert wikitext to DOM (which makes it harder to understand for our users). So please let us know if there are real use cases for supporting nested lists in hlist, and why those are worth the costs. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #56 from Phillip Patriakeas dragonlordofxant...@gmail.com --- The use cases are exactly the same as the use cases of nesting lists when just using HTML lists: anywhere you would potentially want to nest HTML lists is where you would also potentially want to nest hlists. Consider for example the following list: -Cats --Tabby --Calico -Dogs --Chihuahua --Great Dane -Gerbils -Kangaroos This is a perfectly semantic structure, and flattening the lists into a single list destroys the semantics. If you convert it to an hlist (by adding an hlist div or the like around it), it displays instead as: Cats (Tabby - Calico) - Dogs (Chihuahua - Great Dane) - Gerbils - Kangaroos Again, this is perfectly semantic, and the nesting is still visually communicated, and again, flattening to just one list destroys the semantics. And I can tell you from personal experience, both on and off of Wikipedia/Wikimedia wikis, that it takes almost no imagination at all to find a myriad of applications for such nestings in navboxes. But as Erwin said, I don't see how lists being nested is at all relevant to this bug; the functionality is completely separate from the wrapping functionality, which, as he has stated multiple times, has already been removed from the hlist styles. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #57 from Gabriel Wicke gwi...@wikimedia.org --- (In reply to Phillip Patriakeas from comment #56) The use cases are exactly the same as the use cases of nesting lists when just using HTML lists: anywhere you would potentially want to nest HTML lists is where you would also potentially want to nest hlists. Consider for example the following list: -Cats --Tabby --Calico -Dogs --Chihuahua --Great Dane -Gerbils -Kangaroos This is a perfectly semantic structure, and flattening the lists into a single list destroys the semantics. If you convert it to an hlist (by adding an hlist div or the like around it), it displays instead as: Cats (Tabby - Calico) - Dogs (Chihuahua - Great Dane) - Gerbils - Kangaroos Again, this is perfectly semantic, and the nesting is still visually communicated, and again, flattening to just one list destroys the semantics. I see, thanks for the clear explanation! And I can tell you from personal experience, both on and off of Wikipedia/Wikimedia wikis, that it takes almost no imagination at all to find a myriad of applications for such nestings in navboxes. But as Erwin said, I don't see how lists being nested is at all relevant to this bug; the functionality is completely separate from the wrapping functionality, which, as he has stated multiple times, has already been removed from the hlist styles. Nowrap requirements made it hard to find a CSS-only solution. With that gone it might now actually be possible to do away with the newlines. I copied over the .hlist styles from enwiki common.css to http://jsfiddle.net/dT3Qg/5/ and added a single line (display: inline) to the first rule to prevent a wrap before the nested list. For comparison, http://jsfiddle.net/dT3Qg/6/ is the unmodified CSS. As far as I can tell, both variants work with or without newlines in the DOM. Erwin mentioned that nowrap could still be applied manually. Is this an important case we need to handle? Would that normally apply to a single list item? -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 Erwin Dokter er...@darcoury.nl changed: What|Removed |Added Status|REOPENED|RESOLVED Resolution|--- |FIXED --- Comment #58 from Erwin Dokter er...@darcoury.nl --- Someone might still apply nowrap in separate list items, which *could* misbehave if the newlines were removed. That is why I do not like to see them removed. But that discussion should be continued in bug 56809. Your jsfiddle examples are not used entirely correctly, as the hlist class always applies to the enclosing div, not directly to the ul tag. That is why you had to add the extra line. With the wrapping issue resolved, I'm closing this bug. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 Andre Klapper aklap...@wikimedia.org changed: What|Removed |Added Target Milestone|1.22.0 release |1.22.x release --- Comment #46 from Andre Klapper aklap...@wikimedia.org --- [Bumping TM as MediaWiki 1.22.0 tarball was released today.] -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 Bartosz Dziewoński matma@gmail.com changed: What|Removed |Added Target Milestone|1.22.x release |1.22.0 release --- Comment #47 from Bartosz Dziewoński matma@gmail.com --- The fix for this was merged into 1.22 and is still there. We're now discussing possibly reverting it and coming up with a better one. Let's keep the milestone, in case this gets RESOLVED FIXED again with no reverts. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #44 from Erwin Dokter er...@darcoury.nl --- Having the sublist use display:inline produces the same problem that caused me te submit this patch in the first place; the entire sublist won't wrap if the list items are glued together (no linebreak or TIDY). -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #45 from Brad Jorsch bjor...@wikimedia.org --- gwicke's example linked in Comment 43 produces an odd display here: Item Item Item Item Item Item Item Item Item Item Item Item Item Item Item Item Item That last Item shouldn't be on a new line. And if you add another liitem/li before the one containing the sublist, it gets even worse: Item Item Item Item Item Item Item Item Item Item Item Item Item Item Item Item Item Item -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #42 from Gabriel Wicke gwi...@wikimedia.org --- So you would like it *not* to wrap between a list and its first sub-list item, but then wrap between additional sub-list items? -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #43 from Gabriel Wicke gwi...@wikimedia.org --- I think I figured out what you meant by creating a sublist with a lot more list items. The break happened first between foo and bar rather than the nested list items, at least in some screen widths. I have tweaked the css in https://test.wikipedia.org/wiki/User:GWicke/sandbox to make that work too. The change is to set display: inline on the nested list. Also removed the margin on it for uniform display. Anything else that you see amiss? -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #40 from Gabriel Wicke gwi...@wikimedia.org --- I have created a simple nested list test case at https://test.wikipedia.org/wiki/User:GWicke/sandbox The wrapping seems to work as expected, only the margins / padding still needs some tweaking. And the inline rules need to be converted into selectors. Could you point out which issues you see, and maybe adjust / add a test case to capture those? -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #41 from Erwin Dokter er...@darcoury.nl --- I'm sorry, but it does not wrap as expected. It wraps between foo and bar (beginning of sublist) where it should not. Inline-block somehow overrides white-space:normal, making it act as nowrap. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #39 from Erwin Dokter er...@darcoury.nl --- I have seen it. I have tested it in my sandbox on test.wikipeida.org. display+inline-block causes malformed lists; nested lists become unwilling to wrap, regargless of white-space setting. I have tested everything, and this remains the only working solution. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #26 from Erwin Dokter er...@darcoury.nl --- white-space: inline-block is nonsense; it only causes white-space to revert to 'normal', which is NOT desired. Also, the posted Tidy bug hardly seems relevant. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #27 from Bartosz Dziewoński matma@gmail.com --- The Tidy bug is what caused hlists to work in the first place, according to what you're saying :) -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 Bartosz Dziewoński matma@gmail.com changed: What|Removed |Added Status|PATCH_TO_REVIEW |RESOLVED See Also||https://bugzilla.wikimedia. ||org/show_bug.cgi?id=56809 Resolution|--- |FIXED --- Comment #28 from Bartosz Dziewoński matma@gmail.com --- I am going to mark this resolved to avoid fragmenting discussion any further. The talk about possibly reverting this change and CSS solutions should go on bug 56809, pretty please. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 C. Scott Ananian canan...@wikimedia.org changed: What|Removed |Added Status|RESOLVED|REOPENED Resolution|FIXED |--- --- Comment #29 from C. Scott Ananian canan...@wikimedia.org --- Matma: I discussed this with gwicke and others. We'd like to use this bug to continue to deal with the hlist issues, since that's where most of the discussion has already occurred (and in our opinion that bug has not been fixed in an acceptable manner yet). The other bug is just to track the current parsertest regression, if patches are needed to Parsoid, ParserFunctions, Scribunto, etc. I'm copying comment 10 from 56809 here -- Erwin says that: https://test.wikipedia.org/wiki/MediaWiki:Common.css contain current CSS for hlist. https://test.wikipedia.org/wiki/User:Edokter/sandbox contains the test cases. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #30 from C. Scott Ananian canan...@wikimedia.org --- And just recapping the objections to the current patch: 1) it changes longstanding parser behavior, possibility of unintended consequences is high 2) it breaks a number of extensions using the parsertests framework, including Parsoid and potentially others like ParserFunction, Scribunto, etc, by changing parser output in an unexpected way. 3) as gwicke points out in comment 18, removing the linebreak means that stylesheets which *want* single-line behavior can't obtain it. 4) It potentially causes dirty diff issues with Parsoid (see bug 56809 for details) Some possible alternative fixes: 1) I hope gwicke can demonstrate an improved CSS fix 2) perhaps Tidy can/should be run for Special pages as well as normal pages. 3) We suck it up and fix Parsoid, ParserFunction, Scribunto, etc to deal with the new output, evangelize editors etc to let them know the parser has changed, etc. To be clear, we haven't reverted the existing patch yet. My position is just that it was committed prematurely, in view of the side effects, and that we need to take a harder look at the other alternatives. I apologize for not catching this issue earlier, before it was committed -- I usually watch Parser-related patches, but I happened to be on vacation. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #31 from Helder mybugs.m...@gmail.com --- (In reply to comment #29) And the other (important) part from bug 56809 comment 10 is: Any suggested changes will break one or more testcases. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #32 from C. Scott Ananian canan...@wikimedia.org --- That's why we write testcases, Helder. Fixes aren't committed until/unless they pass them. I thought that was obvious. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #33 from C. Scott Ananian canan...@wikimedia.org --- In https://bugzilla.wikimedia.org/show_bug.cgi?id=56809#c14 Erwin writes: That is a scoping problem; Tidy should simply not touch anything that comes out of GeSHi (pre code). This is simply two external extentions clashing. This seems (to me) to indicate that this might actually be a Tidy integration bug: we need better control of how Tidy is run so that we can (a) run it on Special pages and other places where hlist isn't working, and (b) *not* run it on GeSHi and other extension-generated output. Moving tidy's invocation so that it happens before extensions are expanded might help solve (b). Thoughts? (esp from gwicke, since AIUI he hooked up Tidy originally.) -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #34 from Bartosz Dziewoński matma@gmail.com --- (I think thaat comment was about bug 260 and bug 38800.) -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #35 from Erwin Dokter er...@darcoury.nl --- (It was. Tidy/GeSHi conflict is not about hlist. I got a collision when posting that.) -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #36 from Erwin Dokter er...@darcoury.nl --- If extensions rely on long standing parser output, it means the parser can never be changed or improved. I once got admonished by a bot operator because I dared to change the output of a template, throwing the (scraping) bot into a fit. I can't help but feel this is kind of the same situation. Having the parser locked in because of extensions depending on its output is a Very Bad Thing. What happens if the parser is rewritten? It happened before. Is the Parsoid Team going to block that as well? Can Parsoind not be made line-break-agnostic in some way? Having Tidy run on spacial pages *would* solve all hlist problems on the Wikimedia cluster. But it does mean that hlist integration in core is out, because Tidy would be a pre-requisite for other installations. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #37 from Gabriel Wicke gwi...@wikimedia.org --- (In reply to comment #26) white-space: inline-block is nonsense; it only causes white-space to revert to 'normal', which is NOT desired. Also, the posted Tidy bug hardly seems relevant. Please actually look at the examples I posted. Example code in http://jsfiddle.net/dT3Qg/2/ linked in #18: ulli style=white-space: nowrap; display: inline-blockfoo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo foo /lili style=white-space: nowrap; display: inline-blockbar/li/ul -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #38 from Gabriel Wicke gwi...@wikimedia.org --- (And apologies for the typo in #17 that probably threw you off.) -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 C. Scott Ananian canan...@wikimedia.org changed: What|Removed |Added CC||canan...@wikimedia.org --- Comment #22 from C. Scott Ananian canan...@wikimedia.org --- Erwin: did you try setting the CSS 'whitespace' property, as described in comment 17? To be clear, this (merged) patch currently is causing Parsoid to create dirty diffs. We would very much like to revert it, and replace it with a CSS-only fix. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 Gabriel Wicke gwi...@wikimedia.org changed: What|Removed |Added Status|RESOLVED|REOPENED Resolution|FIXED |--- --- Comment #23 from Gabriel Wicke gwi...@wikimedia.org --- As discussed earlier, we should really use whitespace for this. Reopening. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 Gerrit Notification Bot gerritad...@wikimedia.org changed: What|Removed |Added Status|REOPENED|PATCH_TO_REVIEW -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #24 from Gerrit Notification Bot gerritad...@wikimedia.org --- Change 94443 had a related patch set uploaded by Cscott: Revert Have list items occupy their own line https://gerrit.wikimedia.org/r/94443 -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #25 from Gabriel Wicke gwi...@wikimedia.org --- Relevant tidy bug for avoiding newline insertion: https://sourceforge.net/p/tidy/bugs/945/ -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 Erwin Dokter er...@darcoury.nl changed: What|Removed |Added Status|REOPENED|RESOLVED Resolution|--- |FIXED --- Comment #21 from Erwin Dokter er...@darcoury.nl --- Gabriel, this patch only sanitizes native parser HTML output so that Tidy doesn't have to, in order to have certain CSS (such as .hlist) work as expected in those cases where Tidy is *not* available. I don't know where your issues originate from, but if they are from regular wiki pages, then this should not affect them. As for Parsoid, there is no CSS fix. Believe me, I've tried them all. This is the only solution left. List items are is essence block level elements and should be treated and generated as such. I'm not sure what Parsoid does exactly (VE?), but if it processes HTML output that went through Tidy, then this should also not affect Parsoid. If NEW issues arise, please open a new bug. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #15 from Gerrit Notification Bot gerritad...@wikimedia.org --- Change 90696 merged by jenkins-bot: Have list items occupy their own line https://gerrit.wikimedia.org/r/90696 -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 Brad Jorsch bjor...@wikimedia.org changed: What|Removed |Added Status|PATCH_TO_REVIEW |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #16 from Brad Jorsch bjor...@wikimedia.org --- Change merged. It should be deployed to WMF wikis with 1.23wmf1 (unless we decide to have 1.22wmf23 instead), see https://www.mediawiki.org/wiki/MediaWiki_1.23/Roadmap for the schedule. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 Gabriel Wicke gwi...@wikimedia.org changed: What|Removed |Added CC||gwi...@wikimedia.org --- Comment #17 from Gabriel Wicke gwi...@wikimedia.org --- Setting the list items to white-space: inline-block (not inline) works without the newline in the DOM. Support: * http://caniuse.com/#search=inline-block : All modern browsers. * IE 6/7: simulation via display: inline + hasLayout (eg. zoom:1) Omitting the newline from the DOM lets users choose whether to line break or not with CSS, so seems to be preferable. Inserting a newline sounds problematic for this reason, and might just be a tidy bug. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #18 from Gabriel Wicke gwi...@wikimedia.org --- Examples: // force single line, but space cannot be avoided as it is in the DOM http://jsfiddle.net/dT3Qg/ // Without a space in the DOM, both single-line and break-on-li behavior are possible. Break-on-li with inline-block: http://jsfiddle.net/dT3Qg/2/ -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 Gabriel Wicke gwi...@wikimedia.org changed: What|Removed |Added Status|RESOLVED|REOPENED Resolution|FIXED |--- --- Comment #19 from Gabriel Wicke gwi...@wikimedia.org --- Reopening this bug to discuss which solution makes sense in the longer term, both in tidy and Parsoid. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 ssas...@wikimedia.org changed: What|Removed |Added CC||ssas...@wikimedia.org --- Comment #20 from ssas...@wikimedia.org --- It would be great to find a CSS fix for this without relying on presence/absence of newlines if possible. This is especially so for Parsoid that is a bidirectional converter between wikitext and HTML. To minimize dirty diffs, Parsoid does a good deal of bookkeeping to accurately map wikitext segments to HTML segments that they generate. This newline behavior will add to the complexity of our book-keeping and we are really keen to avoid it, if possible. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #12 from Erwin Dokter er...@darcoury.nl --- Working on a patch, but my local repository seems to be broken; I get over a year old files when rebasing. I do hate GIT with a passion. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 Gerrit Notification Bot gerritad...@wikimedia.org changed: What|Removed |Added Status|REOPENED|PATCH_TO_REVIEW -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #13 from Gerrit Notification Bot gerritad...@wikimedia.org --- Change 90696 had a related patch set uploaded by Gerrit Patch Uploader: Have list items occupy their own line https://gerrit.wikimedia.org/r/90696 -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 Erwin Dokter er...@darcoury.nl changed: What|Removed |Added Target Milestone|--- |1.22.0 release --- Comment #14 from Erwin Dokter er...@darcoury.nl --- Patch now ready for review. Changed parserTest.txt as well to reflect changed output. Thanks to MatmaRex for uploading patches. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 Brad Jorsch bjor...@wikimedia.org changed: What|Removed |Added CC||bjor...@wikimedia.org --- Comment #11 from Brad Jorsch bjor...@wikimedia.org --- (In reply to comment #10) It's not clear to me why '/li\nli' would not work In PHP, escapes such as \n are only interpreted inside double-quoted strings. When you use single quotes, it outputs \n literally. So using double quotes, as in /li\n/li, should work. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 --- Comment #10 from Erwin Dokter er...@darcoury.nl --- OK, that means my patch can go in the bin. It's not clear to me why '/li\nli' would not work; it seems to be the standard way in PHP strings to include linebreaks. I wish someone would go in and terminate all closing HTML tags with a linebreak. This is such a trivial solution, but it goes beyond my PHP knowledge. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 kevi...@sympatico.ca changed: What|Removed |Added CC||kevi...@sympatico.ca --- Comment #9 from kevi...@sympatico.ca --- Thanks for this great solution Dokter! Just wanted to add a clarification for n00bs like myself that I had to change it from '/lili' to '/li'.\n.'li' for it to parse correctly. The original suggestion just gave me \n in my actual text. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 Erwin Dokter er...@darcoury.nl changed: What|Removed |Added Summary|Malformed hlist rendering |Malformed hlist rendering |due to Tidy not processing |due to missing linebreaks |Special: pages |between list items -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39617] Malformed hlist rendering due to missing linebreaks between list items
https://bugzilla.wikimedia.org/show_bug.cgi?id=39617 Erwin Dokter er...@darcoury.nl changed: What|Removed |Added Blocks||40062 -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l