? Pending

User tests: Successful: Unsuccessful:

avatar dgrammatiko
dgrammatiko
25 Nov 2018

Pull Request for Issue https://twitter.com/jaffathecake/status/1011214372696395777.

Summary of Changes

TL;DR

The web is a streaming media thus chunks of code served from the server might contain elements partially thus it shouldn't be assumed that the browsers will render the custom elements with all their childrens nodes synchronously. We overcome this by setting an observer to watch the state of the child nodes.

Testing Instructions

Apply this patch, change the editor to none and ensure that everything still works as before

Also you can check the lazy loading of the textarea element (what this PR is fixing) here:
https://codepen.io/dgrammatiko/pen/pQKjeR

screenshot 2018-11-25 at 14 46 35

Expected result

Custom element work in an async mode

Actual result

Custom element works only in a synchronous mode

Documentation Changes Required

Nope

avatar dgrammatiko dgrammatiko - open - 25 Nov 2018
avatar dgrammatiko dgrammatiko - change - 25 Nov 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 25 Nov 2018
Category JavaScript Repository
avatar dgrammatiko dgrammatiko - change - 25 Nov 2018
Title
editor none
[4.0] Custom Elements fix - editor none
avatar dgrammatiko dgrammatiko - edited - 25 Nov 2018
avatar wilsonge wilsonge - change - 27 Nov 2018
Labels Added: ?
avatar wilsonge
wilsonge - comment - 9 Dec 2018

I have tested this item successfully on 0543bd4

Editor none still works after patch in com_content (and tested media, page break and article quick icons successfully)


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/23169.

avatar wilsonge wilsonge - test_item - 9 Dec 2018 - Tested successfully
avatar wilsonge
wilsonge - comment - 9 Dec 2018

@dgrammatiko looks like this combined with your babel minifying code has broken drone. Can you please investigate :)

dce596f 9 Dec 2018 avatar dgrammatiko upd
avatar dgrammatiko
dgrammatiko - comment - 9 Dec 2018

@wilsonge done

avatar wilsonge wilsonge - change - 9 Dec 2018
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-12-09 23:48:13
Closed_By wilsonge
avatar wilsonge wilsonge - close - 9 Dec 2018
avatar wilsonge wilsonge - merge - 9 Dec 2018
avatar wilsonge
wilsonge - comment - 9 Dec 2018

Thanks :)

Add a Comment

Login with GitHub to post a comment