p5.js-web-editor icon indicating copy to clipboard operation
p5.js-web-editor copied to clipboard

[#3691] Add mobile visibility toggle to sketches dropdown

Open NisargJasani0602 opened this issue 2 months ago • 2 comments

Fixes #3691

Changes: Added a mobile-only visibility toggle in the /sketches dropdown menu, allowing users to switch between public and private sketches on smaller screens. Updated SketchListRowBase.jsx to reuse the existing changeVisibility logic for consistency. Verified responsive behavior and correct state updates across desktop and mobile views.

I have verified that this pull request:

  • [x] has no linting errors (npm run lint)
  • [x] has no test errors (npm run test)
  • [x] is from a uniquely-named feature branch and is up to date with the develop branch.
  • [x] is descriptively named and links to an issue number, i.e. Fixes #3691
  • [x] meets the standards outlined in the accessibility guidelines

NisargJasani0602 avatar Oct 23 '25 01:10 NisargJasani0602

Before the Changes: There is no dropdown menu option for toggling the visibility of the saved sketches if you are using the P5.js website in Mobile view Screenshot 2025-11-03 at 8 47 25 PM

After the Changes: There is a new dropdown which allows the User to toggle the visibility (Public/Private) and is also working synchronously with the Desktop view Screenshot 2025-11-03 at 8 44 47 PM Screenshot 2025-11-03 at 8 45 47 PM

NisargJasani0602 avatar Nov 04 '25 01:11 NisargJasani0602

LGTM thank you for the preview.

yugalkaushik avatar Nov 04 '25 16:11 yugalkaushik