Skip to content

Please review. I don't know if there are side effects of that patch!#95

Open
freakyjoe wants to merge 1 commit intoarturadib:masterfrom
freakyjoe:master
Open

Please review. I don't know if there are side effects of that patch!#95
freakyjoe wants to merge 1 commit intoarturadib:masterfrom
freakyjoe:master

Conversation

@freakyjoe
Copy link

An IE "bug-feature" (up to Version 9) limits stylesheets to a total count of 31. Rich apps with many widgets can hit that limit easily.

view.stylize() adds a <style> tag to for each instantiated object.

This patch adds an ID to this style tag and makes sure that it is removed from when .destroy() is called.

An IE "bug-feature" (up to Version 9) limits stylesheets to a total count of 31. Rich apps with many widgets can hit that limit easily.

view.stylize() adds a <style> tag to <head> for each instantiated object.

This patch adds an ID to this style tag and makes sure that it is removed from <head> when .destroy() is called.
@arturadib
Copy link
Owner

hi Johannes, superb! can you please add a test to make sure it works? thanks!

@freakyjoe
Copy link
Author

Hi Artur!
I will try. But might take a day since I've got to meet a deadline on the project that triggered the IE bug.

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

Successfully merging this pull request may close these issues.

2 participants