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] Floating toolbar #1409

Closed
wants to merge 5 commits into from
Closed

[FIX] Floating toolbar #1409

wants to merge 5 commits into from

Conversation

dratwas
Copy link
Contributor

@dratwas dratwas commented Oct 4, 2019

Fixes #
This is a phase 2 of WordPress/gutenberg#17399
In this PR I fixed the floating toolbar for the first element in a group to be rendered above the block and be clickable.

Gutenberg PR - WordPress/gutenberg#17769

To test:
Check if each of floating toolbar in a group block is clickable and rendered in the correct place
Update release notes:

  • If there are user facing changes, I have added an item to RELEASE-NOTES.txt.

src/_native.android.scss Outdated Show resolved Hide resolved
@pinarol pinarol requested review from koke and mkevins October 4, 2019 13:22
@pinarol pinarol added the [Type] Bug Something isn't working label Oct 4, 2019
@lukewalczak
Copy link
Contributor

Closing for now. It will be open when work over the fix resumes.

@dratwas
Copy link
Contributor Author

dratwas commented Feb 14, 2020

Closing for now since it is not working in all cases

@dratwas dratwas closed this Feb 14, 2020
@SergioEstevao SergioEstevao deleted the fix/floating-toolbar branch April 17, 2020 09:30
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