Skip to content

Commit

Permalink
Page Content: Aligned max-width across viewer and editors
Browse files Browse the repository at this point in the history 
For #4916
  • Loading branch information
@ssddanbrown
ssddanbrown committed Apr 5, 2024
1 parent 19f78db commit b9e2d33
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
5 changes: 3 additions & 2 deletions resources/sass/_forms.scss
View file
Original file line number Diff line number Diff line change
Expand Up @@ -128,8 +128,9 @@
body {
display: block;
background-color: #fff;
padding-inline-start: 16px;
padding-inline-end: 16px;
padding-inline-start: 12px;
padding-inline-end: 12px;
max-width: 864px;
}
[drawio-diagram]:hover {
outline: 2px solid var(--color-primary);
Expand Down
1 change: 1 addition & 0 deletions resources/sass/_tinymce.scss
View file
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@
padding-block-end: 1rem;
outline: 0;
display: block;
max-width: calc(870px);

This comment has been minimized.

Sign in to view

Copy link
@melat0nin

melat0nin Apr 6, 2024

Hi Dan, out of interest, what's the purpose of the calc() here, when the value is static? Looking at the MDN page for calc I can't find any explanation for this (this page on CSS Tricks has an example with a single static value, with a comment alongside: 'A little weird but okay' 😆 https://css-tricks.com/a-complete-guide-to-calc-in-css/)

This comment has been minimized.

Sign in to view

Copy link
@ssddanbrown

ssddanbrown Apr 6, 2024

Author Member

@melat0nin No purpose, I had just left it in when testing out values, hence I removed it in 3e23f45

This comment has been minimized.

Sign in to view

Copy link
@melat0nin

melat0nin via email Apr 6, 2024

}

.wysiwyg-input.mce-content-body {
Expand Down

0 comments on commit b9e2d33

Please sign in to comment.