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

DES-882: Modal now resizes when browser is resized #25

Merged
merged 6 commits into from
Jan 18, 2018
Merged

Conversation

bldunn-dev
Copy link
Contributor

No description provided.

@bldunn-dev bldunn-dev requested a review from JasonLantz January 12, 2018 18:26
@bldunn-dev bldunn-dev assigned mhomolak and unassigned mhomolak Jan 12, 2018
@bldunn-dev bldunn-dev requested a review from mhomolak January 12, 2018 19:44
Copy link
Collaborator

@JasonLantz JasonLantz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just wondering if leading edge should be turned on for debounce.... everything else looks good.

@bldunn-dev
Copy link
Contributor Author

Setting { 'leading': true } did not help in this case of resizing.

@bldunn-dev
Copy link
Contributor Author

Ready for QA or something 😃

@bldunn-dev
Copy link
Contributor Author

@JasonLantz Eajaz discovered that the demo was not working in IE 11. This was due to webpack-dev-server 2.11.x having a dependency on strip-ansi 4.0, which used arrow functions, which are not supported in IE 11. For a quick fix, I pinned the webpack-dev-server at 2.9.x, which uses strip-ansi 3.x. Read all about it here: webpack/webpack-dev-server#1278. Here is a possible PR for webpack-dev-server to resolve this issue: webpack/webpack-dev-server#1273

@eajazali87
Copy link
Contributor

all approved. merging it.

@eajazali87 eajazali87 merged commit 31fa602 into master Jan 18, 2018
@mhomolak mhomolak deleted the DES-882 branch February 5, 2018 20:57
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.

5 participants