plone / volto

React-based frontend for the Plone Content Management System
https://demo.plone.org/
MIT License
426 stars 575 forks source link

update control panel layout #4794

Closed danalvrz closed 11 months ago

danalvrz commented 11 months ago

Regarding https://github.com/plone/volto/issues/4789

netlify[bot] commented 11 months ago

Deploy Preview for volto canceled.

Name Link
Latest commit cec017c594a05f7739e58d6a71a53333ff3185fd
Latest deploy log https://app.netlify.com/sites/volto/deploys/646bb00d1e09850008a07130
danalvrz commented 11 months ago

@ksuess I need to keep working on it, for the tests to pass, and to implement the checkbox change. I will work on it on Monday, but feel free to make any changes in the meantime!

ksuess commented 11 months ago

@danalvrz Looks good to me, but needs some work to be responsive. Can you fix this? Thanks.

image

ksuess commented 11 months ago

@danalvrz Please rebase your branch, I've made a change in mine https://github.com/plone/volto/tree/relations-control-panel

danalvrz commented 11 months ago

@ksuess yes, I'll fix the responsiveness and rebase

danalvrz commented 11 months ago

@ksuess I removed the sticky property for the labels because it was breaking the table's layout when horizontally scrolling. But I can revert that change. Current state:

Screenshot 2023-05-22 at 12 11 57 2
ksuess commented 11 months ago

That looks much better. I suggest to work with the grid gap to achieve the distance between source options and target options. image

ksuess commented 11 months ago

Well, it is a challenge, but the sticky labels of the targets is in my eyes important to be able to manage the relations.

ksuess commented 11 months ago

For testing purpose, to create a lot of content with a lot of relations, you can create example conten with: image

danalvrz commented 11 months ago

That looks much better. I suggest to work with the grid gap to achieve the distance between source options and target options.

Using the grid gap was the right call!

cypress[bot] commented 11 months ago

Passing run #5244 ↗︎

0 493 20 0 Flakiness 0

Details:

Fix 'autoprefixer: start value has mixed support, consider using flex-start inst...
Project: Volto Commit: cec017c594
Status: Passed Duration: 14:06 💡
Started: May 22, 2023 6:13 PM Ended: May 22, 2023 6:27 PM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.