Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Add sections to settings #849
base: main
Are you sure you want to change the base?
Add sections to settings #849
Changes from 5 commits
53acfda
8c61374
134abb1
064fa6a
3291c02
f539b03
ba36682
65dd74f
bacbfcb
6538da5
3f5d798
027653d
014d68f
76366e4
dc1df4f
7484e0c
47d844c
a939872
6d524bd
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
Large diffs are not rendered by default.
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.
This does not work for screens sizes under
960px
. At the moment we usedisplay: block !important
here:woocommerce-pdf-invoices-packing-slips/assets/css/settings-styles.css
Line 997 in 53cae65
So the section is never hidden when clicking on the section title. Also the slide animation is not working when the
.form-table
element is set todisplay: table;
. Which we do when viewing the settings full screen.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.
@Terminator-Barbapapa For the full-screen mode, where the
display: table;
, I think there is no easy way to fix it.One way would be to make the
display
propertyblock
, then make the transition, and then revert it totable
, which has other side effects and issues, such as the bad display of elements during the transition.Another option would be to switch to height transition instead of toggling the display.
Another option would be wrapping the
form-table
around a DIV element and working with that, but this is not a good way as it requires many modifications.I tried to fix it using the first option, but several issues appeared that made me think this was not a good approach.
If we're going to fix this issue for full-screen mode, then switching to height transition might be the better option here.
Would you happen to have any suggestions or opinions?
Large diffs are not rendered by default.