congo icon indicating copy to clipboard operation
congo copied to clipboard

feat: add js to check height and decide if to top is required

Open Jh123x opened this issue 1 year ago • 4 comments

Set scroll to top element to available when page height -150 > browser height. -150 is based on testing

Closes #772

Jh123x avatar Feb 06 '24 15:02 Jh123x

Deploy Preview for hugo-congo ready!

Built without sensitive environment variables

Name Link
Latest commit fce82b45b2a4fa19d424e5959b95b7b184f5dca1
Latest deploy log https://app.netlify.com/sites/hugo-congo/deploys/663a4c96941a350007d966c4
Deploy Preview https://deploy-preview-806--hugo-congo.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] avatar Feb 06 '24 15:02 netlify[bot]

I'm not against making this a JS feature as frankly the current implementation using only CSS is a bit hacky, but I still think it should honour the .Site.Params.footer.showScrollToTop parameter.

jpanther avatar Apr 30 '24 04:04 jpanther

I'm not against making this a JS feature as frankly the current implementation using only CSS is a bit hacky, but I still think it should honour the .Site.Params.footer.showScrollToTop parameter.

Hmm, let me try it in another way

Jh123x avatar Apr 30 '24 14:04 Jh123x

It should also obey the rule in the config now

Jh123x avatar Apr 30 '24 15:04 Jh123x