Skip to content
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

fix: Made canvas responsive on mobile and some ui responsiveness #226

Merged
merged 5 commits into from
Jun 12, 2024

Conversation

addegbenga
Copy link
Contributor

@addegbenga addegbenga commented Jun 8, 2024

This PR addresses the following for mobile experience:

  1. Adjusted the canvas to be fully responsive on mobile devices.
  2. Ensured that the canvas scales properly on mobile.
  3. Made several UI tweaks to improve the overall look of Nft tab on mobile.
  4. Improved the usePreventZoom hook

To test, you can run the project on a real mobile device to see the changes in action.

VIDEO-2024-06-08-14-30-51.mov

Copy link

vercel bot commented Jun 8, 2024

@addegbenga is attempting to deploy a commit to the keep-starknet-strange Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Contributor

@b-j-roberts b-j-roberts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very cool!

frontend/src/tabs/ExpandableTab.js Outdated Show resolved Hide resolved
frontend/src/tabs/nfts/NFTs.css Outdated Show resolved Hide resolved
frontend/src/App.js Outdated Show resolved Hide resolved
frontend/src/canvas/CanvasContainer.js Outdated Show resolved Hide resolved
frontend/src/utils/Window.js Outdated Show resolved Hide resolved
@b-j-roberts b-j-roberts merged commit c8b4f8b into keep-starknet-strange:main Jun 12, 2024
3 of 4 checks passed
@b-j-roberts
Copy link
Contributor

Thank you! Cool to be able to explore the canvas on mobile now!

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