Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make each section collapsible to make navigation easier #104

Open
evakdev opened this issue May 31, 2022 · 9 comments
Open

Make each section collapsible to make navigation easier #104

evakdev opened this issue May 31, 2022 · 9 comments

Comments

@evakdev
Copy link
Collaborator

evakdev commented May 31, 2022

I'm adding this as an issue so that there is a discussion.
As of now, the list is quite long and while it is quite organized, it can be hard to navigate. the Index, all main titles, and even some subsections (e.g. basic, advanced for backend) could be made collapsible to facilitate that.
If you believe this is a good idea, I can go make the PR :)

@lnxpy
Copy link
Member

lnxpy commented May 31, 2022

That's so dope Maryam jan. I highly believe that it used to be collapsible but after some PRs, it's been removed for some reason. As it grows, it becomes harder to follow. Totally agree with that. Feel free to add this feature. Much appreciate your love btw. <3 :)

@lnxpy
Copy link
Member

lnxpy commented May 31, 2022

Also, don't forget to add closes #104 text to your commit message so this issue will be close once your PR got merged.

reganto added a commit to reganto/python-engineer-roadmap that referenced this issue Jun 9, 2022
lnxpy added a commit that referenced this issue Jun 9, 2022
Refs #104 -- Added a temporary back to top btn for better navigation
@SepehrRasouli
Copy link
Member

SepehrRasouli commented Jul 25, 2022

Hi @lnxpy, I believe this issue has not been fully solved.
We can make each section collapsible, I believe what @reganto really meant was something like this
Can I work on this ?

@lnxpy
Copy link
Member

lnxpy commented Jul 26, 2022

@bobycloud :)

@SepehrRasouli
Copy link
Member

Any updates @lnxpy ?

@bobycloud
Copy link
Member

Hello, thanks for your suggestion @SepehrRasouli and @evakdev. I'm not sure if it's useful or not because it reduces scrolling but also hides many things from the eyes at first glance. What do you think, @lnxpy and @smoeinbbp?

@smoeinbbp
Copy link
Collaborator

@evakdev maybe it's better to create a PR first and take a look at it and after that we can take a better decision.

@ManiMozaffar
Copy link
Member

#146

@lnxpy
Copy link
Member

lnxpy commented Aug 30, 2024

@bobycloud @smoeinbbp I guess we should close this issue as not planned yet maybe. Adding this capability would cause the README.md file to have a very complex nested HTML-integrated structure which is not a good practice for the new contributors. You make the move.

– Btw, thank you @ManiMozaffar for working on this PR and making your implementation. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants