-
Notifications
You must be signed in to change notification settings - Fork 32
Feature/comprehensive instructions menu #73
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
Feature/comprehensive instructions menu #73
Conversation
- Replace simple shortcuts overlay with comprehensive menu - Add game instructions and objectives section - Integrate keyboard shortcuts documentation - Move theme selection into the menu - Add audio controls to the menu - Implement theme-aware styling - Add responsive design for mobile devices - Add instructions button () to main UI - Support H key to toggle menu - Support M key for mute/unmute - Update touch event handlers for new menu elements Closes #[issue-number]
|
@adamstosho is attempting to deploy a commit to the Maitri Vaghasiya's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Thanks. Please review and then merge the PR |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@adamstosho I did not find the changes on preview. Your changes just removed the theme changing events and added nothing at all. Can you please check it thoroughly from your end first and then raise a review request. |
|
Oops! I accidentally removed the theme buttons. Fixed now - they're back + new menu added. Can you check the preview again? |
|
@adamstosho I cant find the changes, did you locally test it before raising commit request? Share it with Complete PR template with working changes, do not make review request without it, makes no sense. |
|
lol. makes no sense? okay nha
…On Thu, Oct 30, 2025 at 7:46 AM maitri-vv ***@***.***> wrote:
*maitri-vv* left a comment (maitri-vv/3D-Blockstack#73)
<#73 (comment)>
@adamstosho <https://github.com/adamstosho> I cant find the changes, did
you locally test it before raising commit request? Share it with Complete
PR template with working changes, do not make review request without it,
makes no sense.
—
Reply to this email directly, view it on GitHub
<#73 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BOGFZ76KCGSO7VCYHF2QQNT32GX2VAVCNFSM6AAAAACKM3JHFOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTINRWGM2DEMBWGQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
|
yeah coz it wastes time, if u r raising a pr follow the complete pr template and test it before making any changes, if the changes works for ur local machine share the screenshot, that is easier to navigate for me to find the reason why did your commit review changes does not seem to work on my preview link right? |
Kindly review please