Page MenuHomePhabricator

[Regression pre-wmf.2] Red highlight appearing instead of blue on selecting an image
Closed, ResolvedPublic1 Estimated Story Points

Description

When I select an image, the highlight on selection is red instead of blue .

Screen Shot 2015-10-02 at 1.07.24 PM.png (334×535 px, 85 KB)

Event Timeline

Ryasmeen raised the priority of this task from to Needs Triage.
Ryasmeen updated the task description. (Show Details)
Ryasmeen added a project: VisualEditor.
Ryasmeen subscribed.
Jdforrester-WMF triaged this task as Unbreak Now! priority.Oct 2 2015, 9:14 PM
Jdforrester-WMF set Security to None.
Jdforrester-WMF edited a custom field.

This is weird, but is it really UBN?

This comes from .ve-ce-focusableNode-highlight-error

generatedContentsValid starts off false, and is never changed for ve.ce.MWImageNode instances because they don't call their parent render method (seems they generate a URL instead of DOM elements?)

This is weird, but is it really UBN?

Yeah, it's my screw-up for merging something without spotting this. :-)

J.

Change 243499 had a related patch set uploaded (by Esanders):
Follow-up I553e31: Don't show error for non-rendered GCNs

https://gerrit.wikimedia.org/r/243499

Change 243499 merged by jenkins-bot:
Follow-up I553e31: Don't show error for non-rendered GCNs

https://gerrit.wikimedia.org/r/243499