?
avatar astridx
astridx
29 Jan 2018

The issue #17914 described this warning first:

##Steps to reproduce the issue

  1. Enable Inspector in FireFox
  2. Go to the Control Panel
  3. Check the Console tab
  4. The notice is shown there

The issue #17914 is closed, because there was a PR.
The PR https://github.com/webcomponents/custom-elements/pull/112/files is merged.

Why do we still see this warning?

The warning occurs in the file webcomponents-ce.min.js:1:7969. I cant find a file with this name in the repo https://github.com/webcomponents/custom-elements or in the repo https://github.com/webcomponents/webcomponentsjs
Can it be that the file is outdated?

avatar astridx astridx - open - 29 Jan 2018
avatar joomla-cms-bot joomla-cms-bot - change - 29 Jan 2018
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 29 Jan 2018
avatar C-Lodder
C-Lodder - comment - 30 Jan 2018

Yes, this file needs updating.

avatar franz-wohlkoenig franz-wohlkoenig - change - 30 Jan 2018
Status New Discussion
avatar franz-wohlkoenig franz-wohlkoenig - change - 30 Jan 2018
Category Code style
avatar astridx
astridx - comment - 30 Jan 2018

How can we update thar file?

avatar dgt41
dgt41 - comment - 30 Jan 2018

So this is partly my fault as I didn’t realized that there were 2 console.warns in the script: webcomponents/custom-elements#106

avatar dgt41
dgt41 - comment - 30 Jan 2018

Ok, I was wrong, the latest file doesn not contain that console.log. Anyways I'll do a PR to get the latest polyfills and remove that warning

avatar dgt41
dgt41 - comment - 30 Jan 2018

@astridx #19484 fixes this and a few more things

avatar astridx
astridx - comment - 30 Jan 2018

With this patch I neither see the warning in the console of the browser nor in the javascript tests. So it is fine.
But I wonder where the file /media/system/js/polyfills/webcomponents/webcomponents-ce.js comes from. Is it a special file for Joomla! ?

avatar astridx
astridx - comment - 30 Jan 2018

Close as there is PR: #19484

avatar astridx astridx - change - 30 Jan 2018
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2018-01-30 12:33:44
Closed_By astridx
avatar astridx astridx - close - 30 Jan 2018

Add a Comment

Login with GitHub to post a comment