Last modified: 2014-10-03 21:07:25 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 T52036, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 50036 - VisualEditor: Focus highlights for elements using CSS column-count are too tall in Chrome
VisualEditor: Focus highlights for elements using CSS column-count are too ta...
Status: ASSIGNED
Product: VisualEditor
Classification: Unclassified
ContentEditable (Other open bugs)
unspecified
All All
: Normal minor
: ---
Assigned To: Ed Sanders
: upstream
: 50267 71625 (view as bug list)
Depends on:
Blocks: 50269
  Show dependency treegraph
 
Reported: 2013-06-23 07:47 UTC by This, that and the other (TTO)
Modified: 2014-10-03 21:07 UTC (History)
9 users (show)

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


Attachments
Screenshot of typical two-column reflist highlighted in Chrome (387.64 KB, image/png)
2014-07-03 00:55 UTC, Krinkle
Details
Screenshot of case with more than 2 columns (257.69 KB, image/png)
2014-07-03 00:57 UTC, Krinkle
Details

Description This, that and the other (TTO) 2013-06-23 07:47:40 UTC
See [[User:This, that and the other/sandbox/VE4#columns-list]].
Comment 1 This, that and the other (TTO) 2013-06-24 01:26:38 UTC
In Firefox the text is overlapped; in Chrome it is worse - the template appears blank, and when you click on it, you get a weird transverse selection box.
Comment 2 James Forrester 2013-07-11 05:32:32 UTC
*** Bug 50267 has been marked as a duplicate of this bug. ***
Comment 3 This, that and the other (TTO) 2013-07-19 11:32:31 UTC
(In reply to comment #1)
> In Firefox the text is overlapped; in Chrome it is worse - the template
> appears blank

This part is now working fine.

> in Chrome , when you click on it, you get a weird transverse selection box.

This is still a problem.
Comment 4 WhatamIdoing 2014-01-09 00:29:19 UTC
The transverse selection "just happens" to be the width of one column and the height that the text would take if it were not split into columns.

The problem may be in the template.  A similar problem was found to be due to someone starting a set of columns with {{multicol}} but finishing it with {{col-end}} (rather than properly closing it with {{multicol-end}}).  This {{columns-list}} template appears to be a wrapper for {{div col}}, but has no {{div col end}}.
Comment 5 Krinkle 2014-07-03 00:54:48 UTC
Was about to report a similar bug for another template. Narrowed it down to the CSS property "column-count". It's not specific to templates (though unlikely to appear in nodes other than transclusion nodes right now).

Basically the different boundary rectangles we get from the browser for these kind of elements also seem to include a bad layer. That bad layer is horizontally centred and in the shape of how the contents would be were it divided into columns.
Comment 6 Krinkle 2014-07-03 00:55:21 UTC
Created attachment 15825 [details]
Screenshot of typical two-column reflist highlighted in Chrome
Comment 7 Krinkle 2014-07-03 00:57:16 UTC
Created attachment 15826 [details]
Screenshot of case with more than 2 columns

In this case it's even more problematic as the column is quite narrow (the width of an individual column) and expands both ways. Thus the "top right corner" where the context menu is, can be offscreen (higher up on the page).

In the screenshot I scrolled up so that it is just about visible, but this wouldn't be the case for a regular user.
Comment 8 Ed Sanders 2014-07-03 13:42:15 UTC
This is clearly a browser bug, will file upstream, but we may have to find a workaround.
Comment 9 Ed Sanders 2014-07-03 13:52:31 UTC
Upstream: https://code.google.com/p/chromium/issues/detail?id=391271
Comment 10 Bartosz Dziewoński 2014-10-03 21:07:05 UTC
*** Bug 71625 has been marked as a duplicate of this bug. ***

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


Navigation
Links