-
-
Notifications
You must be signed in to change notification settings - Fork 642
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
base: main
Are you sure you want to change the base?
Image full alignment size issue on edit#5936 #6166
Conversation
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
✅ Deploy Preview for plone-components canceled.
|
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.
@plone/volto-team this needs review.
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 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; |
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.
@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 /> |
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.
Please see my comments.
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.