View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
0002383 | NetSurf | Javascript | public | 2015-11-06 20:15 | 2019-06-05 21:06 | ||||||||
Reporter | Chris Young | ||||||||||||
Assigned To | kinnison | ||||||||||||
Severity | minor | Reproducibility | always | ||||||||||
Status | confirmed | Resolution | open | ||||||||||
Platform | Amiga | OS | AmigaOS | OS Version | 4.1 | ||||||||
Product Version | 3.4 | ||||||||||||
Target Version | 4.0 | Fixed in Version | |||||||||||
Summary | 0002383: Website displaying without CSS | ||||||||||||
Description | Digital Spy is loading with no stylesheet. | ||||||||||||
Steps To Reproduce | Visit http://www.digitalspy.com | ||||||||||||
Additional Information | I can't see any mention of css files beyond the default NetSurf ones in the log. I know when they changed their site it went to a heavily Javascripted design, so it wouldn't surprise me if it is trying to do something clever we don't support yet. A quick glance at the code reveals it's using "Modernizr" to create the stylesheets using Javascript. | ||||||||||||
Tags | No tags attached. | ||||||||||||
Fixed in CI build # | |||||||||||||
Reported in CI build # | 3053 | ||||||||||||
URL of problem page | http://www.digitalspy.com | ||||||||||||
Attached Files |
|
Notes | |
Vincent Sanders (administrator) 2015-11-08 10:08 |
the site uses the following script to insert its css with javascript. var s, el; s = document.createElement('link'); s.rel = "stylesheet"; s.href = '//digitalspyuk.cdnds.net/digitalspyuk/20151103130454/css/main' + ( Modernizr.mobile ? '-mobile.css' : '.css' ); el = document.getElementById('write-styles'); document.getElementsByTagName("head")[0].insertBefore( s, el ); |
kinnison (administrator) 2015-11-08 13:52 |
This is using a numeric indexing into the html collection returned by document.getElementsByTagName("head") which is daft (document.head would be faster) Sadly until we do ES6 proxy support for collections this won't work. |
kinnison (administrator) 2015-11-08 17:14 |
In addition, the Modernizr stuff looks like CSSOM |
kinnison (administrator) 2019-06-05 21:06 |
Current state of play, we have proxy support so that bit works, but the CSSOM still won't. |
Issue History | |||
Date Modified | Username | Field | Change |
---|---|---|---|
2015-11-06 20:15 | Chris Young | New Issue | |
2015-11-06 20:15 | Chris Young | File Added: ns.log | |
2015-11-07 13:54 | Chris Young | Category | Layout => Javascript |
2015-11-07 13:54 | Chris Young | Additional Information Updated | View Revisions |
2015-11-08 10:08 | Vincent Sanders | Note Added: 0001040 | |
2015-11-08 10:08 | Vincent Sanders | Status | new => confirmed |
2015-11-08 10:08 | Vincent Sanders | Additional Information Updated | View Revisions |
2015-11-08 13:52 | kinnison | Note Added: 0001041 | |
2015-11-08 14:44 | kinnison | Assigned To | => kinnison |
2015-11-08 14:44 | kinnison | Status | confirmed => assigned |
2015-11-08 14:44 | kinnison | Status | assigned => confirmed |
2015-11-08 17:14 | kinnison | Note Added: 0001042 | |
2016-02-09 11:05 | Michael Drake | Target Version | 3.4 => 4.0 |
2019-06-05 21:06 | kinnison | Note Added: 0001935 |