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

App looks broken when running on production mode IE11 #19

Closed
alvarezguille opened this issue Oct 19, 2018 · 15 comments
Closed

App looks broken when running on production mode IE11 #19

alvarezguille opened this issue Oct 19, 2018 · 15 comments

Comments

@alvarezguille
Copy link
Member

Tested with jetty:run and with package + running tomcat both with -Dvaadin.productionMode and -PproductionMode
Dark theme is not applied, and some other styling issues as seen on the following screenshot
image

Tested with vaadin 12.0.0.alpha4

@caalador caalador self-assigned this Oct 24, 2018
@caalador
Copy link
Contributor

Only thing missing for me is the table-logo.png that doesn't get served.
Have you tried clearing the browser cache and/or clearing all target folders?

@caalador caalador changed the title App looks broken when running on production mode App looks broken when running on production mode IE11 Oct 25, 2018
@SomeoneToIgnore
Copy link
Contributor

SomeoneToIgnore commented Oct 25, 2018

Is the issue fixed now?

@caalador
Copy link
Contributor

No. The logo image is fixed, but the problem is with IE11 and that's not fixed.

@caalador
Copy link
Contributor

All I can come up for this is that the css files use var() which is not supported for IE only for Edge.
It gets better if I remove all the var() usages from the shared-styles.css, but still it's not as it should be.

Perhaps there is a problem with the CSS polyfill for some reason.

@Legioth
Copy link
Member

Legioth commented Oct 25, 2018

Isn't it so that the standard polyfill only applies for styles in <custom-style>. You'd have to manually apply the polyfilling logic if you want to use it with standalone css files?

@caalador
Copy link
Contributor

That could be. Also I noted that for the [theme~=dark] IE gets the colors the wrong way so that color that should be hsla(214, 96%, 96%, 0.9) is actually hsla(213, 41%, 16%, 0.94) which is closer to the wanted background-color of hsl(214, 35%, 21%)

@caalador
Copy link
Contributor

Also the theme is broken for Vaadin 10.0.5 also when running IE11

@caalador
Copy link
Contributor

caalador commented Oct 25, 2018

For the Dark theme it might be a problem with the bundler that it probably doesn't check for the theme variant dark and only brings in the light lumo theme.

The dark theme being the light theme in production mode can be seen also with the skeleton-starter with just adding @Theme(value = Lumo.class, variant = Lumo.DARK) No dark theme is shown only the base Lumo.

@caalador
Copy link
Contributor

The transpilation step leaves me with this when IE11 loads the colors.html
screen shot 2018-10-29 at 13 38 33

@jouni
Copy link
Member

jouni commented Oct 30, 2018

Investigating this as well. I can’t make the dark theme work in IE11 in a pure Polymer app at the moment either.

@jouni
Copy link
Member

jouni commented Oct 30, 2018

Damn, I don’t seem to be able to make style modules work at all (<style include="foobar">).

Update: got that sorted by using at least one Polymer-based web component on the page. Otherwise it seems like style module includes are not applied at all.

@jouni
Copy link
Member

jouni commented Oct 30, 2018

Alright, this seems like a fundamental issue in Lumo itself, not only in this starter (I mean all Flow apps are probably broken in IE11 if they try to use the dark palette).

ShadyCSS seems to have a special case for the html selector. If you combine that with anything else (like html[theme~≈"dark"] or html, [theme~="dark"]) it will silently ignore polyfiling any custom properties inside that selector.

It seems we’ve (or I’ve) been assuming that the document root element would be polyfilled in all cases, no matter what selector matches it.

Needs to be fixed in vaadin-lumo-styles, and how the dark theme is applied on the global level, to support IE11. This could result in a breaking change, or at least deprecating the current way of applying the dark palette for the global scope.

pleku referenced this issue in vaadin/bookstore-starter-flow Oct 31, 2018
@jouni
Copy link
Member

jouni commented Oct 31, 2018

Please follow the issue I just created for vaadin-lumo-styles.

SomeoneToIgnore referenced this issue in vaadin/bookstore-starter-flow Nov 1, 2018
@caalador caalador removed their assignment May 24, 2019
@pleku pleku transferred this issue from vaadin/bookstore-starter-flow Dec 11, 2019
@anasmi
Copy link
Contributor

anasmi commented Jan 9, 2020

Seems to work in the latest version of starter:

IE_Production

@anasmi
Copy link
Contributor

anasmi commented Jan 20, 2020

Closing

@anasmi anasmi closed this as completed Jan 20, 2020
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

6 participants