layer5 icon indicating copy to clipboard operation
layer5 copied to clipboard

Fix : Enhance Image Viewer Usability with Standard Navigation

Open Jeffrin2005 opened this issue 9 months ago • 14 comments

Description

  • Fixes #6378

Merge Checklist

  • [x] Prep screenshot or demo video for changelog entry, and attach it to issue.

https://github.com/user-attachments/assets/3b4fe2c3-1fbc-4c52-a480-4658104b5fe6

  • [x] Completion of QA in Mobile Devices
  • [x] Completion of QA in Desktop Devices

Notes for Reviewers

Signed commits

  • [x] Yes, I signed my commits.

Jeffrin2005 avatar Apr 09 '25 10:04 Jeffrin2005

🚀 Preview for commit 5d7320b2af36b5e3c9a62a13d997339fba411590 at: https://67f6518833bdfdce99cfc21d--layer5.netlify.app

l5io avatar Apr 09 '25 10:04 l5io

@Jeffrin2005 Please don't change the formatting, rest everything looks good

vr-varad avatar Apr 09 '25 11:04 vr-varad

@vr-varad Formating means - the alignment of codes , right ?

Jeffrin2005 avatar Apr 09 '25 11:04 Jeffrin2005

🚀 Preview for commit 12009941c99eb7bbdb6545c815cd6e82470a2d87 at: https://67f66c032056d6149b0ead0a--layer5.netlify.app

l5io avatar Apr 09 '25 12:04 l5io

@vr-varad Fixed DCO errors.

Jeffrin2005 avatar Apr 10 '25 17:04 Jeffrin2005

LGTM @Jeffrin2005 @vishalvivekm @amitamrutiya

vr-varad avatar Apr 10 '25 17:04 vr-varad

🚀 Preview for commit 462b86a1077a8f9cc978be06bf9611d4749c6471 at: https://67f801b931f528330e505f50--layer5.netlify.app

l5io avatar Apr 10 '25 17:04 l5io

@Jeffrin2005 Thank you for your contribution! Let's discuss this during the website call today at 5:30 PM IST

Add it as an agenda item to the meeting minutes, if you would :)

vishalvivekm avatar Apr 14 '25 08:04 vishalvivekm

🚀 Preview for commit 513d6ca6cf4565def838801cac666044e92db762 at: https://67fccb30c95f4abf8ace2400--layer5.netlify.app

l5io avatar Apr 14 '25 08:04 l5io

Screen Shot 2025-04-14 at 16 20 35

vishalvivekm avatar Apr 14 '25 10:04 vishalvivekm

🚀 Preview for commit 3910ba603e157f9bc515d3588465bb4b223122f4 at: https://67ff23f402d715b1102d71fb--layer5.netlify.app

l5io avatar Apr 16 '25 03:04 l5io

🚀 Preview for commit 90bb7217cb201828175f4354e3b8ba7bf8ebfb26 at: https://680133a22c08f712c7a8eb14--layer5.netlify.app

l5io avatar Apr 17 '25 17:04 l5io

@vishalvivekm

https://github.com/user-attachments/assets/55884d65-07f0-4ec6-848e-b64c57ca25fe

Now removed the box (behind the arrows) , and also fixed the responsive issue. Can you review this .

Jeffrin2005 avatar May 01 '25 05:05 Jeffrin2005

@vishalvivekm @leecalcote Can you review this PR?

Jeffrin2005 avatar May 09 '25 11:05 Jeffrin2005

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jun 27 '25 00:06 stale[bot]

🚀 Preview for commit 5818ce05c915744368b0af7c9af392561c204e6f at: https://685fbcdaf9c1d68479d769fe--layer5.netlify.app

l5io avatar Jun 28 '25 10:06 l5io