You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Focus the field for Body and click the plus icon. Note the "INSERT BLOCK" label followed by none of the content blocks (see screen below).
Expected behaviour
Clicking the plus icon in the Body field shows "INSERT BLOCK" label followed by the content blocks we specify for the field.
Screenshots
System information
OS: macOS 10.15.7
Browser: FF or Chrome
node v14.4.0 (npm v7.5.3)
Additional context
Just poking around the Keystone 5 repository, I decided to run the Todo example directly, and replicated the issue there.
In the file ./packages/fields-content/src/views/editor/add-block.js, this assignment does not destructure as intended (unable to diagnose why) and is always undefined: let { Sidebar } = blocks[key]
Also, the UI completely breaks if you add an image and set it to be aligned left or right. No error message appears if you upload an image which is too large. Will confuse users if this happens. Makes Keystone a bit unusable.
Bug report
Describe the bug
The Content field renders no content blocks in the admin app editor.
To Reproduce
yarn create keystone-app content-field-test
. I used the Todo starter.yarn add @keystonejs/fields-content
.yarn dev
.Expected behaviour
Clicking the plus icon in the Body field shows "INSERT BLOCK" label followed by the content blocks we specify for the field.
Screenshots
System information
Additional context
Just poking around the Keystone 5 repository, I decided to run the Todo example directly, and replicated the issue there.
In the file ./packages/fields-content/src/views/editor/add-block.js, this assignment does not destructure as intended (unable to diagnose why) and is always undefined:
let { Sidebar } = blocks[key]
This is merely a very quick and dirty workaround.
Doing this allows the content blocks/toolbar to show, though styling is rather broken and I am sure this would result in downstream issues.
The text was updated successfully, but these errors were encountered: