Closed suryahanumandla closed 3 years ago
Thank you for your contribution @suryahanumandla. I really like suggestion 1 (multiple containers).
Can you update your PR #68 to incorporate those changes?
+1 for suggestion 1, it looks great.
@suryahanumandla it will be great if this can be added today.
Sure @inovizz @bhansa
@bhansa for time being I've updated all community partner assets with same pixels and background.
Okay @suryahanumandla, Is the preview looking good for community partners?
Once done, please resolve the conflicts and we will be good for merge.
@bhansa will be fixing it in few mins
latest community partners looks like
@bhansa @inovizz please review it and let me know if any changes needed.
Apparently, I merged the PR and then realized that the mobile view had CSS issues. So had to revert the merge.
Could you please fix the issues in mobile and re-open the PR.
Sorry for the trouble.
@suryahanumandla
@inovizz I haven't verified mobile view. sure will look into it.
A quick fix to make speakers and community partners alignment to horizontal, only flaw is it adds horizontal scroll
Suggestion 1:
I feel scroll is not a good idea, we can make multiple containers and show the speakers and community partners in multiple sections. It would look like this
Suggestion 2:
We can make use of any carousel library