Skip to content

docs: update Theme Screen (Vintage) screenshot in README#776

Open
Arunodoy18 wants to merge 1 commit intoAOSSIE-Org:masterfrom
Arunodoy18:fix/775-update-readme-screenshot
Open

docs: update Theme Screen (Vintage) screenshot in README#776
Arunodoy18 wants to merge 1 commit intoAOSSIE-Org:masterfrom
Arunodoy18:fix/775-update-readme-screenshot

Conversation

@Arunodoy18
Copy link
Copy Markdown

@Arunodoy18 Arunodoy18 commented Feb 28, 2026

Summary

Replaces the outdated Theme Screen (Vintage) screenshot in README.md with the updated version from the latest playstore screenshots.

Changes

  • Updated the Theme Screen (Vintage) image reference in the App Screenshots section
  • No other sections, headings, or formatting modified

Closes #775

Summary by CodeRabbit

  • Documentation
    • Updated image path in the documentation to reference locally stored assets, ensuring consistent and reliable access to room screen previews.

@Arunodoy18 Arunodoy18 requested a review from M4dhav as a code owner February 28, 2026 18:27
@github-actions
Copy link
Copy Markdown
Contributor

🎉 Welcome @Arunodoy18!
Thank you for your pull request! Our team will review it soon. 🔍

  • Please ensure your PR follows the contribution guidelines. ✅
  • All automated tests should pass before merging. 🔄
  • If this PR fixes an issue, link it in the description. 🔗

We appreciate your contribution! 🚀

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Feb 28, 2026

📝 Walkthrough

Walkthrough

The README.md documentation file was updated to replace an outdated image reference in a table row. Specifically, the "Upcoming Room Screen (Cream)" image for the Vintage column was changed from an external GitHub asset URL to a local image path for consistency and maintainability.

Changes

Cohort / File(s) Summary
Documentation Update
README.md
Updated image source reference from GitHub asset URL to local path (docs/store_listing/images/playstore_screenshots/8.png) for the Upcoming Room Screen screenshot in the Vintage column.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A screenshot so fresh, no longer stale,
From GitHub clouds to local file trail,
The vintage room now shines with pride,
In docs/store\_listing, it shall reside! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: updating the Theme Screen (Vintage) screenshot in README, which is directly aligned with the file modifications.
Linked Issues check ✅ Passed The pull request successfully addresses issue #775 by replacing the outdated Theme Screen (Vintage) screenshot in README.md with the updated image from the latest Play Store screenshots.
Out of Scope Changes check ✅ Passed The changes are narrowly scoped to updating a single image reference in README.md for the Theme Screen (Vintage) row, with no other modifications to content, structure, or unrelated sections.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@README.md`:
- Line 91: The three <img> tags in the README row (the elements with src values
"https://github.com/user-attachments/assets/ba7da784-48a6-4512-a4c-9f12b8ad13c1",
"docs/store_listing/images/playstore_screenshots/8.png", and
"https://github.com/user-attachments/assets/a46c7da4-2df4-4c62-9e4c-9c92102339e9")
are missing alt attributes; add meaningful alt text for each image (e.g., brief
descriptions of the screenshot or graphic) by adding alt="..." to each <img> tag
so the row satisfies MD045 and improves accessibility.

ℹ️ Review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between bf1dbe2 and b71c7b7.

📒 Files selected for processing (1)
  • README.md

Comment thread README.md
| Explore Story (Amber) | Theme Screen (Vintage) | Upcoming Room Screen (Cream) |
| :---------------------------------------------------------------------------------------------------------------------- | :---------------------------------------------------------------------------------------------------------------------- | :--------------------------------------------------------------------------------------------------------------------- |
| <img src="https://github.com/user-attachments/assets/ba7da784-48a6-4512-a4c8-9f12b8ad13c1" width="250" height="auto" /> | <img src="https://github.com/user-attachments/assets/ba9273f2-ceef-441d-8f94-4e0bc53b3e99" width="250" height="auto" /> | <img src="https://github.com/user-attachments/assets/a46c7da4-2df4-4c62-9e4c-9c92102339e9" width="250" height="auto"/> |
| <img src="https://github.com/user-attachments/assets/ba7da784-48a6-4512-a4c8-9f12b8ad13c1" width="250" height="auto" /> | <img src="docs/store_listing/images/playstore_screenshots/8.png" width="250" height="auto" /> | <img src="https://github.com/user-attachments/assets/a46c7da4-2df4-4c62-9e4c-9c92102339e9" width="250" height="auto"/> |
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Add alt text to images in this row (accessibility/lint).

At Line 91, the <img> tags are missing alt attributes, which triggers MD045 and reduces accessibility for screen readers.

✅ Suggested patch
-| <img src="https://github.com/user-attachments/assets/ba7da784-48a6-4512-a4c8-9f12b8ad13c1" width="250" height="auto" /> | <img src="docs/store_listing/images/playstore_screenshots/8.png" width="250" height="auto" />                           | <img src="https://github.com/user-attachments/assets/a46c7da4-2df4-4c62-9e4c-9c92102339e9" width="250" height="auto"/> |
+| <img src="https://github.com/user-attachments/assets/ba7da784-48a6-4512-a4c8-9f12b8ad13c1" alt="Explore Story screen in Amber theme" width="250" height="auto" /> | <img src="docs/store_listing/images/playstore_screenshots/8.png" alt="Theme screen in Vintage theme" width="250" height="auto" /> | <img src="https://github.com/user-attachments/assets/a46c7da4-2df4-4c62-9e4c-9c92102339e9" alt="Upcoming Room screen in Cream theme" width="250" height="auto" /> |
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
| <img src="https://github.com/user-attachments/assets/ba7da784-48a6-4512-a4c8-9f12b8ad13c1" width="250" height="auto" /> | <img src="docs/store_listing/images/playstore_screenshots/8.png" width="250" height="auto" /> | <img src="https://github.com/user-attachments/assets/a46c7da4-2df4-4c62-9e4c-9c92102339e9" width="250" height="auto"/> |
| <img src="https://github.com/user-attachments/assets/ba7da784-48a6-4512-a4c8-9f12b8ad13c1" alt="Explore Story screen in Amber theme" width="250" height="auto" /> | <img src="docs/store_listing/images/playstore_screenshots/8.png" alt="Theme screen in Vintage theme" width="250" height="auto" /> | <img src="https://github.com/user-attachments/assets/a46c7da4-2df4-4c62-9e4c-9c92102339e9" alt="Upcoming Room screen in Cream theme" width="250" height="auto" /> |
🧰 Tools
🪛 markdownlint-cli2 (0.21.0)

[warning] 91-91: Images should have alternate text (alt text)

(MD045, no-alt-text)


[warning] 91-91: Images should have alternate text (alt text)

(MD045, no-alt-text)


[warning] 91-91: Images should have alternate text (alt text)

(MD045, no-alt-text)

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@README.md` at line 91, The three <img> tags in the README row (the elements
with src values
"https://github.com/user-attachments/assets/ba7da784-48a6-4512-a4c-9f12b8ad13c1",
"docs/store_listing/images/playstore_screenshots/8.png", and
"https://github.com/user-attachments/assets/a46c7da4-2df4-4c62-9e4c-9c92102339e9")
are missing alt attributes; add meaningful alt text for each image (e.g., brief
descriptions of the screenshot or graphic) by adding alt="..." to each <img> tag
so the row satisfies MD045 and improves accessibility.

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.

DOC: Update README.md with latest app screenshots.

1 participant