NPM Resource Changed ? Pending

User tests: Successful: Unsuccessful:

avatar Fedik
Fedik
6 Jan 2020

Summary of Changes

This will load webcomponents with asset manager.

Testing Instructions

Apply patch run npm install
Click around, try create/edit content, all should work as before.

Expected result

Everything works

Actual result

Everything works

Documentation Changes Required

Need to update Asset documentation to describe how to add webcomponent to it.

ping @wilsonge

avatar Fedik Fedik - open - 6 Jan 2020
avatar Fedik Fedik - change - 6 Jan 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Jan 2020
Category Administration com_associations com_categories com_config com_fields com_media com_modules Repository NPM Change Layout Libraries
avatar jwaisner jwaisner - test_item - 6 Jan 2020 - Tested successfully
avatar jwaisner
jwaisner - comment - 6 Jan 2020

I have tested this item successfully on 0afe53a


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

avatar dgrammatiko
dgrammatiko - comment - 6 Jan 2020

@Fedik are you also covering this hidden functionality:

self::_('stylesheet', str_replace('.js', '.css', $file), $options);

avatar Fedik
Fedik - comment - 6 Jan 2020

nope, it not hidden now
it need to explicitly call to the style if webcomponent have a style, eg:

Factory::getDocument()->getWebAssetManager()
->useStyle('webcomponent.field-simple-color')
->useScript('webcomponent.field-simple-color');

avatar dgrammatiko
dgrammatiko - comment - 6 Jan 2020

much better ?

avatar wilsonge wilsonge - change - 7 Jan 2020
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-01-07 10:25:14
Closed_By wilsonge
Labels Added: NPM Resource Changed ?
avatar wilsonge wilsonge - close - 7 Jan 2020
avatar wilsonge wilsonge - merge - 7 Jan 2020
avatar wilsonge
wilsonge - comment - 7 Jan 2020

Thanks!

avatar wilsonge
wilsonge - comment - 7 Jan 2020

Thanks!

Add a Comment

Login with GitHub to post a comment