-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Block Editor List #8273
Block Editor List #8273
Conversation
This reverts commit 5b910c1.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have looked at the last set of changes you have done @nielslyngsoe in your commits and they all look good.
@Shazwazza I had a few minor things mentioned - let me know if you want to deal with them and if we now want to say this is done and merge this into v8/dev and then start making smaller granular PRs?
…ditors. this documentation will be transfered to Our documentation.
Hi! Don't know if it's okay for me to report a bug already, but here it goes 😇
The error is thrown here: https://github.com/umbraco/Umbraco-CMS/blob/v8/feature/block-editor-list/src/Umbraco.Web.UI.Client/src/views/propertyeditors/blocklist/prevalue/blocklist.blockconfiguration.controller.js#L102 |
Removal of NestedContent to BlockList migration - due for community project
@stefankip thanks for being an early tester in this chaos. @nielslyngsoe is looking into this now and will be in a new PR :) |
@warrenbuckley just checking out all the new block editors on the block (🤣) and noticed this. |
@stefankip thanks for finding this and feel free to report anything. |
Hi, I was playing around with the block editor and found a small issue. If you have a settings model added to a block, then add the block to the page. If you then click on the block settings and when the popup opens click on the content tab, then close the popup and click on the block settings again, you will see that it opens the popup with both tabs enabled. And I can't add a thumbnail when the image extension is in caps (.PNG). |
WIP PR for the block editor so we can start the review process.
Test notes:
— Block List Editor, Every possible thing you can think about, it's new so haven't been tested before.