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

Image full alignment size issue on edit#5936 #6166

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

Conversation

cihanandac
Copy link

Fix for issue #5936

This change fixes the full aligned images size problem on the edit screen by changing the padding-left and width values. Transitions are set so that opening/closing of sidebar and toolbar would look seamless.

This change fixes the styling of the full aligned images size problem by
changing padding-left and width values depending on the sidebar and toolbar
position
Copy link

netlify bot commented Jul 13, 2024

Deploy Preview for plone-components canceled.

Name Link
🔨 Latest commit 6d8352c
🔍 Latest deploy log https://app.netlify.com/sites/plone-components/deploys/669ba4252df4af00087d0d91

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.

@plone/volto-team this needs review.

packages/volto/news/5936.bugfix Outdated Show resolved Hide resolved
packages/volto/news/5936.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.

I approve the news item only. This still needs review from a @plone/volto-team member.

.view-editview {
&.has-sidebar {
.block.align.full img {
width: calc(100vw - 375px) !important;
Copy link
Member

Choose a reason for hiding this comment

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

@cihanandac We can't do this. If you change the width in edit view. How will a user know its applied a full width, which means the viewport width? Ideally in Volto we wish to keep Edit and View display components the same. For eg:

<ListingBody {...props} path={getBaseUrl(pathname)} isEditMode />

Copy link
Member

@nileshgulia1 nileshgulia1 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 comments.

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.

3 participants