@pikapikamart
Posted
Hey, great work on this one. The layout is really good and it resizes well when going in mobile state.
But a problem is that, at first the accordion is working. But after a while I get the same error as pointed by @mbart13 and your accordion does not work anymore and prompting errors.
But when I click the arrow img, the accordion works, but clicking the question the error is prompted.
Tried reloading a bunch, then your accordion now works. That seems really strange.
Okay, so it works now. But clicking multiple accordion resizes the whole container. Though it is fine but for me, I want to prevent that. It will be awesome if you could cap the container of it.
So to do that, you could add a max-height
and overflow-y: scroll
. This way, even if a user clicks multiple accordion, it will not resize the container itself because the article
will prevent the element overlapping the max-height and you can just scroll on it. This for the desktop state.
But by doing so, there will be a scrollbar right. You could get rid of that by simple adding
.example::-webkit-scrollbar {
display: none;
}
/* Hide scrollbar for IE, Edge and Firefox */
.example {
-ms-overflow-style: none; /* IE and Edge */
scrollbar-width: none; /* Firefox */
}
this, just change the selector. This will remove the scrollbar but keeps its functionality
Overall, your work is really good^^
@maciekw129
Posted
@pikamart Hey, Thank you very much for all of the advices. I have added max-height and scroll, and it really works better ;)