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

IE11 Support #1973

Closed
totaam opened this issue Oct 1, 2018 · 17 comments
Closed

IE11 Support #1973

totaam opened this issue Oct 1, 2018 · 17 comments

Comments

@totaam
Copy link
Collaborator

totaam commented Oct 1, 2018

Issue migrated from trac ticket # 1973

component: html5 | priority: minor | resolution: fixed

2018-10-01 19:12:02: mjharkin created the issue


@antoine Looks like I'm a little late on this as I think you've covered most of this in r20580 but maybe missing:

Fullscreen request:Added IE11 fullscreen request

startsWith and includes Polyfills
My approach was to add a seperate polyfill script so it's easier to remove when IE finally dies. For reference added IE11 compatability

@totaam
Copy link
Collaborator Author

totaam commented Oct 2, 2018

2018-10-02 06:45:06: antoine changed owner from antoine to mjharkin

@totaam
Copy link
Collaborator Author

totaam commented Oct 2, 2018

2018-10-02 06:45:06: antoine edited the issue description

@totaam
Copy link
Collaborator Author

totaam commented Oct 2, 2018

2018-10-02 06:45:06: antoine commented


Merged in r20583 + r20584, thanks.

Can you please specify what copyright applies to those polyfills? (should be the same as ECMA6 I guess - looks like a BSD variant?)

@totaam
Copy link
Collaborator Author

totaam commented Oct 2, 2018

2018-10-02 13:22:33: mjharkin commented


Sorry, started here https://github.com/paulmillr/es6-shim (MIT) but it was overkill for a few polyfills so ended up just grabbing them from elsewhere, I've already give myself a slap on the wrists!

Just tested r20584 but I think:

window.console = window.console || {
	  log: function () {}
};
	

needs to be in index.html otherwise Developer tools needs to be open.

@totaam
Copy link
Collaborator Author

totaam commented Oct 2, 2018

2018-10-02 13:51:12: antoine commented


needs to be in index.html otherwise Developer tools needs to be open.
No idea where that needs to go, I've tried a few obvious places and none helped.
(and since having the developer console open "fixes" things, catch 22)

@totaam
Copy link
Collaborator Author

totaam commented Oct 2, 2018

2018-10-02 14:09:10: mjharkin commented


Sorry my mistake I should have tidied up before submitting the change:

What's missing is actually (1st diff from mjharkin/Xpra@5904fe3 )
This could equally be added to es6-shim.js instead of index.html, depends on your preference.

var console = console || { log: function(){} };

The following should be removed from es6-shim.js

window.console = window.console || {
	  log: function () {}
};

@totaam
Copy link
Collaborator Author

totaam commented Oct 2, 2018

2018-10-02 15:29:46: antoine commented


Thanks, applied in r20592.

Can we close this ticket?

@totaam
Copy link
Collaborator Author

totaam commented Oct 2, 2018

2018-10-02 15:57:19: mjharkin commented


r20592 changes in index.html and connect.html can be removed.
Otherwise this is good to close.

Thanks.

@totaam
Copy link
Collaborator Author

totaam commented Oct 2, 2018

2018-10-02 16:13:40: antoine changed status from new to closed

@totaam
Copy link
Collaborator Author

totaam commented Oct 2, 2018

2018-10-02 16:13:40: antoine set resolution to fixed

@totaam
Copy link
Collaborator Author

totaam commented Oct 2, 2018

2018-10-02 16:13:40: antoine commented


Good catch, done in r20593.

@totaam
Copy link
Collaborator Author

totaam commented Mar 4, 2019

2019-03-04 12:11:42: mjharkin changed status from closed to reopened

@totaam
Copy link
Collaborator Author

totaam commented Mar 4, 2019

2019-03-04 12:11:42: mjharkin removed resolution (was fixed)

@totaam
Copy link
Collaborator Author

totaam commented Mar 4, 2019

2019-03-04 12:11:42: mjharkin commented


Try catch blocks without an argument were added recently.
IE11 doesn't like this, fix is to add the argument:

mjharkin/Xpra@a3258e7

@totaam
Copy link
Collaborator Author

totaam commented Mar 4, 2019

2019-03-04 12:21:59: antoine changed status from reopened to closed

@totaam
Copy link
Collaborator Author

totaam commented Mar 4, 2019

2019-03-04 12:21:59: antoine set resolution to fixed

@totaam
Copy link
Collaborator Author

totaam commented Mar 4, 2019

2019-03-04 12:21:59: antoine commented


Thanks!
Applied in r21961.

@totaam totaam closed this as completed Mar 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant