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 added this to Inbox in v5.0.0 via automation Mar 24, 2021
@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
@GeoSot GeoSot moved this from Inbox to Review in v5.0.0 Apr 14, 2021
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
v5.0.0 automation moved this from Review to Approved Apr 14, 2021
@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
@XhmikosR XhmikosR merged commit 7eadf73 into main Apr 18, 2021
v5.0.0 automation moved this from Approved to Done Apr 18, 2021
@XhmikosR XhmikosR deleted the rohit/dropdown-hide-refactoring branch April 18, 2021 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
v5.0.0
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants