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

Broken layout and style in Edge 40 (EdgeHTML 15) #39

Closed
1 of 8 tasks
feuerste opened this issue Jun 9, 2017 · 4 comments
Closed
1 of 8 tasks

Broken layout and style in Edge 40 (EdgeHTML 15) #39

feuerste opened this issue Jun 9, 2017 · 4 comments

Comments

@feuerste
Copy link

feuerste commented Jun 9, 2017

Description

In the latest Edge 40.15063.0.0 from Windows 10 Creators Update, the layout and style of the paper-listbox is broken, also when used e.g. inside paper-dropdown-menu.

Expected outcome

paper-listbox-edge38
paper-dropdown-menu-edge38

Actual outcome

paper-listbox-edge40
paper-dropdown-menu-edge40

Live Demo

https://www.webcomponents.org/element/PolymerElements/paper-listbox/demo/demo/index.html
https://www.webcomponents.org/element/PolymerElements/paper-dropdown-menu/demo/demo/index.html

Steps to reproduce

  1. Download and install Windows 10 Creators Update from https://www.microsoft.com/en-us/software-download/windows10
  2. Run any of the above live demos in Edge 40

Browsers Affected

  • Chrome
  • Firefox
  • Safari 9
  • Safari 8
  • Safari 7
  • Edge 40
  • IE 11
  • IE 10
@rediche
Copy link

rediche commented Jul 5, 2017

I've noticed this too. It can be replicated on browserstack.com.

Edge 40 is running EdgeHTML 15 for clarification.

This was not broken on Edge 38 with EdgeHTML 14.

@rediche
Copy link

rediche commented Jul 6, 2017

I've done some further research on this issue.

Edge 40 & EdgeHTML 15 seems to be causing a wide variety of styling issues across the board.

Here is some similar cases:

A way I've found to fix some of the issues of paper-listbox/paper-item/paper-styles is to go directly into roboto.html in the package font-roboto and add type="text/css. It does not fix all the issues, but some. The paper-listbox will be vertical again. Only issue I can see on that solution, is that the selected item "jumps around" on screen. I also have no idea how that fix actually works, so problem is probably to be found elsewhere.

Overall it seems like the issues are going back to paper-styles, perhaps even Edge 40's support of CSS Custom Properties. It is already known that Edge 40 has some issues with them.

@feuerste
Copy link
Author

feuerste commented Aug 3, 2017

I just found a workaround for the Edge 40 issues:
By enforcing

paper-item:focus,
paper-icon-item:focus {
  position: relative;
  opacity: 1.0;
  background-color: var(--paper-grey-500);
}

in a shared style file I could fix it throughout our application.
This should also work for PolymerElements/paper-styles#145 and PolymerElements/paper-item#115.

@feuerste
Copy link
Author

I don't get the problems any more with webcomponentsjs 1.0.7, which uses shadycss 1.0.4, see also webcomponents/shadycss@74577b1 and webcomponents/shadycss@58575e5

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

2 participants