jenkins-bot has submitted this change and it was merged.
Change subject: BugID 43076 - Fixes to alien and phantom corner cases
......................................................................
BugID 43076 - Fixes to alien and phantom corner cases
The {{coords}} template is correctly rendered as an inlineAlien, which
previously did not have shields. However, the template has a deeply nested span
that is positioned and floated. The fix is to shift shielding logic up to
AlienNode (from AlienBlockNode) and to show phantoms on all shields (both block
and inline). I've also updated the aliens demo to exhibit this corner case.
Change-Id: Ifce60c7762c0ead5c6fe29d6eabf601c1565cbfa
---
M demos/ve/pages/aliens.html
M modules/ve/ce/nodes/ve.ce.AlienBlockNode.js
M modules/ve/ce/nodes/ve.ce.AlienInlineNode.js
M modules/ve/ce/nodes/ve.ce.AlienNode.js
4 files changed, 43 insertions(+), 63 deletions(-)
Approvals:
Trevor Parscal: Looks good to me, approved
jenkins-bot: Verified
--
To view, visit https://gerrit.wikimedia.org/r/39834
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ifce60c7762c0ead5c6fe29d6eabf601c1565cbfa
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Christian <[email protected]>
Gerrit-Reviewer: Inez <[email protected]>
Gerrit-Reviewer: Trevor Parscal <[email protected]>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
