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

Refactor dropdown's hide functionality #33466

Merged
merged 4 commits into from
Apr 18, 2021
Merged

Conversation

rohit2sharma95
Copy link
Collaborator

Share the same code between hide and clearMenus methods.
Ref: #33451 (comment)

@rohit2sharma95 rohit2sharma95 changed the title 🔨 Refactor dropdown's hide functionality Refactor dropdown's hide functionality Mar 24, 2021
@rohit2sharma95 rohit2sharma95 force-pushed the rohit/dropdown-hide-refactoring branch 2 times, most recently from fe7018e to f2ea4ef Compare April 7, 2021 20:22
@rohit2sharma95 rohit2sharma95 force-pushed the rohit/dropdown-hide-refactoring branch from f2ea4ef to b5d02f4 Compare April 14, 2021 05:57
@rohit2sharma95 rohit2sharma95 marked this pull request as ready for review April 14, 2021 05:59
@rohit2sharma95 rohit2sharma95 requested a review from a team as a code owner April 14, 2021 05:59
js/src/dropdown.js Outdated Show resolved Hide resolved
js/src/dropdown.js Outdated Show resolved Hide resolved
js/src/dropdown.js Show resolved Hide resolved
@rohit2sharma95 rohit2sharma95 force-pushed the rohit/dropdown-hide-refactoring branch from b5d02f4 to 625a88a Compare April 14, 2021 09:28
@GeoSot GeoSot self-requested a review April 14, 2021 10:28
@XhmikosR XhmikosR requested a review from alpadev April 15, 2021 04:59
@rohit2sharma95 rohit2sharma95 force-pushed the rohit/dropdown-hide-refactoring branch from 71eb8e9 to b89f45a Compare April 16, 2021 05:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants