Skip to content
This repository has been archived by the owner on Oct 20, 2022. It is now read-only.

fix(PopupMenu): add prop popupMenuDropdownClassName #495

Merged
merged 2 commits into from
Jan 14, 2019
Merged

fix(PopupMenu): add prop popupMenuDropdownClassName #495

merged 2 commits into from
Jan 14, 2019

Conversation

Unic0arn
Copy link
Contributor

Allow passing className down to dropdown menu.

This allows specifying z-index in consumer.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.005%) to 83.816% when pulling 3cbdf15 on add-className-to-popupMenuDropdown into 7b27066 on master.

@coveralls
Copy link

coveralls commented Jan 11, 2019 •
edited
Loading

Coverage Status

Coverage increased (+0.005%) to 83.816% when pulling 3cbdf15 on add-className-to-popupMenuDropdown into 7b27066 on master.

@Unic0arn Unic0arn merged commit cf6b05c into master Jan 14, 2019
Unic0arn added a commit that referenced this pull request Jan 14, 2019
@Unic0arn
Revert "fix(PopupMenu): add prop popupMenuDropdownClassName (#495)"
c7bfaa0 
This reverts commit cf6b05c.
@Unic0arn Unic0arn mentioned this pull request Jan 14, 2019
Merged
Unic0arn added a commit that referenced this pull request Jan 14, 2019
@Unic0arn
Revert "fix(PopupMenu): add prop popupMenuDropdownClassName (#495)"
10a7b69 
This reverts commit cf6b05c.
@iamstarkov iamstarkov deleted the add-className-to-popupMenuDropdown branch February 4, 2019 11:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@KitsuneKelso KitsuneKelso KitsuneKelso approved these changes

@jesperwiner jesperwiner jesperwiner approved these changes

@Dhq Dhq Dhq approved these changes

@mjmonline mjmonline Awaiting requested review from mjmonline

@robineng robineng Awaiting requested review from robineng

@JacobBlomgren JacobBlomgren Awaiting requested review from JacobBlomgren

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Unic0arn @coveralls @Dhq @jesperwiner @KitsuneKelso