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

Dropdown - refine handling of keydown in input and textarea thanks to @pvdlg #24557

Merged
merged 2 commits into from
Oct 27, 2017

Conversation

Johann-S
Copy link
Member

@Johann-S Johann-S commented Oct 26, 2017

Add changes from this PR #21802 and avoid to modify our old unit tests by adding new ones, plus it fix the errors.

Again it's thanks to @pvdlg 👍

Fixes : #21366

@XhmikosR
Copy link
Member

@Johann-S: why didn't you add 2 commits, one with @pvdlg's changes and the second with your fixes. That way the author would be properly attributed.

@Johann-S
Copy link
Member Author

because I was lazy to pull his repository and update his branch but I let him the credits by adding his name on the commit message and on my PR

@XhmikosR
Copy link
Member

Still, he should be the author. Don't get me wrong, but we need to give credit where credit's due.

I can do it if you want.

@Johann-S Johann-S force-pushed the v4-dev-johann-dropdown-keydown-input branch 4 times, most recently from 2cbf004 to 40bafe7 Compare October 27, 2017 07:19
@XhmikosR XhmikosR force-pushed the v4-dev-johann-dropdown-keydown-input branch from 40bafe7 to d2bdfbb Compare October 27, 2017 15:56
@XhmikosR XhmikosR merged commit 41fbec1 into v4-dev Oct 27, 2017
@XhmikosR XhmikosR deleted the v4-dev-johann-dropdown-keydown-input branch October 27, 2017 16:20
@mdo mdo mentioned this pull request Oct 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants