-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
webapp does not work and makes crash IE11 #403
Comments
@volaya this looks like yours |
The app in that zip file works perfectly with no error in the js console, both if i open it directly or if i serve ir (using python SimpleHTTPServer) How do you get that error, @bartvde Line 410 is correct, but apparently missing a semicolon at the end. I I fixed that. Maybe your browser (I am using Chrome) is less tolerant to that? |
@volaya possibly, this report is specific for IE11, on other browsers (FF, Chrome) is ok. |
right this was IE, and IE is sensitive to that |
so probably needs to be retested on WAB master right? |
@bartvde just retested on master, no changes. |
@bartvde @volaya even with changes that fixed #409 this is a no go |
ok checking your zips |
yeah I get the same in IE11 |
can we pinpoint this somehow to some of the functionality @gioman I wonder if it's caused by the rendering extensions or something else |
the webapp has no widgets, overlays or basemaps active. |
can we check a project without the rendering extensions in it? |
@volaya this is only added in the basic theme right? |
I have checked several simple projects and beside a few glitches/problems (see layer list) they worked on IE11. |
@gioman can you check this exact same project with the tabbed theme? |
it's caused by the pattern fills based on svg |
@bartvde so no need to test the tabbed version? |
@bartvde @volaya trying to compile this project in tabbed mode fails:
|
can you attach the app.jsx file? |
1 similar comment
can you attach the app.jsx file? |
This can be fixed by exporting svgs to pngs like we do for markers. However, this might not be ready for release, so I suggest leaving it like that and documenting this limitation (svg pattern fills + IE). I have opened a JIRA ticket to address this later and not forget about it @SrNetoChan what o you think? |
@bartvde attached |
ah looks to be the trailing frontal comma on the tablist:
|
no sorry it's this line:
|
Seen trough a web server (apache). This is the webapp as generated by the new manual test (tester plugin) "osm".
cb8876bcc68d4ea4b4a878b87f078c26.zip
The text was updated successfully, but these errors were encountered: