[BUG] hiding header or footer does not extend workspace view appropriately

[Byolock]

Environment

Self-Hosted (Docker)

Version

2.0.3

Describe the problem

Hiding the footer or header does not lead to the iframe of the selected app extending into the free space, instead the background of the selected theme is visible.

Steps to reproduce

1. add

   "hideComponents": {
      "hideSearch": true,
      "hideHeading": true,
      "hideNav": true,
      "hideSettings": true,
      "hideFooter": true
    },

to your configuration
2. apply configuration
3. switch to workspace view
4. select any app

Tested with Firefox and Brave Browser.
Probably also relates is the fact that hiding header and not footer leads to the footer not beeing at the bottom of the site.
I attached screenshots for both problems.

Additional info

[Background Image from the Adventure theme can be seen below the iframe]

[Footer is floating if hiding header]

Please tick the boxes

• You are using a supported version of Dashy (check the first two digits of the version number)
• You've checked that this issue hasn't already been raised
• You've checked the docs and troubleshooting guide
• You agree to the code of conduct

[Lissy93]

Heya @Byolock
Thanks for the bug repot, and apologies for the delay in implementing this.
It should now be fixed in #557 / 2.0.6
Feel free to reach out if you need anything else :)

If you are using a custom theme, and experience a similar issue in the future, then you can adjust the --footer-height CSS variable

[Byolock]

@Lissy93 No worries for the delay. Thanks for the change, however it seems to not fully fix my issue. The footer is now at the bottom but the iframe does still not extend all the way to the bottom of the window :

I tried to change --foother-height that didn't do anything, but im also not sure which value would be right. Tried to change from 128px to 64px and then to 256px and didn't see any visual difference.

[liss-bot]

This issue has gone 6 weeks without an update. To keep the ticket open, please indicate that it is still relevant in a comment below. Otherwise it will be closed in 5 working days.

[Lissy93]

/keep-open

This issue isn't resolved. I think I need to approach it differently to get the solution

[Byolock]

@Lissy93 Maybe this is something that raises some new ideas ?

I finally had some time trying different css changes to see if I would get the the iframe to fit properly. I set the height of the iframe to calc(100% - var(--header-height) + var(--footer-height)). That change made the iframe fit nearly correctly (it is a few pixels to high but barely noticable). header-height value is 6.3rem footer-height value is 4rem. My guess is this is nearly working by pure chance or does that make any sense to you?

Using calc(100% - var(--footer-height)) uses more space than the default calc(100% - var(--header-height) but still misses a fair bit of space.

I did wonder, I have "hide header" enabled in the configuration, so why is the header height necessary to determine the height of the IFrame in the first place?

Some further experimenting got me to the idea that calc(100% - var(--footer-height)) is probably correct and the missing space is because I can still see a small header even though I disabled it in the configuration. The header class has a padding property of 0.5rem. A look on the footer class revealed a padding of 0.25rem. That got me to 0.75rem, so I changed the Iframe height to : calc(100% - var(--footer-height) + 0.75rem). That wasn't correct either but it was obvious this change used up exactly half of the remaining space which should be used by the iFrame, so I changed it to calc(100% - var(--footer-height) + 1.5rem).
This change made the iFrame now exactly as high as it should be. The interresting thing is, 1.5rem is exactly the margin-top property of the footer class. So maybe the correct solution to this problem is to take '100% height minus the height of the footer without its margin-top property' for the height of the IFrame?

My understanding of CSS and the terminology is very limeted so I hope what i've written makes any sense.

[CrazyWolf13]

#1463 Seems related

[liss-bot]

The fix for this issue has now been released in 2.1.2 ✨

If you haven't done so already, please update your instance to 2.1.2 or later. See 2.1.2 for full info.

Feel free to reach out if you need any more support. If you are enjoying Dashy, consider supporting the project.