Skip to content

Add real gameplay screenshots to README#8

Closed
PraveenGitGenius wants to merge 1 commit into
mainfrom
PraveenGitGenius-patch-1
Closed

Add real gameplay screenshots to README#8
PraveenGitGenius wants to merge 1 commit into
mainfrom
PraveenGitGenius-patch-1

Conversation

@PraveenGitGenius
Copy link
Copy Markdown
Collaborator

This PR adds real screenshots of the KeyQuest main menu and tutorial gameplay to the repository.

These screenshots help users understand the actual interface and game modes more clearly.

Added real screenshots of the main menu and tutorial gameplay to improve README documentation.
@PraveenGitGenius
Copy link
Copy Markdown
Collaborator Author

Hiiii... I’ve added the screenshots for the main menu and tutorial gameplay.
Please review and let me know if any changes are needed.

@csm120
Copy link
Copy Markdown
Collaborator

csm120 commented Mar 11, 2026

Thanks for putting this together. The screenshots themselves look like a useful start, but I’m not going to merge this PR as-is because it does not yet complete the issue scope.

Right now the diff only adds the image files under docs/assets/, but it does not actually update the README or other user-facing docs to display them. That means users still will not see the screenshots anywhere.

To make this PR mergeable, please update it to include:

  • the README content where the screenshots should appear
  • descriptive alt text for each screenshot
  • any matching user-facing doc/site updates if needed so the screenshots are not only stored as assets

If you want to keep the change small, updating README.md with the screenshots and clear alt text would be a good next step.

@csm120
Copy link
Copy Markdown
Collaborator

csm120 commented Mar 22, 2026

Thanks for the contribution. I?m closing this out from the review side because it only adds the screenshot assets and does not complete the user-facing documentation change by itself.

Please continue the work in PR #9, or open one replacement PR that includes both:

  • the screenshot image files
  • the README update that displays them with descriptive alt text

I?m trying to keep issue #4 to one mergeable PR rather than split the assets and README changes across separate pull requests.

@csm120
Copy link
Copy Markdown
Collaborator

csm120 commented Mar 22, 2026

Closing this in favor of one complete, mergeable PR for the screenshots work. Please continue in PR #9 or a replacement PR that includes both the image assets and the README update.

@csm120 csm120 closed this Mar 22, 2026
@csm120 csm120 deleted the PraveenGitGenius-patch-1 branch March 22, 2026 15:40
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