Skip to content

Improve QR rendering quality on mobile devices#359

Open
kavin553 wants to merge 2 commits into
Dev-Card:mainfrom
kavin553:fix-qr-mobile-quality
Open

Improve QR rendering quality on mobile devices#359
kavin553 wants to merge 2 commits into
Dev-Card:mainfrom
kavin553:fix-qr-mobile-quality

Conversation

@kavin553

Copy link
Copy Markdown

Summary

Improved QR rendering quality on mobile devices by increasing QR code resolution for better visibility and scan reliability.

Closes #275


Type of Change

  • Bug fix
  • UI / Design change

What Changed

  • Increased QR code render size from 200 to 512 in the mobile app.
  • Improved QR sharpness and readability on mobile screens.
  • Enhanced scan reliability for shared DevCard QR codes.

How to Test

  1. Run the mobile/web app locally using "pnpm dev"
  2. Open the Scan DevCard screen.
  3. Verify that the QR code appears sharper and scans properly on mobile devices.

Checklist

  • My code follows the project's coding style ("pnpm -r run lint" passes).
  • TypeScript compiles without errors ("pnpm -r run typecheck").
  • I have added or updated tests for the changes I made.
  • All tests pass locally ("pnpm -r run test").
  • No new "console.log" or debug statements left in the code.

Screenshots / Recordings

Added improved high-resolution QR rendering for mobile devices.


Additional Context

This change improves QR clarity and scan performance, especially on smaller mobile screens and compressed image previews.

@kavin553

Copy link
Copy Markdown
Author

Hi @maintainers 👋

I’ve implemented improvements for QR rendering quality on mobile devices by increasing the QR resolution for better visibility and scan reliability.

Working on submitting the PR for Issue #275 🚀

@Harxhit Harxhit added the gssoc:approved Required label for every approved PR. Gives the base +50 points and enables contribution tracking. label May 27, 2026
@ShantKhatri ShantKhatri requested a review from blankirigaya June 6, 2026 17:13
@blankirigaya

Copy link
Copy Markdown
Collaborator

@kavin553 hey we have changed our package manager from pnpm to npm so can you do the same for this pr?

@vercel

vercel Bot commented Jun 15, 2026

Copy link
Copy Markdown

@kavin553 is attempting to deploy a commit to the Prashantkumar Khatri's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

Copy link
Copy Markdown

CI — All Checks Passed

Backend — SKIP

Check Result
Lint -
Test -
Typecheck -

Mobile — SKIP

Check Result
Lint -
Test -

Web — SKIP

Check Result
Check -
Build -

Last updated: Mon, 15 Jun 2026 09:35:20 GMT

@kavin553

Copy link
Copy Markdown
Author

Hi @blankirigaya 👋

Updated this PR to align with the repository’s npm migration.

Changes made:

  • removed pnpm-lock.yaml
  • regenerated dependencies using npm
  • synced branch with latest main updates
  • revalidated QR rendering improvements on mobile devices

The QR rendering quality enhancement remains intact and fully functional.

Thank you 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gssoc:approved Required label for every approved PR. Gives the base +50 points and enables contribution tracking.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve QR download preview quality on mobile devices

3 participants