Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

md-progress-linear: IE 11 Flickering, Constant resizing of md-dialog container width #2364

Closed
Londovir opened this issue Apr 16, 2015 · 4 comments
Labels
browser: IE This issue is specific to Internet Explorer for: internal contributor The team will address this issue and community PRs are not requested.

Comments

@Londovir
Copy link

Relevant CodePen: http://codepen.io/londovir/pen/PwMMZJ

Under Chrome, an md-progress-linear that is embedded within an md-dialog component will show a slightly resizing (likely due to the transformX/scale CSS transformation done for a "determinate" style md-progress-linear) every time the attached value attribute is changed. However, the overall outer md-dialog will not resize accordingly. In addition, there is no apparently "flicker" of the darker colored progress bar portion as it is redrawn/resized by the CSS transformation - it smoothly glides smaller or larger.

Under IE 11, the exact same CodePen is causing multiple issues. The dark color "value bar" flickers out of existence before coming back, making for a jarring visual experience. In addition, the outer md-dialog component visually resizes in the horizontal (and often vertical) directions, causing a noticeable "shiver" of the md-dialog component it's embedded within. We have tried various combinations of wrapping different elements in outer DIVs with various attributes to try and prevent this layout resizing from taking place, but nothing has yet worked that doesn't mandate a fixed size for the outer md-dialog, which we are trying to avoid if possible.

@ThomasBurleson ThomasBurleson added the for: internal contributor The team will address this issue and community PRs are not requested. label Apr 24, 2015
@ThomasBurleson ThomasBurleson added the browser: IE This issue is specific to Internet Explorer label May 27, 2015
@ThomasBurleson ThomasBurleson modified the milestone: Backlog May 27, 2015
@Splaktar
Copy link
Member

Splaktar commented Jul 6, 2015

Just tested this out with the latest master and all of the flickering and shivering appears to be gone. @Londovir can you please confirm this?

@Londovir
Copy link
Author

Londovir commented Jul 6, 2015

@Splaktar - Wish I could confirm, but I still have it in the horizontal direction in the CodePen I referenced and with IE 11 (v11.0.9600.17842). Here's a GIF of what I'm seeing on my screen under IE11:

IE11
animation 2

It is definitely still flickering in the horizontal direction. You can see the effect on the entire dialog layout by looking at the buttons, especially, along with the slight blurring of the text as well. I'm referencing the bower-material master, so I presume I'm pulling the most recent commit in this CodePen. Please note that the flashing of the darker portion of the progress bar is not an artifact due to the GIF creation, but is what I'm actually seeing in IE11. In Chrome, although I do not have the overall dialog box flickering in the horizontal direction, there is a shrinking and growing by a pixel or two that is happening. It is subtle, but can be seen if you look closely (paricularly at the lighter colored right edge of the progress bar, and how it aligns with the end of the text line above it):

Chrome
animation 1

I believe in Chrome this is "absorbed" by the layout which doesn't cause the dialog itself to horizontally resize, but in IE11 this constant changing of the width (?) of the bar by a pixel back and forth is causing the horizontal resizing you see in the GIF. What is interesting is I don't see this resizing behavior in Chrome on the demo page for the MD project...just in the custom dialog in my CodePen, so there must be something happening in my layout for the problem to be triggered.

Thanks.

@Splaktar
Copy link
Member

@Londovir I am not seeing that on Windows 7 with IE 11. What version of Windows are you running? Do you have all of the latest Windows Updates applied? I'll look into this a little more when I have some time.

@Londovir
Copy link
Author

You may be on to something. I just checked a Windows 7SP1 x86 VM I have running IE11 (11.0.9600.17843), and it is rock steady as expected. I don't even get the odd vertical 1px change in progress bar height like I see in Chrome as you see above. I will check my Win10 Insider Edge VM tomorrow at work to see if it is being replicated there. It does appear in my Win8.1 IE (Metro Mode) as well - and on more than one Win8.1 machine - so it is unlikely to be a specific graphics card driver or some other issue that I can think of, as it is happening on a variety of Win8.1 machines.

ThomasBurleson added a commit that referenced this issue Sep 3, 2015
…ons, perf upgrades

synchronize progressLinear with similar logic used in progressCircular.

* improve animation performances
* watch md-mode for changes
* refactor animation SCSS
* enable hiding and no-animations with undefined/empty md-mode attributes
* for both indicators, use `display:inline-block;`
* update demos with enable switch
* fix query mode
* update Select to use enhanced progressCircular component
* fix autocomplete styling of progress-linear.md-mode-indeterminate

BREAKING-CHANGES

Before:

```css
md-progress-linear {
  display: block;
}
md-progress-circular {
   // display not set
   // position not set
}
```

```css
md-progress-linear {
  display: inline-block;
  position: relative;
}
md-progress-circular {
  display: inline-block;
  position: relative;
}
```

Fixes #4421. Fixes #4409. Fixes #2540. Fixes #2364. Fixes #1926. Fixes #3802.
ThomasBurleson added a commit that referenced this issue Sep 3, 2015
…ons, perf upgrades

synchronize progressLinear with similar logic used in progressCircular.

* improve animation performances
* watch md-mode for changes
* refactor animation SCSS
* enable hiding and no-animations with undefined/empty md-mode attributes
* for both indicators, use `display:inline-block;`
* update demos with enable switch
* fix query mode
* update Select to use enhanced progressCircular component
* fix autocomplete styling of progress-linear.md-mode-indeterminate
*  auto-inject md-mode attribute if missing
  *  use 'determinate' if value attribute is defined
  *  otherwise use 'indeterminate'
  *  $log.debug() notify user (via $log.debug) of injection
  *  add API doc details regarding md-mode auto-injection
* fix tests

BREAKING-CHANGES

Before:

```css
md-progress-linear {
  display: block;
}
md-progress-circular {
   // display not set
   // position not set
}
```

```css
md-progress-linear {
  display: inline-block;
  position: relative;
}
md-progress-circular {
  display: inline-block;
  position: relative;
}
```

Fixes #4421. Fixes #4409. Fixes #2540. Fixes #2364. Fixes #1926. Fixes #3802.
kennethcachia pushed a commit to kennethcachia/material that referenced this issue Sep 23, 2015
…ons, perf upgrades

synchronize progressLinear with similar logic used in progressCircular.

* improve animation performances
* watch md-mode for changes
* refactor animation SCSS
* enable hiding and no-animations with undefined/empty md-mode attributes
* for both indicators, use `display:inline-block;`
* update demos with enable switch
* fix query mode
* update Select to use enhanced progressCircular component
* fix autocomplete styling of progress-linear.md-mode-indeterminate
*  auto-inject md-mode attribute if missing
  *  use 'determinate' if value attribute is defined
  *  otherwise use 'indeterminate'
  *  $log.debug() notify user (via $log.debug) of injection
  *  add API doc details regarding md-mode auto-injection
* fix tests

BREAKING-CHANGES

Before:

```css
md-progress-linear {
  display: block;
}
md-progress-circular {
   // display not set
   // position not set
}
```

```css
md-progress-linear {
  display: block;
  position: relative;
}
md-progress-circular {
  display: block;
  position: relative;
}
```

Fixes angular#4421. Fixes angular#4409. Fixes angular#2540. Fixes angular#2364. Fixes angular#1926. Fixes angular#3802. Closes angular#4454.
@Splaktar Splaktar removed this from the - Backlog milestone Feb 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
browser: IE This issue is specific to Internet Explorer for: internal contributor The team will address this issue and community PRs are not requested.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants