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.

Sign up for GitHub

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

Jump to bottom

Make the secondary navigations reactive. #536

Open
1 task done
Dhruvi16 opened this issue Oct 6, 2020 · 5 comments
Open
1 task done

Make the secondary navigations reactive. #536

Dhruvi16 opened this issue Oct 6, 2020 · 5 comments
Assignees
@Dhruvi16
Labels
💻 aspect: code Concerns the software code in the repository 🕹 aspect: interface Concerns end-users' experience with the software 🌟 goal: addition Addition of new feature ✨ goal: improvement Improvement to an existing feature help wanted Open to participation from the community 🟩 priority: low Low priority and doesn't need to be rushed 💪 skill: javascript Requires proficiency in 'JavaScript' 🏁 status: ready for work Ready for work
Projects
Backlog

Comments

@Dhruvi16
Copy link
Member

Dhruvi16 commented Oct 6, 2020

Problem

The guidelines pages have table of contents and that does not show what section is active right now.

Description

Make the table of content and table of progress reactive. This will highlight the active section as the user scrolls. Here is a similar demo.

Implementation

  • I would be interested in implementing this feature.
@Dhruvi16 Dhruvi16 added 🟩 priority: low Low priority and doesn't need to be rushed 🚦 status: awaiting triage Has not been triaged & therefore, not ready for work ✨ goal: improvement Improvement to an existing feature 💻 aspect: code Concerns the software code in the repository labels Oct 6, 2020
@Dhruvi16 Dhruvi16 self-assigned this Oct 6, 2020
@Dhruvi16 Dhruvi16 added 🌟 goal: addition Addition of new feature 💪 skill: javascript Requires proficiency in 'JavaScript' 🕹 aspect: interface Concerns end-users' experience with the software labels Oct 6, 2020
@Dhruvi16 Dhruvi16 mentioned this issue Oct 6, 2020
Closed
7 tasks
@kgodey kgodey added 🏁 status: ready for work Ready for work and removed 🚦 status: awaiting triage Has not been triaged & therefore, not ready for work labels Oct 16, 2020
@kgodey kgodey added this to [TEMPORARY] Deprioritize in Active Sprint Dec 2, 2020
@kgodey kgodey removed this from [TEMPORARY] Deprioritize in Active Sprint Dec 2, 2020
@kgodey kgodey added this to Pending Review in Backlog Dec 2, 2020
@kgodey kgodey added this to [TEMPORARY] Deprioritize in Active Sprint Dec 2, 2020
@kgodey kgodey removed this from [TEMPORARY] Deprioritize in Active Sprint Dec 2, 2020
@kgodey kgodey added this to [TEMPORARY] Deprioritize in Active Sprint Dec 2, 2020
@kgodey kgodey removed this from [TEMPORARY] Deprioritize in Active Sprint Dec 2, 2020
@kgodey kgodey moved this from Pending Review to Open Source Community in Backlog Dec 17, 2020
@kenkirito
Copy link

can i work on this issue
@dhruvkb
Copy link
Member

dhruvkb commented Feb 5, 2021

@kenkirito since there is already @Dhruvi16's PR #537 that addresses this issue, if you could review that PR (by verifying that it works as expected), that would be very helpful.
@kenkirito
Copy link

sure
@Arpitchugh
Copy link

Is the issue still open? @dhruvkb you have a pending change expected in the PR
@dhruvkb
Copy link
Member

dhruvkb commented Mar 8, 2023

The issue might be open, I'm not in touch with the code anymore. Maybe ask in the Slack channel?
@TimidRobot TimidRobot added the help wanted Open to participation from the community label Jul 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💻 aspect: code Concerns the software code in the repository 🕹 aspect: interface Concerns end-users' experience with the software 🌟 goal: addition Addition of new feature ✨ goal: improvement Improvement to an existing feature help wanted Open to participation from the community 🟩 priority: low Low priority and doesn't need to be rushed 💪 skill: javascript Requires proficiency in 'JavaScript' 🏁 status: ready for work Ready for work
7 participants
@kgodey @TimidRobot @dhruvkb @Dhruvi16 @Arpitchugh @kenkirito and others