Conversation
|
Please rebase your PR branch onto the latest develop (do not merge develop into your feature branch). Try: Resolve conflicts and run local checks (npm run lint, npm run build) Notes: |
cfbb324 to
0e77160
Compare
|
Done |
SyedFahad7
left a comment
There was a problem hiding this comment.
@GauravKarakoti
Add try/catch around localStorage, so JSON parsing to handle invalid/corrupted stored data safely.
Prefer using tool url as the favorite identifier instead of title to avoid key collisions.
Add an aria-label on the star toggle button for accessibility.
|
Done |
|
Thanks @GauravKarakoti , the requested fixes are implemented: localStorage parse safety, URL-based favorite keying, and aria-label on the star toggle. One process issue remains: I suggest to raise a fresh PR so its easy to re-review and safely merge to develop! Closing this PR for now :) |
Description
Added a favourite tool feature.
Favourite tools are pushed to top.
Added a filter for favourite tools.
Fixes #61
Screenshots of relevant screens
Screen.Recording.2026-03-03.112937.mp4
Developer's checklist
If changes are made in the code: