web-app icon indicating copy to clipboard operation
web-app copied to clipboard

fix(ui): hide "Ways to help" section under a spoiler

Open embarq opened this issue 3 years ago • 3 comments

Closes https://github.com/selfdefined/web-app/issues/325

embarq avatar Apr 28 '21 02:04 embarq

It was made to hide the "Ways to help" content only on mobile. I found that for the parent section 800px breakpoint is used, so I took the same value into consideration.

Currently, the script will collapse the section on <800px screens, for desktop it will look as it was before.

However, the spoiler behaviour won't change on resize. For example if you open the homepage on mobile-like screen, and then resize it to let's say 810px - the spoiler will remain working(perhaps it should not). If you would like to have it adjusting to resize, let me know.

embarq avatar Apr 28 '21 02:04 embarq

@ovlb Man, I can't express how grateful I am. Thanks for so many useful and interesting suggestions! I tried my best in an attempt to make the implementation better, would be awesome if you could check the result once again.

P.S. Thanks for being so helpful 👍 As you might notice, I have quite a limited knowledge of a11y and how to implement it in the right way. Looking forward to learning and practice more tho

embarq avatar May 04 '21 02:05 embarq

@embarq Thanks for your updates. I had quite a packed week, so please bear with me. I want to review it with the time it deserves. Aiming for Monday. Hope you have a nice weekend.

O*

ovlb avatar May 07 '21 20:05 ovlb