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

fix(localized-rich-text-input): dropdown fix #1197

Merged
merged 2 commits into from
Nov 25, 2019

Conversation

montezume
Copy link
Contributor

Summary

Fixes #1195

Copy link
Contributor

@jonnybel jonnybel left a comment

Choose a reason for hiding this comment

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

Nice

@montezume montezume force-pushed the ml-localized-rich-text-z-index branch from 7e374bb to bfb1791 Compare November 25, 2019 10:27
@@ -76,5 +83,31 @@ describe('LocalizedRichTextInput', () => {

// check that there is now a strong tag in the document.
numOfTags = await getNumberOfTags('strong');

Copy link
Contributor Author

Choose a reason for hiding this comment

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

added Percy test so we can stop this regression in future

Copy link
Contributor

@tdeekens tdeekens left a comment

Choose a reason for hiding this comment

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

I trust in that test.

@montezume montezume self-assigned this Nov 25, 2019
@kodiakhq kodiakhq bot merged commit ab54b57 into master Nov 25, 2019
@kodiakhq kodiakhq bot deleted the ml-localized-rich-text-z-index branch November 25, 2019 10:39
@montezume
Copy link
Contributor Author

I trust in your trust of that test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Type: Bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

LocalizedRichTextInput: Dropdown z-index problem
3 participants