-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
File Block: file name needs to use PlainText component #10967
Comments
Noting that all the keyboard shortcuts for text formatting are also working here. Maybe this shouldn't be a /cc: @iseulde |
Why can't we just use the |
@ocean90, I have created a PR for solution of above issue. Kindly review it. |
Worth noting that |
(From what I understand, |
WordPress 5.7 beta 3 and Gutenberg plugin 10. Added a file block. Removed the text link. Is the issue still valid based on what was reported? |
^ Working on it! |
Thank you @aurooba |
Should the file name, button, and the file become inner blocks? I will also bring in this somewhat associated issue (which also could use a closer look): |
@paaljoachim I'm going to try two things today: innerBlocks is a good idea, you're right, why can't we make the block more flexible? Also I'm not sure the PlainText component is correct, I think RichText is still the right way to go with the link functionality exposed so it can be changed/edited. I have to test a bit more, because obviously we don't want the initial issue @ocean90 reported to be a thing. #15097 definitely feels related. I'm going to tackle this issue first, and then I can start some explorations around the PDF issue as well. :) |
That is great! Thank you @aurooba |
I am no longer able to reproduce the issue. |
Closing this as can't reproduce with gutenberg 18.5.0 |
Describe the bug
formattingControls
ofRichText
should be honored in pasting within the block.`To Reproduce
Expected behavior
No formatting should be applied if
formattingControls
is empty.Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
The text was updated successfully, but these errors were encountered: