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

use getClientRects for IE10+ clientWidth/Height calculation #96

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jonirrings
Copy link

a quick fix for the bug mentioned in #2

@yoyo837
Copy link

yoyo837 commented May 11, 2022

getClientRects does not support in ie8-.

@jonirrings
Copy link
Author

jonirrings commented May 11, 2022

@yoyo837 emmm, so let's add an API detection? or just deprecate those before IE11 ?

@jonirrings jonirrings changed the title use getClientRects to replace clientWidth/Height calculation use getClientRects for IE10+ clientWidth/Height calculation May 11, 2022
@yoyo837
Copy link

yoyo837 commented May 11, 2022

This is a polyfill, I think it should be an polyfill for all browsers that which version who need it.

@jonirrings
Copy link
Author

agreed, old code restored.
new code only works when getClientRects available.

@yoyo837
Copy link

yoyo837 commented May 11, 2022

By the way, the maintainer of this repo doesn't seem to be active.

@jonirrings
Copy link
Author

sigh……
waiting for him to reply

@ikonan
Copy link

ikonan commented Jun 14, 2022

@weswigham
Copy link
Contributor

...?

@jonirrings
Copy link
Author

@weswigham as stated above, can you help to review this pr?

@weswigham
Copy link
Contributor

I'm just someone who contributed a PR once in the past, not a maintainer.

@jonirrings
Copy link
Author

@weswigham whatever, thx for your attention.
Waiting for the maintainer ....

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

Successfully merging this pull request may close these issues.

None yet

4 participants