? Pending

User tests: Successful: Unsuccessful:

avatar dgt41
dgt41
9 Sep 2017

Pull Request for Issue # .

Summary of Changes

A change in the way we use styling for custom elements, from injected by the JS we are moving to inline css tags generated in the back end.
So the only actual change here is removing the js code that injected the inline css and restructuring the folder so no it looks like:

---webcomponents
-----js
-------some-component.min.js
-----css
-------some-component.min.css

Testing Instructions

Apply patch edit/save an article. Alert appears normally

Expected result

Actual result

screen shot 2017-09-09 at 19 26 30

Documentation Changes Required

The change will be documented in the other repo (joomla-projects/custom-elements)

@wilsonge

avatar joomla-cms-bot joomla-cms-bot - change - 9 Sep 2017
Category Administration Templates (admin) Libraries JavaScript Front End Templates (site)
avatar dgt41 dgt41 - open - 9 Sep 2017
avatar dgt41 dgt41 - change - 9 Sep 2017
Status New Pending
avatar wilsonge
wilsonge - comment - 10 Oct 2017

Is this now merged with what we did yesterday? Or does it need more work?

avatar dgt41
dgt41 - comment - 10 Oct 2017
			// Add the css if exists
			HTMLHelper::_('stylesheet', str_replace('.js', '.css', $value), $options);

I guess this covers the css-js splitting

avatar dgt41 dgt41 - change - 10 Oct 2017
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2017-10-10 09:02:23
Closed_By dgt41
Labels Added: ?
avatar dgt41 dgt41 - close - 10 Oct 2017

Add a Comment

Login with GitHub to post a comment