vartheme is providing a fix that applies a top padding to body via javascript to ensure that the toolbar is not overflowing content.
This fix is provided with "global-scripts" library (js/scripts.js) in vartheme. When creating a subtheme there is an own global-scripts library created in the subtheme but the script used in the subtheme does not provide the fix for the toolbar.
I think the script.js in subtheme should apply this fix too.
There is also something done about IPE tray padding in this script, that should probably put into the subtheme too.
I am providing patch that just copies these fixes to subtheme template too.
But maybe it would make sense to put this fixes out to separate library and inherit this from base theme? Since the global-scripts library is indicating to be a place for customizations?
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | vartheme_subtheme_does_not_use_toolbar_top_space_fix-3033781-002.patch | 910 bytes | stefan.korn |
Comments
Comment #2
stefan.kornThis is the patch
Comment #3
rajab natshahThank you Stefan for reporting and the patch.
You are right on the issue.
We could apply your patch
But I have noticed that
vartheme/global-scripts: falseis off
I will comment it only so that we will load the Vartheme global scripts in the sub theme.
Comment #4
rajab natshahComment #5
rajab natshahComment #6
rajab natshahComment #8
rajab natshahComment #9
rajab natshahComment #10
rajab natshah