Skip to content
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

Open
gioman opened this issue May 30, 2017 · 25 comments
Open

webapp does not work and makes crash IE11 #403

gioman opened this issue May 30, 2017 · 25 comments
Assignees
Labels

Comments

@gioman
Copy link

gioman commented May 30, 2017

Seen trough a web server (apache). This is the webapp as generated by the new manual test (tester plugin) "osm".

cb8876bcc68d4ea4b4a878b87f078c26.zip

@bartvde
Copy link
Contributor

bartvde commented May 30, 2017

selection_282

@volaya this looks like yours

@volaya
Copy link
Contributor

volaya commented May 31, 2017

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?

@gioman
Copy link
Author

gioman commented May 31, 2017

@volaya possibly, this report is specific for IE11, on other browsers (FF, Chrome) is ok.

@bartvde
Copy link
Contributor

bartvde commented May 31, 2017

right this was IE, and IE is sensitive to that

@bartvde
Copy link
Contributor

bartvde commented May 31, 2017

so probably needs to be retested on WAB master right?

@bartvde bartvde removed their assignment May 31, 2017
@gioman
Copy link
Author

gioman commented May 31, 2017

@bartvde just retested on master, no changes.

webapp.zip

ea1c2f297a084d74aa59687c264e457a.zip

@gioman
Copy link
Author

gioman commented May 31, 2017

@bartvde @volaya even with changes that fixed #409 this is a no go

screenshot_20170531_122406

58fc04d67b1c4d1cb96fc9eece1f641a.zip

webapp.zip

@bartvde
Copy link
Contributor

bartvde commented May 31, 2017

ok checking your zips

@bartvde
Copy link
Contributor

bartvde commented May 31, 2017

yeah I get the same in IE11

@bartvde
Copy link
Contributor

bartvde commented May 31, 2017

can we pinpoint this somehow to some of the functionality @gioman I wonder if it's caused by the rendering extensions or something else

@gioman
Copy link
Author

gioman commented May 31, 2017

the webapp has no widgets, overlays or basemaps active.

@bartvde
Copy link
Contributor

bartvde commented May 31, 2017

can we check a project without the rendering extensions in it?

@bartvde
Copy link
Contributor

bartvde commented May 31, 2017

@volaya this is only added in the basic theme right?

@gioman
Copy link
Author

gioman commented May 31, 2017

can we check a project without the rendering extensions in it?

I have checked several simple projects and beside a few glitches/problems (see layer list) they worked on IE11.

@bartvde
Copy link
Contributor

bartvde commented May 31, 2017

@gioman can you check this exact same project with the tabbed theme?

@bartvde
Copy link
Contributor

bartvde commented May 31, 2017

it's caused by the pattern fills based on svg

@gioman
Copy link
Author

gioman commented May 31, 2017

@bartvde so no need to test the tabbed version?

@gioman
Copy link
Author

gioman commented May 31, 2017

@bartvde @volaya trying to compile this project in tabbed mode fails:

2017-05-31T08:04:44	2	WebAppBuilder: Cannot build webapp: Network error #400: Error downloading https://api.boundlessgeo.io/v1/websdk/package - server replied: SyntaxError: /tmp/websdk/f3064414-eaa9-46b6-bffc-61d24b906ba8/webapp/app.jsx: Unexpected token (2017:109)
			                                                                    ref:"table", pointZoom:16, map: map,
			                                                                    sortable:true, pageSize:20})
			                                )], open: this.state.leftNavOpen, onRequestClose: this.leftNavClose.bind(this}),
			      React.createElement("div", undefined,
			        React.createElement(Header, toolbarOptions ),
			        React.createElement("div", {className: 'map', style: {left: this.state.leftNavOpen ? leftNavWidth : 0, width: this.state.leftNavOpen ? 'calc(100% - ' + leftNavWidth + 'px)' : '100%'}},
			    at Parser.pp.raise (/tmp/websdk/f3064414-eaa9-46b6-bffc-61d24b906ba8/webapp/node_modules/babylon/lib/parser/location.js:24:13)
			    at Parser.pp.unexpected (/tmp/websdk/f3064414-eaa9-46b6-bffc-61d24b906ba8/webapp/node_modules/babylon/lib/parser/util.js:82:8)
			    at Parser.pp.expect (/tmp/websdk/f3064414-eaa9-46b6-bffc-61d24b906ba8/webapp/node_modules/babylon/lib/parser/util.js:76:33)
			    at Parser.pp.parseExprList (/tmp/websdk/f3064414-eaa9-46b6-bffc-61d24b906ba8/webapp/node_modules/babylon/lib/parser/expression.js:785:12)
			    at Parser.pp.parseSubscripts (/tmp/websdk/f3064414-eaa9-46b6-bffc-61d24b906ba8/webapp/node_modules/babylon/lib/parser/expression.js:270:29)
			    at Parser.pp.parseExprSubscripts (/tmp/websdk/f3064414-eaa9-46b6-bffc-61d24b906ba8/webapp/node_modules/babylon/lib/parser/expression.js:240:17)
			    at Parser.pp.parseMaybeUnary (/tmp/websdk/f3064414-eaa9-46b6-bffc-61d24b906ba8/webapp/node_modules/babylon/lib/parser/expression.js:217:19)
			    at Parser.pp.parseExprOps (/tmp/websdk/f3064414-eaa9-46b6-bffc-61d24b906ba8/webapp/node_modules/babylon/lib/parser/expression.js:163:19)
			    at Parser.pp.parseMaybeConditional (/tmp/websdk/f3064414-eaa9-46b6-bffc-61d24b906ba8/webapp/node_modules/babylon/lib/parser/expression.js:145:19)
			    at Parser.pp.parseMaybeAssign (/tmp/websdk/f3064414-eaa9-46b6-bffc-61d24b906ba8/webapp/node_modules/babylon/lib/parser/expression.js:112:19)
			

@bartvde
Copy link
Contributor

bartvde commented May 31, 2017

can you attach the app.jsx file?

1 similar comment
@bartvde
Copy link
Contributor

bartvde commented May 31, 2017

can you attach the app.jsx file?

@volaya
Copy link
Contributor

volaya commented May 31, 2017

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?

@gioman
Copy link
Author

gioman commented May 31, 2017

can you attach the app.jsx file?

@bartvde attached

app.jsx.zip

@SrNetoChan
Copy link
Contributor

@volaya I have documented in here:

47b41b1

Is it ok?

@bartvde
Copy link
Contributor

bartvde commented May 31, 2017

ah looks to be the trailing frontal comma on the tablist:

tabList: [,
React.createElement```

@bartvde
Copy link
Contributor

bartvde commented May 31, 2017

no sorry it's this line:

this.leftNavClose.bind(this}) will fix in WAB

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants