Skip to content

Style side nav bar, add icons, persist active item state #91

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Jospin6
Copy link

@Jospin6 Jospin6 commented Mar 20, 2025

  • Styled the sidebar navigation for the dashboard.
  • Added icons using font_awesome_rails.
  • Implemented sidebar_controller.js to persist the active item state.

Changes:

  • Improved sidebar UI/UX.
  • Integrated FontAwesome for better iconography.
  • Ensured active item state persists across page reloads.

@Tresor11
Copy link
Member

Thank you for the PR @Jospin6
Currently we are using tailwind for styling so i think it would be better to use https://heroicons.com/ which was made by the same makers

@Jospin6
Copy link
Author

Jospin6 commented Mar 20, 2025

Thank you for the feedback! 🙌 That makes sense, I’ll update the PR to use Heroicons for consistency with Tailwind. I’ll push the changes shortly

@Tresor11
Copy link
Member

@Jospin6 After looking at the change you made, I think you might be updating the wrong navigation.
I'll add more details to this issue #86 to better define what you need to work on

@Jospin6
Copy link
Author

Jospin6 commented Mar 20, 2025

Great, but I've made changes in the dashboard section of the application; is there any other part I should work on apart from the dashboard? Give me the feedback, and I'll get to work on it right away

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

Successfully merging this pull request may close these issues.

2 participants