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

[Updated] Horizontal scrolling performance is terrible in IE9 #2273

Closed
acostaf opened this issue Dec 4, 2014 · 7 comments
Closed

[Updated] Horizontal scrolling performance is terrible in IE9 #2273

acostaf opened this issue Dec 4, 2014 · 7 comments
Assignees
Milestone

Comments

@acostaf
Copy link

acostaf commented Dec 4, 2014

On this grid you may see the Horizontal and Vertical Scrollbar completely out of place, that just happens when you see it on IE9
grid issue 2

Same grid opened on Chome 41
grid issue 3

In this grid taken from http://ui-grid.info/docs/#/tutorial/213_auto_resizing you also may see the same issue since I am opening it on IE9
grid issue 1

@alexandery
Copy link

I'm seeing the same issue with styling on IE9. Scrollbars are out of place, as well as column header popup menus - those popup in random spots.

@alexandery
Copy link

I've replied to another thread on IE9 issues, but this in case you still need IE9 fix, here is what fixed it for us:

.ui-grid {
position: relative;
}

.ui-grid-render-container
{
width: 100%;
height: 100%;
position: relative;
}

There is something weird with .ui-grid-viewport style - it has some set height the source of which we couldn't find anywhere.

@PaulL1 PaulL1 added this to the 3.0 milestone Dec 10, 2014
@PaulL1
Copy link
Contributor

PaulL1 commented Jan 14, 2015

Could someone look at this on the latest unstable. There have been quite a few changes gone in on scrolling which may resolve this. Also, the grid menus are now positioned absolute, which should fix the popup menus.

@alexandery
Copy link

@PaulL1 I just checked one of the tutorial pages with IE9 and see that the scrollbars are in place, however there seems to be an issue with a horizontal scrolling - as if a scrollbar appears even when it doesn't need to be there and when engaged it does move columns but not the header.

This is broken in FF as well, not just IE.

Here is the screenshot:

scroll ie9

Also, opening http://ui-grid.info/docs/#/tutorial/191_horizontal_scrolling in IE9 takes extremely long time (like a minute?) and almost as bad in FF. Horizontal scrolling is extremely choppy and slow in both browsers, but especially in IE9.

@swalters
Copy link
Contributor

I can say that it is a total pig on IE9. Unusable. Maybe IE9 fires more scroll events than the other browsers and therefore we need to throttle it.

@PaulL1 PaulL1 assigned swalters and c0bra and unassigned swalters Jan 22, 2015
@c0bra
Copy link
Contributor

c0bra commented Jan 22, 2015

Note to self: look at using track by $index on col repeater unless there's custom cellTemplates in the column defs, and alter the repeater template if a cellTemplate appears or all disappear from the column defs.

@c0bra
Copy link
Contributor

c0bra commented Jan 30, 2015

Changing the topic of this bug as the scrollbar positioning in IE9 is fixed, but the horizontal performance issue is still present.

@c0bra c0bra changed the title Horizontal Scrollbar and Vertical Scrollbar broken on IE9 with version 3.0.0-rc.16 [Updated] Horizontal scrolling performance is terrible in IE9 Jan 30, 2015
@c0bra c0bra modified the milestones: 3.1, 3.0 Apr 16, 2015
mportuga pushed a commit that referenced this issue Mar 11, 2018
Since this browser is frequently the one that gives us issues for new changes and we are moving
towards angular 5 support, I am dropping it from our supported list.

BREAKING CHANGE: IE9 is no longer officially supported.

Closes #2273, #2552, #3593, #3854, #4439
mportuga pushed a commit that referenced this issue Mar 11, 2018
Since this browser is frequently the one that gives us issues for new changes and we are moving
towards angular 5 support, I am dropping it from our supported list.

BREAKING CHANGE: IE9 is no longer officially supported.

Closes #2273, #2552, #3593, #3854, #4439
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants