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:Order blocks in sidebar doesn't respect layout restrictions: requ… #6489

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

alihamza1221
Copy link

@alihamza1221 alihamza1221 commented Nov 18, 2024


fixed : Order blocks in sidebar doesn't respect layout restrictions: required and fixed position

I added the checks to display the delete block and drag block button according to values set like fixed and required.
After my change:

order.block.fixes.video.mp4

Closes #6481

Copy link

netlify bot commented Nov 18, 2024

Deploy Preview for plone-components canceled.

Name Link
🔨 Latest commit 27f4370
🔍 Latest deploy log https://app.netlify.com/sites/plone-components/deploys/6741742e231c4d00089d5323

@alihamza1221 alihamza1221 marked this pull request as ready for review November 22, 2024 06:46
@alihamza1221
Copy link
Author

@plone/volto-team I have made the appropriate changes to fix the issue and It works as wanted please If you can review this PR.

Copy link
Collaborator

@stevepiercy stevepiercy left a comment

Choose a reason for hiding this comment

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

Please see my note about the change log entry.

Otherwise, this looks really good, but it should get a thorough technical review by the @plone/volto-team. Thank you!

packages/volto/news/6481.bugfix Outdated Show resolved Hide resolved
Copy link
Collaborator

@stevepiercy stevepiercy left a comment

Choose a reason for hiding this comment

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

News looks good, still needs a technical review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Order blocks in sidebar doesn't respect layout restrictions: required and fixed position
2 participants