open-source-practice icon indicating copy to clipboard operation
open-source-practice copied to clipboard

[IDEA] Implement Collapsible Dropdowns Instead of a ToC for Member Names

Open dorf8839 opened this issue 3 years ago • 4 comments

Description

Scrolling through the list of names in the readme can be cumbersome. The implementation of a table of contents is a good first step to help get to name faster but it still leaves the need to scroll up through the entire list of names to get back.

I believe a cleaner UX for this list would be to utilize collapsible sections for each letter. This could also eliminate the need for the Table of Contents, cleaning up the UI a bit more for this README.md file.

Screenshots

image

Additional information

I have left the ToC in the screenshot just to show the comparison but I feel this is unneeded if the collapsible sections change is implemented. Whatever the decision is though, if approved, I would like to have this issue assigned to me. Thank you!

dorf8839 avatar Oct 02 '22 16:10 dorf8839

Welcome @dorf8839! Thank you so much for your first issue! Welcome to the community. Also join us in the EddieHub Discord for realtime chat and calls http://discord.eddiehub.org

github-actions[bot] avatar Oct 02 '22 16:10 github-actions[bot]

Hello @dorf8839 :wave:

It's a good idea to have thought of this markdown feature.

I believe a cleaner UX for this list would be to utilize collapsible sections for each letter. This could also eliminate the need for the Table of Contents, cleaning up the UI a bit more for this README.md file.

I understand that the UX will be better, but I think it's better for the community to keep the display of all names on the README, so we can see other contributors' names in scrolling. 🙂 I like the list displayed and not hidden.

mkubdev avatar Oct 04 '22 07:10 mkubdev

@Pradumnasaraf take a look in into it.

RakeshSangem avatar Oct 07 '22 01:10 RakeshSangem

@Pradumnasaraf take a look in into it.

@mkubdev answered it pretty well, thank you

Pradumnasaraf avatar Oct 07 '22 16:10 Pradumnasaraf

Closing this PR. As the justification is already made

Pradumnasaraf avatar Nov 21 '22 07:11 Pradumnasaraf