The slight horizontal scrolling whilst on mobile is pretty annoying.
@bender@twtxt.net Hmmm this should not be happening… @darch@neotxt.dk can you look into this? I did notice this too at first, but then it went away 🤷♂️
Hi @bender@twtxt.net and thanks for letting us know. I have never noticed this, but I see it now too. What device and browser are you using?
@darch@neotxt.dk For me I noticed it on iOS on Mobile Safari
@darch@neotxt.dk Chrome/Android for me, the footer is pushing the rest.
@justamoment@twtxt.net thanks for that. I hope that will be easy to fix
@darch@neotxt.dk There seems to be a negative margin on the right section of the footer on mobile.
@justamoment@twtxt.net yup I just had a look at it: https://git.mills.io/yarnsocial/yarn/src/branch/footer_overflow
not sure if I fixed the issue yet or just created more problems…
@darch@neotxt.dk dunno 🤷 there’s too many raw selectors I can’t figure out how they run over each other.
Why there’s a negative margin in a flexbox child anyway?
@bender@twtxt.net try to visit my pod http://neotxt.dk and see if it’s better there now
@bender@twtxt.net great to hear. And yeah I know I need to fix the top banner on my server 😉
@darch@neotxt.dk it looks great centered like that on mobile! 😎👍
@justamoment@twtxt.net thanks I will make a PR over the weekend then
Nice work y’all 🤗
@bender@twtxt.net how much does it move and what phone and browser are you using?
@darch@neotxt.dk on my phone about 3 words (from the “Abuse” link in the footer), I quickly checked and it seems to be done negative margin in the footer, I might be wrong though. I only checked with Eruda.
@justamoment@twtxt.net can you post a screenshot?
I think i nailed it now: https://git.mills.io/yarnsocial/yarn/src/branch/body_overflow_bug
@darch@neotxt.dk Glad you already fixed, here the screenshots anyway.
Before scrolling:
After scrolling:
@justamoment@twtxt.net and @bender@twtxt.net try to visit my pod https://neotxt.dk where I’m now running the overflow_bug branch and let me know if the issue is fixed when using your phones, so I can make a PR.
Added horizontal scroll for the filters now too:
There is a PR waiting to be merge now fixing the overflow causing horizontal scroll on mobile
It includes the horizontal scroll for filters on mobile, but it commented out in the CSS for the time being.
@darch@neotxt.dk Thanks! 🙏 @justamoment@twtxt.net or @markwylde@twtxt.net Do you guys mind reviewing this? 🙏
@prologic@twtxt.net On It! 🕵️