Last modified: 2013-11-12 20:37:58 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T51925, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 49925 - VisualEditor: Floated templates rendering oddly and overlapping text
VisualEditor: Floated templates rendering oddly and overlapping text
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
ContentEditable (Other open bugs)
unspecified
All All
: High normal
: VE-deploy-2013-07-04
Assigned To: Rob Moen
:
: 50381 50416 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-06-21 04:38 UTC by Oliver Keyes
Modified: 2013-11-12 20:37 UTC (History)
9 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
not displaying properly (57.87 KB, image/jpeg)
2013-06-23 11:10 UTC, Elitre
Details
not displaying properly (62.84 KB, image/jpeg)
2013-06-23 11:11 UTC, Elitre
Details
what the template should look like (read mode) (28.22 KB, image/png)
2013-06-23 16:39 UTC, James Forrester
Details
what the template does look like (once in VE) (29.53 KB, image/png)
2013-06-23 16:39 UTC, James Forrester
Details
what the template does look like when selected (30.17 KB, image/png)
2013-06-23 16:40 UTC, James Forrester
Details
Screenshot (54.00 KB, image/png)
2013-06-28 10:23 UTC, Oliver Keyes
Details
Crud (232.07 KB, image/png)
2013-06-30 17:22 UTC, Oliver Keyes
Details

Description Oliver Keyes 2013-06-21 04:38:25 UTC
See screenshot, which is of https://en.wikipedia.org/wiki/Craig_Short - Firefox 21.0, Windows 7.
Comment 1 Elitre 2013-06-23 11:10:44 UTC
Created attachment 12608 [details]
not displaying properly
Comment 2 Elitre 2013-06-23 11:11:12 UTC
Created attachment 12609 [details]
not displaying properly
Comment 3 Elitre 2013-06-23 11:14:28 UTC
I don't think I can see your attachment, Oliver, but I think the bug may have to do with to these sshots, which both refer to adding a link: in the first one, the recycle bin can barely be seen as it gets hidden, the second one shows the link icon not being displayed as a square. Looks like this can happen both with Chrome and Firefox.
Comment 4 Elitre 2013-06-23 11:18:02 UTC
I thinks it depends on the skin: Monobook might be causing this.
Comment 5 James Forrester 2013-06-23 16:39:26 UTC
Created attachment 12612 [details]
what the template should look like (read mode)
Comment 6 James Forrester 2013-06-23 16:39:57 UTC
Created attachment 12613 [details]
what the template does look like (once in VE)
Comment 7 James Forrester 2013-06-23 16:40:19 UTC
Created attachment 12614 [details]
what the template does look like when selected
Comment 8 James Forrester 2013-06-23 16:41:01 UTC
(In reply to comment #4)
> I thinks it depends on the skin: Monobook might be causing this.

That's a different bug - created bug 50057.

This is about CE's rendering of floated templates - I've attached some example screenshots to demonstrate.
Comment 9 Oliver Keyes 2013-06-28 10:23:26 UTC
Created attachment 12673 [details]
Screenshot

This is likely a blocker for any beta rollout on Monday. I've attached a screenshot to show how bad it can get, and why :/.
Comment 10 Inez Korczyński 2013-06-28 14:08:43 UTC
For some reason wrapper in which we put transclusions/templates gets HTML attributes of the transclusion/template carried over - including CSS classes - that should not happen, and wasn't happening in the past.

I will investigate it, however adding Roan to this ticket, it seems like it is his area.
Comment 11 James Forrester 2013-06-28 17:21:18 UTC
+another example: https://en.wikipedia.org/wiki/Aranath?veaction=edit
Comment 12 James Forrester 2013-06-28 17:21:23 UTC
*** Bug 50381 has been marked as a duplicate of this bug. ***
Comment 13 Gerrit Notification Bot 2013-06-28 20:21:56 UTC
Change 71109 had a related patch set uploaded by Jforrester:
Set renderHtmlAttributes to false, for both block and inline transclusions

https://gerrit.wikimedia.org/r/71109
Comment 14 Gerrit Notification Bot 2013-06-28 23:46:02 UTC
Change 71109 merged by jenkins-bot:
Stop floated transclusions floating out of themselves

https://gerrit.wikimedia.org/r/71109
Comment 15 James Forrester 2013-06-28 23:46:34 UTC
The above patch fixes this bug, we believe. Marking as such.
Comment 16 This, that and the other (TTO) 2013-06-29 10:13:06 UTC
*** Bug 50416 has been marked as a duplicate of this bug. ***
Comment 17 Oliver Keyes 2013-06-30 17:22:42 UTC
Created attachment 12695 [details]
Crud

It doesn't :(. See new screenshot.
Comment 18 Oliver Keyes 2013-06-30 17:23:20 UTC
Unless the patch hasn't been deployed yet, I guess.
Comment 19 James Forrester 2013-06-30 17:35:28 UTC
(In reply to comment #18)
> Unless the patch hasn't been deployed yet, I guess.

It hasn't. Deployed on Monday alongside the config change. Not sure if the example in comment 17 will be entirely fixed by it though - please check after release?
Comment 20 Oliver Keyes 2013-06-30 17:37:27 UTC
Shall do.
Comment 21 WhatamIdoing 2013-11-12 20:37:58 UTC
Follow up:  https://en.wikipedia.org/wiki/Leo_Strauss?veaction=edit is still screwed up.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links