Core: scrollParent not working properly in IE for static or relative elements

Confirmed the first part of this.

scrollParent

linus.yan87,

Could you clarify or post a test case for this:

''"In fact, elements which have dimension(height|width is not auto) and overflow is auto|scroll, they are also scroll parent."''

Replying to [comment:2 tj.vantoll]:

linus.yan87, Could you clarify or post a test case for this: ''"In fact, elements which have dimension(height|width is not auto) and overflow is auto|scroll, they are also scroll parent."''

Test case address: http://jsfiddle.net/linus/N8Pmc/. In firefox, it will alert HTMLDivElement; But, in IE, it alerts HTMLDocument.

I also ran into this problem but i needed the draggable stuff to scroll when over flow was hidden so I extended the scrollParent function and that is when I found the IE issue.

Here is the extend I did to find this issue

http://jsfiddle.net/g4M4t/3/

Here is the, I guess, kind of fix I came up with. I'm not sure if its a good fix or not.

http://jsfiddle.net/g4M4t/6/

Working example

http://jsfiddle.net/N8Pmc/2/

Replying to [comment:4 ScottGaines]:

I also ran into this problem but i needed the draggable stuff to scroll when over flow was hidden so I extended the scrollParent function and that is when I found the IE issue. Here is the extend I did to find this issue http://jsfiddle.net/g4M4t/3/ Here is the, I guess, kind of fix I came up with. I'm not sure if its a good fix or not. http://jsfiddle.net/g4M4t/6/ Working example http://jsfiddle.net/N8Pmc/2/

overflow:hidden is not allowed to have scroll bar, so it's not scroll parent. I'm inclined to advise you to change the height dynamically.

Replying to [comment:5 linus.yan87]:

Replying to [comment:4 ScottGaines]: > I also ran into this problem but i needed the draggable stuff to scroll when over flow was hidden so I extended the scrollParent function and that is when I found the IE issue. > > Here is the extend I did to find this issue > http://jsfiddle.net/g4M4t/3/ > > Here is the, I guess, kind of fix I came up with. I'm not sure if its a good fix or not. > http://jsfiddle.net/g4M4t/6/ > > > Working example > http://jsfiddle.net/N8Pmc/2/ overflow:hidden is not allowed to have scroll bar, so it's not scroll parent. I'm inclined to advise you to change the height dynamically.

It wasn't the height I needed but the width and changing the width did not fit my needs for the project I was working on. I don't want a scroll bar but the scroll-ability. I couldn't get the desired effect with changing the width. If you have a better idea I'd love to hear it.

This is what I was doing when I found this IE issue that this topic is about (still in development)

http://jsfiddle.net/rbb4X/ (the one with the IE Issue)

Click and drag the tab to the side makes the other tabs scroll so you can sort across all of the tabs works in firefox chrome and safari but not IE. Also if I didn't extend the the scroll parent to use hidden the it would only work if it had a scroll bar, and in everything but IE. Removing the if statement in the scroll parent function where it test for IE and just used the part in the else fixed it in IE 9. Now I'm not sure it the IE test was there for an older version of IE or not I don't have an older version of IE to test with.

With the dragging I couldn't see a way the default scroll in the draggable widget would work when you have a case when you do not want a scroll bar (overflow:hidden).

Doing what you suggested would seem like your suggesting to extend another widget to do a functional aspect that I can see being useful in many ways that basically already exists in another form but not supported. I don't see how an overflow of hidden would not be considered a scrollParent just because i dose not use a scrollbar even though it dose have the capabilities of scrolling.

I could extend the draggable widget to do what i need it to do but extending the scrollParent function was the fastest least hacky was to do it

Looks like this was fixed by https://github.com/jquery/jquery-ui/commit/44b21807