Add badge and polish & concise README#323
Merged
rainxchzed merged 9 commits intomainfrom Mar 13, 2026
Merged
Conversation
- Updated all GitHub repository links and shields across all localized README files to point to the new `OpenHub-Store/GitHub-Store` organization and repository. - Reorganized `media-resources` by moving mobile and desktop screenshots into specific subdirectories (`screenshots/mobile/` and `screenshots/desktop/`). - Updated `ProfileViewModel` to use the new repository URL for the issue reporting feature. - Refactored the main `README.md` to simplify the features list, update project statistics, and add a new "Get it on GitHub Store" badge. - Relocated the app icon and added new graphical assets (`app_icon.png`, `ghs_download_badge.png`) to the `media-resources` directory. - Fixed broken image references and adjusted layout formatting in the primary documentation.
- Fix indentation of `</br>` tags for improved code consistency.
- Add spacing between app store badges for better readability. - Set a fixed height for the GitHub Store download badge to maintain consistency with other badges. - Standardize the "Featured In" section header by replacing the `div` container with a `p` tag and removing extra whitespace.
- Reorganize the badge layout to group the Obtainium and GitHub Store links within the same paragraph. - Increase the height of the GitHub Store download badge from 60 to 80 to match the F-Droid badge.
- Replace `div` with `p` for the Download header alignment. - Reduce the height of the GitHub download badge from 80 to 70.
- Reduced the height of the GitHub Store download badge image from 80 to 64 in `README.md`.
Contributor
WalkthroughThis PR rebrands repository references from Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 1 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Note
Due to the large number of review comments, Critical, Major severity comments were prioritized as inline comments.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
README.md (1)
166-173:⚠️ Potential issue | 🟡 MinorRe-add install/update flow detail in the “Rich details screen” section.
Line 166 now omits installation/update behavior, while this section still describes detail-screen capabilities. Keeping that flow explicit improves accuracy and consistency.
Based on learnings: Repository detail screens should display full GitHub repository information including owner profile, stats, releases with download links, readme rendering, and installation/update flow.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@README.md` around lines 166 - 173, The "Rich details screen" section in README.md should reintroduce the installation/update flow: update the "Rich details screen" paragraph to explicitly describe how repository install/update is handled (e.g., show install/update buttons, current installed version, available release versions with download links and installer file sizes, update progress/status, and deep-linking to start installs), and ensure it references existing items like "Release notes with Markdown formatting", "List of installers with platform labels and file sizes", and "Deep linking support" so readers understand the end-to-end install/update behavior and UI elements.
🟡 Minor comments (23)
docs/README-ES.md-49-50 (1)
49-50:⚠️ Potential issue | 🟡 MinorHelloGitHub widget query uses old repository id.
Line 50 still references
rid=rainxchzed%2FGithub-Store, which is inconsistent with the updated branding.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/README-ES.md` around lines 49 - 50, The HelloGitHub widget in the README-ES uses an outdated repository id in the img src query parameter (rid=rainxchzed%2FGithub-Store); update that rid value to the correct repository identifier used by the project (replace rainxchzed%2FGithub-Store with the current repo id) so the widget points to the right repository—modify the img src URL string where the rid query appears to match the updated branding.docs/README-ES.md-23-23 (1)
23-23:⚠️ Potential issue | 🟡 MinorMissing
altattributes on updated images.Lines 23, 26, 35, 75–79, 116, and 119 still violate MD045 (
no-alt-text).Also applies to: 26-26, 35-35, 75-79, 116-116, 119-119
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/README-ES.md` at line 23, Add descriptive alt attributes to all bare <img .../> badge tags in README-ES.md (e.g., the badges with label=Downloads and the other updated image tags at the remaining occurrences) so they no longer trigger MD045/no-alt-text; locate each <img src="..."/> tag and add a concise alt="..." value that describes the badge (for example alt="Downloads badge"), keeping text brief and meaningful.docs/README-HI.md-49-50 (1)
49-50:⚠️ Potential issue | 🟡 MinorHelloGitHub widget
ridis still using the previous repo slug.Line 50 should be updated from
rainxchzed%2FGithub-StoretoOpenHub-Store%2FGitHub-Store.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/README-HI.md` around lines 49 - 50, Update the HelloGitHub widget RID in the image URL so it uses the new repository slug: replace the query value "rid=rainxchzed%2FGithub-Store" with "rid=OpenHub-Store%2FGitHub-Store" in the <img> src URL (the tag containing "https://abroad.hellogithub.com/v1/widgets/recommend.svg?...") so the widget points to the correct repository.docs/README-PL.md-49-50 (1)
49-50:⚠️ Potential issue | 🟡 MinorHelloGitHub widget still references old repo in
rid.Line 50 should use
OpenHub-Store%2FGitHub-Storeto stay consistent with the updated links.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/README-PL.md` around lines 49 - 50, Update the HelloGitHub widget's rid query parameter in the <img> tag so it references the new repository name; locate the <img src="https://abroad.hellogithub.com/v1/widgets/recommend.svg?rid=..."> element and replace the current rid value "rainxchzed%2FGithub-Store" with "OpenHub-Store%2FGitHub-Store" to match the updated links.docs/README-ZH.md-23-23 (1)
23-23:⚠️ Potential issue | 🟡 MinorAdd
alttext to changed images.Several changed
<img>tags are missingaltattributes (MD045), including Lines 23, 26, 35, 75–79, 116, and 119.♿ Proposed fix pattern
-<img src="https://img.shields.io/github/downloads/OpenHub-Store/GitHub-Store/total?..."/> +<img alt="Total downloads badge" src="https://img.shields.io/github/downloads/OpenHub-Store/GitHub-Store/total?..."/> -<img src="https://img.shields.io/github/stars/OpenHub-Store/GitHub-Store?..."/> +<img alt="GitHub stars badge" src="https://img.shields.io/github/stars/OpenHub-Store/GitHub-Store?..."/>Apply the same to the other changed image lines in this file.
Also applies to: 26-26, 35-35, 75-79, 116-116, 119-119
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/README-ZH.md` at line 23, The changed <img> tag (e.g., the badge markup shown) is missing an alt attribute causing MD045 failures; update that <img> element and the other changed image tags (lines referenced in the review) to include descriptive alt text (for example alt="Downloads badge" or similar contextual text) so each <img> has a meaningful alt attribute across the modified entries.docs/README-JA.md-49-50 (1)
49-50:⚠️ Potential issue | 🟡 MinorHelloGitHub badge
ridshould be renamed too.Line 50 still points to
rainxchzed%2FGithub-Storein the image URL query.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/README-JA.md` around lines 49 - 50, Update the HelloGitHub badge image URL so the rid query parameter matches the repository rename: in the HTML snippet containing the <img src="https://abroad.hellogithub.com/v1/widgets/recommend.svg?rid=..." /> replace the old rid value "rainxchzed%2FGithub-Store" with the new repository identifier (URL-encoded) so the badge points to the correct repo; ensure the same change is applied to the surrounding <a href="..."> if it also contains the old repo name.docs/README-HI.md-23-23 (1)
23-23:⚠️ Potential issue | 🟡 MinorAdd
altattributes to changed<img>tags.MD045 applies to the updated image lines (23, 26, 35, 75–79, 116, 119).
Also applies to: 26-26, 35-35, 75-79, 116-116, 119-119
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/README-HI.md` at line 23, The updated <img> tags lack alt attributes required by MD045; update each <img> element (e.g., the badge with src starting "https://img.shields.io/github/downloads/OpenHub-Store/GitHub-Store/total?..." and the other badge image tags on the same lines) to include a concise alt attribute (for example alt="Downloads" or an appropriate description for each badge) so every <img> has an alt value.docs/README-KR.md-23-23 (1)
23-23:⚠️ Potential issue | 🟡 MinorChanged images should include
alttext.MD045 is valid here for Lines 23, 26, 35, 75–79, 116, and 119.
Also applies to: 26-26, 35-35, 75-79, 116-116, 119-119
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/README-KR.md` at line 23, The markdown image tags (<img .../>) lack alt text which triggers MD045; update each <img> element that appears in the README (e.g., the badge images currently using <img src="..."/> ) to include a descriptive alt="" attribute (for example alt="Downloads badge") so every image tag has non-empty alt text; ensure you update all occurrences noted in the review (the various badge <img> tags) so the MD045 warning is resolved.docs/README-PL.md-23-23 (1)
23-23:⚠️ Potential issue | 🟡 MinorUpdated image tags are missing
altattributes.Please add
alttext for changed images at Lines 23, 26, 35, 75–79, 116, and 119.Also applies to: 26-26, 35-35, 75-79, 116-116, 119-119
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/README-PL.md` at line 23, Add descriptive alt attributes to the updated <img> tags so they are accessible and screen-reader friendly; locate the specific image elements (e.g., the shield image with src starting "https://img.shields.io/github/downloads/OpenHub-Store/GitHub-Store/total?..." and the other changed images mentioned) and add brief alt text like "Downloads badge" or equivalent descriptive text for each changed tag.docs/README-RU.md-23-23 (1)
23-23:⚠️ Potential issue | 🟡 MinorAccessibility/doc lint: missing
alttext on changed images.The updated image tags at Lines 23, 26, 35, 75–79, 116, and 119 need
altattributes.Also applies to: 26-26, 35-35, 75-79, 116-116, 119-119
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/README-RU.md` at line 23, The img tags added to the README-RU (the <img> elements containing the badge URLs with label=Downloads and similar badges) are missing alt attributes; update each changed <img> element to include a meaningful alt attribute (e.g., alt="Downloads badge" or alt="GitHub Stars badge") describing the image purpose so all badge images (the Downloads badge and the other modified badges) have proper accessibility text.docs/README-RU.md-49-50 (1)
49-50:⚠️ Potential issue | 🟡 MinorHelloGitHub embed source is not fully migrated.
Line 50 still contains
rid=rainxchzed%2FGithub-Store; update it to the OpenHub repo id.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/README-RU.md` around lines 49 - 50, Replace the outdated HelloGitHub embed rid value in the img src query string — find the <img> tag whose src contains "recommend.svg?rid=rainxchzed%2FGithub-Store" and change the rid parameter to the correct OpenHub repo id (i.e. replace "rid=rainxchzed%2FGithub-Store" with "rid=<OpenHub-repo-id>" so the embedded widget points to the OpenHub repository).docs/README-JA.md-23-23 (1)
23-23:⚠️ Potential issue | 🟡 MinorPlease add
alttext to the modified image tags.Changed image lines (23, 26, 35, 75–79, 116, 119) are missing
altattributes.Also applies to: 26-26, 35-35, 75-79, 116-116, 119-119
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/README-JA.md` at line 23, Several <img .../> badge tags in the README-JA have no alt attributes (e.g., the badge with label=Downloads in the src URL shown); add meaningful alt text to each modified/added <img> tag so assistive tech and validators have descriptions. Locate the modified <img> elements (the badge-style SVG URLs and other badges around the changes) and update them to include alt="Downloads badge" or an appropriate descriptive string for each image.docs/README-KR.md-49-50 (1)
49-50:⚠️ Potential issue | 🟡 MinorHelloGitHub widget source still points to old repo id.
Line 50 keeps
rid=rainxchzed%2FGithub-Store; it should match the new OpenHub repo.🔧 Proposed fix
- <img src="https://abroad.hellogithub.com/v1/widgets/recommend.svg?rid=rainxchzed%2FGithub-Store&claim_uid=&theme=dark" alt="Featured｜HelloGitHub" style="width: 250px; height: 54px;" width="250" height="54" /> + <img src="https://abroad.hellogithub.com/v1/widgets/recommend.svg?rid=OpenHub-Store%2FGitHub-Store&claim_uid=&theme=dark" alt="Featured｜HelloGitHub" style="width: 250px; height: 54px;" width="250" height="54" />🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/README-KR.md` around lines 49 - 50, Update the HelloGitHub widget embed so its img src query parameter rid uses the new OpenHub repo id instead of the old rainxchzed%2FGithub-Store value; locate the <img> element (the src attribute that contains "recommend.svg?rid=...") and replace the rid value with the correct repo identifier for OpenHub so the widget points to the new repository.docs/README-ZH.md-49-50 (1)
49-50:⚠️ Potential issue | 🟡 MinorUpdate the HelloGitHub widget
ridto the new repository.Line 50 still uses
rid=rainxchzed%2FGithub-Store, which is inconsistent with the renamed links.🔧 Proposed fix
- <img src="https://abroad.hellogithub.com/v1/widgets/recommend.svg?rid=rainxchzed%2FGithub-Store&claim_uid=&theme=dark" alt="Featured｜HelloGitHub" style="width: 250px; height: 54px;" width="250" height="54" /> + <img src="https://abroad.hellogithub.com/v1/widgets/recommend.svg?rid=OpenHub-Store%2FGitHub-Store&claim_uid=&theme=dark" alt="Featured｜HelloGitHub" style="width: 250px; height: 54px;" width="250" height="54" />🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/README-ZH.md` around lines 49 - 50, Update the HelloGitHub widget `rid` query parameter in the img tag's src (the line with the recommend.svg widget) to match the renamed repository; replace the existing rid=rainxchzed%2FGithub-Store with rid=OpenHub-Store%2FGitHub-Store so the widget points to the new repository (change the src attribute on the <img> tag accordingly).docs/README-FR.md-86-86 (1)
86-86:⚠️ Potential issue | 🟡 MinorPolish French phrasing for readability.
Line 86 reads awkwardly (“la foire aux questions et les informations utiles”). Consider a more natural form, e.g. “la FAQ et des informations utiles”.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/README-FR.md` at line 86, The French phrasing on the README line that currently says "Consultez le [Wiki](https://github.com/OpenHub-Store/GitHub-Store/wiki) de GitHub Store pour la foire aux questions et les informations utiles" should be made more natural; update that sentence to use "la FAQ et des informations utiles" (e.g. "Consultez le [Wiki](https://github.com/OpenHub-Store/GitHub-Store/wiki) de GitHub Store pour la FAQ et des informations utiles") so the wording reads smoothly and idiomatically.docs/README-IT.md-49-50 (1)
49-50:⚠️ Potential issue | 🟡 MinorKeep HelloGitHub link and widget source consistent.
Line 49 uses
OpenHub-Store/GitHub-Store, but Line 50 still hasrid=rainxchzed%2FGithub-Store. Please align the widgetridwith the new repo.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/README-IT.md` around lines 49 - 50, The HelloGitHub widget's repo identifier is inconsistent: the anchor href uses "OpenHub-Store/GitHub-Store" but the img src still has rid=rainxchzed%2FGithub-Store; update the rid query value in the img src to match the new repo "OpenHub-Store/GitHub-Store" (URL-encoded) so the rid parameter and the anchor href are consistent (update the src string containing "recommend.svg?rid=...").docs/README-FR.md-49-50 (1)
49-50:⚠️ Potential issue | 🟡 MinorFix stale HelloGitHub widget repository id.
Line 49 points to
OpenHub-Store/GitHub-Store, but Line 50 still usesrid=rainxchzed%2FGithub-Store. Update the widgetridto avoid mismatched branding/content.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/README-FR.md` around lines 49 - 50, The HelloGitHub widget rid parameter is inconsistent: the anchor uses "OpenHub-Store/GitHub-Store" but the img src still has "rid=rainxchzed%2FGithub-Store"; update the widget rid in the img src query string to match the repository id used in the link (replace the existing rid value in the src URL) so the widget reflects the correct repository branding (locate the <a> tag and the <img> src attribute in the shown snippet and change the rid query parameter accordingly).README.md-293-293 (1)
293-293:⚠️ Potential issue | 🟡 MinorTighten wording for grammar and clarity.
- Line 293: add article (“a high school student”).
- Line 311: remove duplicate phrasing (“other developers and non-tech buddies” is clearer).
Also applies to: 311-311
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@README.md` at line 293, Update the two sentences in the README: change "GitHub Store is built and maintained by high school student. Your support helps him:" to "GitHub Store is built and maintained by a high school student. Your support helps him:" (add the article "a"), and on the later sentence that reads "other developers and non-tech buddies" remove the duplicated phrasing so it reads simply "other developers and non-tech buddies" (ensure any repeated words are deleted so the phrase appears only once).docs/README-TR.md-49-50 (1)
49-50:⚠️ Potential issue | 🟡 MinorUpdate HelloGitHub widget
ridto OpenHub-Store.Line 49 was migrated, but Line 50 still embeds
rid=rainxchzed%2FGithub-Store. This should be updated to the new repository identifier.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/README-TR.md` around lines 49 - 50, The HelloGitHub widget embed still uses the old rid query param value; update the rid in the <img src="..."> URL by replacing "rid=rainxchzed%2FGithub-Store" with the new repository identifier "rid=OpenHub-Store" (URL-encode if your tool requires) so the <img> tag in the README-TR.md widget points to OpenHub-Store.docs/README-TR.md-23-29 (1)
23-29:⚠️ Potential issue | 🟡 MinorAddress missing alt text on changed images.
These changed image tags still have no
altattribute (matching MD045 warnings). Please add concise alt text for accessibility.Also applies to: 35-35, 75-79, 116-119
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/README-TR.md` around lines 23 - 29, Several badge <img> tags in README-TR.md are missing alt attributes; add concise alt text to each image tag (e.g., the badge with src containing "label=Downloads", the badge linking to stargazers with "github-stars...label=Stars", and the "65K+-Users" badge) and likewise add alt text for the other changed images noted (the ones flagged at the other sections). Locate the <img> tags by their src strings (containing "Downloads", "github-stars...label=Stars", and "65K+-Users") and add a short alt="" value describing the badge (e.g., alt="Downloads", alt="GitHub stars", alt="65K+ users"). Ensure each image tag uses the alt attribute with concise, human-readable text.docs/README-FR.md-23-30 (1)
23-30:⚠️ Potential issue | 🟡 MinorAdd missing
alttext to changed images.The changed image tags in these lines are missing
alt, which hurts accessibility and also matches the markdownlintMD045warnings.Also applies to: 75-76, 116-116
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/README-FR.md` around lines 23 - 30, The badge image tags are missing alt attributes (e.g., the <img> with src containing "github-downloads", the <img> inside the <a> with src containing "github-stars", and the <img> with src containing "65K+-Users" as well as the other occurrences noted), so add meaningful alt text to each <img> tag (brief, descriptive strings like "Downloads badge", "GitHub stars badge", "Users badge" or similar) to satisfy accessibility/MD045; ensure both standalone <img> and nested <img> inside anchors are updated.README.md-50-51 (1)
50-51:⚠️ Potential issue | 🟡 MinorSynchronize HelloGitHub widget source with the new repo.
Line 50 links to
OpenHub-Store/GitHub-Store, but Line 51 still usesrid=rainxchzed%2FGithub-Store. Update the widgetridto prevent stale project metadata.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@README.md` around lines 50 - 51, Update the HelloGitHub widget's rid query parameter so it matches the new repository; specifically, in the anchor/img snippet where href="https://hellogithub.com/en/repository/OpenHub-Store/GitHub-Store" and src contains "rid=rainxchzed%2FGithub-Store", replace the rid value with the URL-encoded new repo path (e.g. rid=OpenHub-Store%2FGitHub-Store) so the widget pulls the correct project metadata.docs/README-BN.md-49-50 (1)
49-50:⚠️ Potential issue | 🟡 MinorUpdate HelloGitHub widget
ridto the new repository.Line 49 points to
OpenHub-Store/GitHub-Store, but Line 50 still embedsrid=rainxchzed%2FGithub-Store. This can display mismatched project data.Suggested fix
- <img src="https://abroad.hellogithub.com/v1/widgets/recommend.svg?rid=rainxchzed%2FGithub-Store&claim_uid=&theme=dark" alt="Featured｜HelloGitHub" style="width: 250px; height: 54px;" width="250" height="54" /> + <img src="https://abroad.hellogithub.com/v1/widgets/recommend.svg?rid=OpenHub-Store%2FGitHub-Store&claim_uid=&theme=dark" alt="Featured｜HelloGitHub" style="width: 250px; height: 54px;" width="250" height="54" />🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/README-BN.md` around lines 49 - 50, The HelloGitHub widget embeds a mismatched repository id: the anchor href uses "OpenHub-Store/GitHub-Store" while the image src still has rid=rainxchzed%2FGithub-Store; update the rid parameter in the img src query to match the anchor href (percent-encode the slash, e.g. rid=OpenHub-Store%2FGitHub-Store) so the widget shows the correct repository.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Outside diff comments:
In `@README.md`:
- Around line 166-173: The "Rich details screen" section in README.md should
reintroduce the installation/update flow: update the "Rich details screen"
paragraph to explicitly describe how repository install/update is handled (e.g.,
show install/update buttons, current installed version, available release
versions with download links and installer file sizes, update progress/status,
and deep-linking to start installs), and ensure it references existing items
like "Release notes with Markdown formatting", "List of installers with platform
labels and file sizes", and "Deep linking support" so readers understand the
end-to-end install/update behavior and UI elements.
---
Minor comments:
In `@docs/README-BN.md`:
- Around line 49-50: The HelloGitHub widget embeds a mismatched repository id:
the anchor href uses "OpenHub-Store/GitHub-Store" while the image src still has
rid=rainxchzed%2FGithub-Store; update the rid parameter in the img src query to
match the anchor href (percent-encode the slash, e.g.
rid=OpenHub-Store%2FGitHub-Store) so the widget shows the correct repository.
In `@docs/README-ES.md`:
- Around line 49-50: The HelloGitHub widget in the README-ES uses an outdated
repository id in the img src query parameter (rid=rainxchzed%2FGithub-Store);
update that rid value to the correct repository identifier used by the project
(replace rainxchzed%2FGithub-Store with the current repo id) so the widget
points to the right repository—modify the img src URL string where the rid query
appears to match the updated branding.
- Line 23: Add descriptive alt attributes to all bare <img .../> badge tags in
README-ES.md (e.g., the badges with label=Downloads and the other updated image
tags at the remaining occurrences) so they no longer trigger MD045/no-alt-text;
locate each <img src="..."/> tag and add a concise alt="..." value that
describes the badge (for example alt="Downloads badge"), keeping text brief and
meaningful.
In `@docs/README-FR.md`:
- Line 86: The French phrasing on the README line that currently says "Consultez
le [Wiki](https://github.com/OpenHub-Store/GitHub-Store/wiki) de GitHub Store
pour la foire aux questions et les informations utiles" should be made more
natural; update that sentence to use "la FAQ et des informations utiles" (e.g.
"Consultez le [Wiki](https://github.com/OpenHub-Store/GitHub-Store/wiki) de
GitHub Store pour la FAQ et des informations utiles") so the wording reads
smoothly and idiomatically.
- Around line 49-50: The HelloGitHub widget rid parameter is inconsistent: the
anchor uses "OpenHub-Store/GitHub-Store" but the img src still has
"rid=rainxchzed%2FGithub-Store"; update the widget rid in the img src query
string to match the repository id used in the link (replace the existing rid
value in the src URL) so the widget reflects the correct repository branding
(locate the <a> tag and the <img> src attribute in the shown snippet and change
the rid query parameter accordingly).
- Around line 23-30: The badge image tags are missing alt attributes (e.g., the
<img> with src containing "github-downloads", the <img> inside the <a> with src
containing "github-stars", and the <img> with src containing "65K+-Users" as
well as the other occurrences noted), so add meaningful alt text to each <img>
tag (brief, descriptive strings like "Downloads badge", "GitHub stars badge",
"Users badge" or similar) to satisfy accessibility/MD045; ensure both standalone
<img> and nested <img> inside anchors are updated.
In `@docs/README-HI.md`:
- Around line 49-50: Update the HelloGitHub widget RID in the image URL so it
uses the new repository slug: replace the query value
"rid=rainxchzed%2FGithub-Store" with "rid=OpenHub-Store%2FGitHub-Store" in the
<img> src URL (the tag containing
"https://abroad.hellogithub.com/v1/widgets/recommend.svg?...") so the widget
points to the correct repository.
- Line 23: The updated <img> tags lack alt attributes required by MD045; update
each <img> element (e.g., the badge with src starting
"https://img.shields.io/github/downloads/OpenHub-Store/GitHub-Store/total?..."
and the other badge image tags on the same lines) to include a concise alt
attribute (for example alt="Downloads" or an appropriate description for each
badge) so every <img> has an alt value.
In `@docs/README-IT.md`:
- Around line 49-50: The HelloGitHub widget's repo identifier is inconsistent:
the anchor href uses "OpenHub-Store/GitHub-Store" but the img src still has
rid=rainxchzed%2FGithub-Store; update the rid query value in the img src to
match the new repo "OpenHub-Store/GitHub-Store" (URL-encoded) so the rid
parameter and the anchor href are consistent (update the src string containing
"recommend.svg?rid=...").
In `@docs/README-JA.md`:
- Around line 49-50: Update the HelloGitHub badge image URL so the rid query
parameter matches the repository rename: in the HTML snippet containing the <img
src="https://abroad.hellogithub.com/v1/widgets/recommend.svg?rid=..." /> replace
the old rid value "rainxchzed%2FGithub-Store" with the new repository identifier
(URL-encoded) so the badge points to the correct repo; ensure the same change is
applied to the surrounding <a href="..."> if it also contains the old repo name.
- Line 23: Several <img .../> badge tags in the README-JA have no alt attributes
(e.g., the badge with label=Downloads in the src URL shown); add meaningful alt
text to each modified/added <img> tag so assistive tech and validators have
descriptions. Locate the modified <img> elements (the badge-style SVG URLs and
other badges around the changes) and update them to include alt="Downloads
badge" or an appropriate descriptive string for each image.
In `@docs/README-KR.md`:
- Line 23: The markdown image tags (<img .../>) lack alt text which triggers
MD045; update each <img> element that appears in the README (e.g., the badge
images currently using <img src="..."/> ) to include a descriptive alt=""
attribute (for example alt="Downloads badge") so every image tag has non-empty
alt text; ensure you update all occurrences noted in the review (the various
badge <img> tags) so the MD045 warning is resolved.
- Around line 49-50: Update the HelloGitHub widget embed so its img src query
parameter rid uses the new OpenHub repo id instead of the old
rainxchzed%2FGithub-Store value; locate the <img> element (the src attribute
that contains "recommend.svg?rid=...") and replace the rid value with the
correct repo identifier for OpenHub so the widget points to the new repository.
In `@docs/README-PL.md`:
- Around line 49-50: Update the HelloGitHub widget's rid query parameter in the
<img> tag so it references the new repository name; locate the <img
src="https://abroad.hellogithub.com/v1/widgets/recommend.svg?rid=..."> element
and replace the current rid value "rainxchzed%2FGithub-Store" with
"OpenHub-Store%2FGitHub-Store" to match the updated links.
- Line 23: Add descriptive alt attributes to the updated <img> tags so they are
accessible and screen-reader friendly; locate the specific image elements (e.g.,
the shield image with src starting
"https://img.shields.io/github/downloads/OpenHub-Store/GitHub-Store/total?..."
and the other changed images mentioned) and add brief alt text like "Downloads
badge" or equivalent descriptive text for each changed tag.
In `@docs/README-RU.md`:
- Line 23: The img tags added to the README-RU (the <img> elements containing
the badge URLs with label=Downloads and similar badges) are missing alt
attributes; update each changed <img> element to include a meaningful alt
attribute (e.g., alt="Downloads badge" or alt="GitHub Stars badge") describing
the image purpose so all badge images (the Downloads badge and the other
modified badges) have proper accessibility text.
- Around line 49-50: Replace the outdated HelloGitHub embed rid value in the img
src query string — find the <img> tag whose src contains
"recommend.svg?rid=rainxchzed%2FGithub-Store" and change the rid parameter to
the correct OpenHub repo id (i.e. replace "rid=rainxchzed%2FGithub-Store" with
"rid=<OpenHub-repo-id>" so the embedded widget points to the OpenHub
repository).
In `@docs/README-TR.md`:
- Around line 49-50: The HelloGitHub widget embed still uses the old rid query
param value; update the rid in the <img src="..."> URL by replacing
"rid=rainxchzed%2FGithub-Store" with the new repository identifier
"rid=OpenHub-Store" (URL-encode if your tool requires) so the <img> tag in the
README-TR.md widget points to OpenHub-Store.
- Around line 23-29: Several badge <img> tags in README-TR.md are missing alt
attributes; add concise alt text to each image tag (e.g., the badge with src
containing "label=Downloads", the badge linking to stargazers with
"github-stars...label=Stars", and the "65K+-Users" badge) and likewise add alt
text for the other changed images noted (the ones flagged at the other
sections). Locate the <img> tags by their src strings (containing "Downloads",
"github-stars...label=Stars", and "65K+-Users") and add a short alt="" value
describing the badge (e.g., alt="Downloads", alt="GitHub stars", alt="65K+
users"). Ensure each image tag uses the alt attribute with concise,
human-readable text.
In `@docs/README-ZH.md`:
- Line 23: The changed <img> tag (e.g., the badge markup shown) is missing an
alt attribute causing MD045 failures; update that <img> element and the other
changed image tags (lines referenced in the review) to include descriptive alt
text (for example alt="Downloads badge" or similar contextual text) so each
<img> has a meaningful alt attribute across the modified entries.
- Around line 49-50: Update the HelloGitHub widget `rid` query parameter in the
img tag's src (the line with the recommend.svg widget) to match the renamed
repository; replace the existing rid=rainxchzed%2FGithub-Store with
rid=OpenHub-Store%2FGitHub-Store so the widget points to the new repository
(change the src attribute on the <img> tag accordingly).
In `@README.md`:
- Line 293: Update the two sentences in the README: change "GitHub Store is
built and maintained by high school student. Your support helps him:" to "GitHub
Store is built and maintained by a high school student. Your support helps him:"
(add the article "a"), and on the later sentence that reads "other developers
and non-tech buddies" remove the duplicated phrasing so it reads simply "other
developers and non-tech buddies" (ensure any repeated words are deleted so the
phrase appears only once).
- Around line 50-51: Update the HelloGitHub widget's rid query parameter so it
matches the new repository; specifically, in the anchor/img snippet where
href="https://hellogithub.com/en/repository/OpenHub-Store/GitHub-Store" and src
contains "rid=rainxchzed%2FGithub-Store", replace the rid value with the
URL-encoded new repo path (e.g. rid=OpenHub-Store%2FGitHub-Store) so the widget
pulls the correct project metadata.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 14b22f16-369a-4328-91b4-2e39911a5ae7
⛔ Files ignored due to path filters (16)
media-resources/app_icon.pngis excluded by!**/*.pngmedia-resources/banner.jpegis excluded by!**/*.jpegmedia-resources/ghs_download_badge.pngis excluded by!**/*.pngmedia-resources/screenshots/desktop/desktop_linux_appimage.jpgis excluded by!**/*.jpgmedia-resources/screenshots/desktop/desktop_linux_details.jpgis excluded by!**/*.jpgmedia-resources/screenshots/desktop/desktop_linux_home.jpgis excluded by!**/*.jpgmedia-resources/screenshots/desktop/desktop_linux_installing_rpm.jpgis excluded by!**/*.jpgmedia-resources/screenshots/desktop/desktop_windows_details.pngis excluded by!**/*.pngmedia-resources/screenshots/desktop/desktop_windows_details_install.pngis excluded by!**/*.pngmedia-resources/screenshots/desktop/desktop_windows_home.pngis excluded by!**/*.pngmedia-resources/screenshots/mobile/01.jpgis excluded by!**/*.jpgmedia-resources/screenshots/mobile/02.jpgis excluded by!**/*.jpgmedia-resources/screenshots/mobile/03.jpgis excluded by!**/*.jpgmedia-resources/screenshots/mobile/04.jpgis excluded by!**/*.jpgmedia-resources/screenshots/mobile/05.jpgis excluded by!**/*.jpgmedia-resources/screenshots/mobile/06.jpgis excluded by!**/*.jpg
📒 Files selected for processing (13)
README.mddocs/README-BN.mddocs/README-ES.mddocs/README-FR.mddocs/README-HI.mddocs/README-IT.mddocs/README-JA.mddocs/README-KR.mddocs/README-PL.mddocs/README-RU.mddocs/README-TR.mddocs/README-ZH.mdfeature/profile/presentation/src/commonMain/kotlin/zed/rainxch/profile/presentation/ProfileViewModel.kt
- Updated app icon and download badge paths to point to the `media-resources` directory. - Updated the Trendshift badge repository ID and link. - Added a new download badge for "GitHub Store". - Refined the "Features" and "How it works" sections, removing the redundant technical overview and cross-platform UX details. - Updated the "Support the project" section to reflect current project status and goals. - Standardized Markdown formatting with consistent line breaks and centered elements.
Contributor
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/README-RU.md`:
- Line 3: Several <img> tags (e.g., the one with
src="../media-resources/app_icon.png") lack alt attributes causing MD045; update
each changed <img> element to include a concise, descriptive alt text (for
example alt="App icon" for app_icon.png) so accessibility and markdownlint
pass—search for the bare <img src="…"> occurrences in the README-RU.md and add
appropriate alt="…" values for each image.
- Around line 50-51: The HelloGitHub widget uses two inconsistent repository
identifiers: the anchor href "OpenHub-Store/GitHub-Store" and the widget rid
parameter "rainxchzed%2FGithub-Store"; update the rid query value to match the
href by replacing "rainxchzed%2FGithub-Store" with the URL-encoded
"OpenHub-Store%2FGitHub-Store" so both the href and the src rid point to the
same repository identity.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 3d4e643a-d372-4176-abd9-d3741e5c4f7e
📒 Files selected for processing (1)
docs/README-RU.md
| <div align="center"> | ||
| </br> | ||
| <img src="../composeApp/src/commonMain/composeResources/drawable/app-icon.png" width="200" /> | ||
| <img src="../media-resources/app_icon.png" width="200" /> |
Contributor
There was a problem hiding this comment.
Add missing alt text for changed images.
Static analysis is correct here (MD045): these changed <img> tags are missing alt text. Please add concise descriptive alt values for accessibility and markdownlint compliance.
♿ Suggested patch pattern
-<img src="../media-resources/app_icon.png" width="200" />
+<img src="../media-resources/app_icon.png" width="200" alt="GitHub Store app icon" />
-<img src="https://img.shields.io/github/downloads/OpenHub-Store/GitHub-Store/total?..."/>
+<img src="https://img.shields.io/github/downloads/OpenHub-Store/GitHub-Store/total?..." alt="Total downloads badge" />
-<img src="../media-resources/screenshots/mobile/01.jpg" width="18%" />
+<img src="../media-resources/screenshots/mobile/01.jpg" width="18%" alt="Скриншот мобильного интерфейса 1" />Also applies to: 24-24, 27-27, 30-30, 36-36, 72-76, 112-112, 116-116
🧰 Tools
🪛 markdownlint-cli2 (0.21.0)
[warning] 3-3: 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 `@docs/README-RU.md` at line 3, Several <img> tags (e.g., the one with
src="../media-resources/app_icon.png") lack alt attributes causing MD045; update
each changed <img> element to include a concise, descriptive alt text (for
example alt="App icon" for app_icon.png) so accessibility and markdownlint
pass—search for the bare <img src="…"> occurrences in the README-RU.md and add
appropriate alt="…" values for each image.
- Update app icon and badge image sources to use the new `media-resources` path. - Update Trendshift repository ID and badge link. - Add a new download badge for GitHub Store. - Adjust image heights and alignment for badges and download buttons. - Simplify the "Features" section by removing the cross-platform experience details and specific "Install latest version" button mentions. - Remove the "How GitHub Store works" overview section. - Refine the "Support the project" and "Disclaimer" sections for clarity and updated project context. - Clean up Markdown formatting, including line breaks and list structures.
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
docs/README-ES.md (1)
24-24: Consider adding alt text to badge images for accessibility.Several badge images lack descriptive alt text, which impacts screen reader users. While not critical for decorative badges, adding descriptive alt text improves accessibility.
♿ Example improvements
-<img src="https://img.shields.io/github/downloads/OpenHub-Store/GitHub-Store/total?color=aeff4d&style=for-the-badge&logo=data%3Aimage%2Fsvg%2Bxml%3Bbase64%2CPHN2ZyB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMjAwMC9zdmciIHZpZXdCb3g9IjAgMCAyNCAyNCI+PHRpdGxlPmRvd25sb2FkPC90aXRsZT48cGF0aCBkPSJNNSwyMEgxOVYxOEg1TTE5LDlIMTVWM0g5VjlINUwxMiwxNkwxOSw5WiIgZmlsbD0id2hpdGUiIC8+PC9zdmc+&label=Downloads&labelColor=4b731a"/> +<img alt="Total Downloads" src="https://img.shields.io/github/downloads/OpenHub-Store/GitHub-Store/total?color=aeff4d&style=for-the-badge&logo=data%3Aimage%2Fsvg%2Bxml%3Bbase64%2CPHN2ZyB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMjAwMC9zdmciIHZpZXdCb3g9IjAgMCAyNCAyNCI+PHRpdGxlPmRvd25sb2FkPC90aXRsZT48cGF0aCBkPSJNNSwyMEgxOVYxOEg1TTE5LDlIMTVWM0g5VjlINUwxMiwxNkwxOSw5WiIgZmlsbD0id2hpdGUiIC8+PC9zdmc+&label=Downloads&labelColor=4b731a"/>Similar improvements can be applied to other badge images on lines 27, 30, and 36.
Also applies to: 27-27, 30-30, 36-36
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/README-ES.md` at line 24, The badge image tag is missing an alt attribute which reduces accessibility; update the <img> elements (e.g., the Downloads badge image shown) to include descriptive alt text like alt="Downloads badge" (or Spanish equivalent) and do the same for the other badges referenced on lines 27, 30, and 36 so screen readers get meaningful descriptions.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/README-ES.md`:
- Line 51: The HelloGitHub widget image tag is using the old repository
identifier in its src query parameter (rid=rainxchzed%2FGithub-Store); update
the rid value in the recommend.svg URL to match the new repository path used
elsewhere (replace rainxchzed%2FGithub-Store with the new repo identifier,
properly URL-encoded) so the widget consistently points to the correct repo.
---
Nitpick comments:
In `@docs/README-ES.md`:
- Line 24: The badge image tag is missing an alt attribute which reduces
accessibility; update the <img> elements (e.g., the Downloads badge image shown)
to include descriptive alt text like alt="Downloads badge" (or Spanish
equivalent) and do the same for the other badges referenced on lines 27, 30, and
36 so screen readers get meaningful descriptions.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 74690c21-49bc-4f76-82b5-f0cb388022c6
📒 Files selected for processing (1)
docs/README-ES.md
- Update `app-icon.png` path to `media-resources/app_icon.png` across all localized README files. - Refactor badge layouts and update several third-party repository links (Trendshift, HelloGitHub) across multiple languages. - Improve translation quality and natural phrasing for Chinese (ZH), Japanese (JA), Bengali (BN), French (FR), Hindi (HI), Italian (IT), Turkish (TR), and Korean (KR) documentation. - Standardize the "How to support" and "Legal notice" sections in international versions. - Add "Get it on GitHub Store" download badge to multiple localized READMEs. - Clean up formatting, line breaks, and list styles for better readability across all documentation files.
Contributor
There was a problem hiding this comment.
Actionable comments posted: 15
♻️ Duplicate comments (1)
docs/README-RU.md (1)
3-3:⚠️ Potential issue | 🟡 MinorMissing image
alttext remains unresolved.Line 3 and the listed changed image lines still have no
altattributes (MD045).Also applies to: 24-24, 27-27, 30-30, 72-76, 112-112, 116-116
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/README-RU.md` at line 3, The image tags in README-RU.md (e.g., the <img src="../media-resources/app_icon.png" width="200" /> instance and the other images at the noted lines) are missing alt attributes, causing MD045 failures; update each <img> tag (including the ones around lines referenced: 3, 24, 27, 30, 72-76, 112, 116) to include a concise, descriptive alt attribute (for example alt="App icon" or a context-appropriate description) so every image tag has an alt text value.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/README-BN.md`:
- Line 3: Add descriptive alt text to each <img> tag that currently lacks an alt
attribute (e.g., the tag "<img src=\"../media-resources/app_icon.png\"
width=\"200\" />") to satisfy accessibility and MD045; update all instances
mentioned (around the image tags at the specified locations) by inserting
appropriate alt="..." values that describe the image content succinctly.
In `@docs/README-ES.md`:
- Line 3: The missing alt attribute on several <img> tags in docs/README-ES.md
is causing markdownlint MD045 failures; update each <img> element (e.g., the one
at the top that currently reads <img src="../media-resources/app_icon.png"
width="200" /> and the others at the listed locations) to include a meaningful
alt="..." attribute describing the image (e.g., alt="App icon" or appropriate
descriptive text) so every <img> tag has an alt value for accessibility and
linting.
In `@docs/README-FR.md`:
- Line 3: Add concise alt attributes to the changed <img> tags (e.g., the <img
src="../media-resources/app_icon.png" width="200" /> instance and the other
image tags referenced at lines 24, 27, 30, 36, 71-75, 111, and 116) to satisfy
accessibility and markdownlint MD045; update each tag to include a meaningful
alt="..." value that describes the image (for example alt="App icon" for
app_icon.png) while preserving existing src and width attributes.
In `@docs/README-HI.md`:
- Line 3: Multiple <img> tags (e.g., <img src="../media-resources/app_icon.png"
width="200" /> and the other image tags around lines mentioned) are missing alt
attributes causing accessibility/MD045 failures; update each image tag in
README-HI.md to include a meaningful alt text (for example alt="App icon" for
app_icon.png and descriptive alt text matching each image's content) so every
<img ... /> becomes <img ... alt="..."/> throughout the file (apply same fix to
the other changed image tags noted).
In `@docs/README-IT.md`:
- Line 3: Update all <img> tags in README-IT.md that currently lack alt
attributes (for example the tag '<img src="../media-resources/app_icon.png"
width="200" />' and the other changed image tags) by adding meaningful alt text
for informative images or an empty alt (alt="") for purely decorative images so
they pass MD045; use concise Italian descriptions for content images (e.g.,
alt="Icona dell'app" or similar) and alt="" for decorative icons.
- Line 94: Change the heading "### 📋 Note Legali" to a level-2 heading so it
reads "## 📋 Note Legali"; update the header in the docs/README-IT.md (the
heading string "📋 Note Legali") to use two hashes instead of three to fix the
heading level jump and keep the TOC/MD001 hierarchy consistent.
In `@docs/README-JA.md`:
- Line 94: Change the heading "### 📋 法的事項" to a second-level heading "## 📋
法的事項" to normalize the Markdown heading hierarchy (resolve MD001); locate the
heading string "### 📋 法的事項" in the README-JA.md content and replace the leading
triple-hash with double-hash so the document uses a consistent heading level.
- Line 3: The changed <img> tags (e.g., the one referencing
"../media-resources/app_icon.png" and the other images in this diff) are missing
alt attributes; update each <img> element (all occurrences in the diff) to
include a concise, descriptive alt="..." string (for example alt="App icon" or
localized Japanese text) so the images pass MD045 and improve accessibility;
ensure the alt text is meaningful and localized where appropriate.
In `@docs/README-KR.md`:
- Line 3: Several <img> tags in README-KR.md are missing alt attributes (e.g.,
<img src="../media-resources/app_icon.png" width="200" />); add meaningful alt
text to each image tag to satisfy accessibility/lint rules (MD045). Locate each
<img> occurrence and change it to include an alt attribute (for example alt="앱
아이콘" or context-appropriate Korean description) and ensure every image
(including the ones around the other changed lines) has non-empty alt text.
In `@docs/README-PL.md`:
- Line 23: Several <img> badge tags in the Markdown (for example the badge with
src
"https://img.shields.io/github/downloads/OpenHub-Store/GitHub-Store/total?..."
and the other badge/image tags changed at the referenced locations) are missing
alt attributes and causing MD045 failures; update each affected <img> tag to
include a concise, descriptive alt attribute (e.g., alt="Downloads badge" or
alt="License badge") so they are non-empty and meaningful, apply the same fix to
the other changed badge/image tags noted (lines 26, 75-79, 116), and verify the
markdown linter no longer reports MD045.
- Around line 47-50: The Trendshift anchor opened with '<a
href="https://trendshift.io/repositories/15655"...>' is not closed before the
next anchor '<a
href="https://hellogithub.com/en/repository/OpenHub-Store/GitHub-Store"...>';
fix by inserting a closing </a> immediately after the Trendshift image tag (the
<img src="https://trendshift.io/api/badge/repositories/15655" ... />) so each
link is its own properly closed anchor.
In `@docs/README-TR.md`:
- Line 3: Add meaningful alt attributes to all <img> tags that currently lack
them (e.g. the tag shown as <img src="../media-resources/app_icon.png"
width="200" /> and the other image tags referenced in the comment) so they pass
MD045 accessibility checks; update each tag to include an appropriate alt="..."
value that describes the image content (for decorative images use alt="").
Ensure every modified <img> tag across the file has a non-empty alt attribute or
explicit empty alt for purely decorative images.
In `@docs/README-ZH.md`:
- Line 3: Add concise alt attributes to the changed <img> tags (e.g., the tag
with src="../media-resources/app_icon.png" and the other image tags noted in the
comment) so they no longer violate MD045; update each img element to include a
brief localized or neutral alt text (for example a Chinese label like "应用图标" or
a neutral "app icon") and ensure all instances mentioned (the app_icon.png line
and the other changed image lines) are updated consistently.
In `@README.md`:
- Line 3: Several <img> tags in README.md (for example the image with src
"media-resources/app_icon.png" and other images around lines noted in the
review) are missing alt attributes and trigger MD045; update each <img> element
to include a meaningful alt="..." describing the image (e.g., alt="App icon" for
media-resources/app_icon.png) so screen readers can convey the content, ensuring
all instances mentioned (lines ~3, 24, 27, 36, 72-76, 112) get appropriate alt
text that is concise and descriptive.
- Line 293: Update the two awkward phrases in the README: change "GitHub Store
is built and maintained by high school student." to "GitHub Store is built and
maintained by a high school student." and replace "tech and non-tech buddies"
with a clearer phrase such as "technical and non‑technical friends" (locate the
exact occurrences by searching for those strings in the README and update them
accordingly).
---
Duplicate comments:
In `@docs/README-RU.md`:
- Line 3: The image tags in README-RU.md (e.g., the <img
src="../media-resources/app_icon.png" width="200" /> instance and the other
images at the noted lines) are missing alt attributes, causing MD045 failures;
update each <img> tag (including the ones around lines referenced: 3, 24, 27,
30, 72-76, 112, 116) to include a concise, descriptive alt attribute (for
example alt="App icon" or a context-appropriate description) so every image tag
has an alt text value.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: a90510e2-40ef-4d5d-9491-a7dd4480b708
📒 Files selected for processing (12)
README.mddocs/README-BN.mddocs/README-ES.mddocs/README-FR.mddocs/README-HI.mddocs/README-IT.mddocs/README-JA.mddocs/README-KR.mddocs/README-PL.mddocs/README-RU.mddocs/README-TR.mddocs/README-ZH.md
| <div align="center"> | ||
| </br> | ||
| <img src="../composeApp/src/commonMain/composeResources/drawable/app-icon.png" width="200" /> | ||
| <img src="../media-resources/app_icon.png" width="200" /> |
Contributor
There was a problem hiding this comment.
Add missing image alt text.
The changed image tags on these lines omit alt attributes, which breaks accessibility and MD045 lint compliance.
Also applies to: 24-24, 27-27, 30-30, 72-76, 112-112, 116-116
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/README-BN.md` at line 3, Add descriptive alt text to each <img> tag that
currently lacks an alt attribute (e.g., the tag "<img
src=\"../media-resources/app_icon.png\" width=\"200\" />") to satisfy
accessibility and MD045; update all instances mentioned (around the image tags
at the specified locations) by inserting appropriate alt="..." values that
describe the image content succinctly.
| <div align="center"> | ||
| </br> | ||
| <img src="../composeApp/src/commonMain/composeResources/drawable/app-icon.png" width="200" /> | ||
| <img src="../media-resources/app_icon.png" width="200" /> |
Contributor
There was a problem hiding this comment.
Include alt attributes for changed images.
Line 3 and the listed changed <img> tags are missing alt, so markdownlint/accessibility checks still fail (MD045).
Also applies to: 24-24, 27-27, 30-30, 72-76, 112-112, 116-116
🧰 Tools
🪛 markdownlint-cli2 (0.21.0)
[warning] 3-3: 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 `@docs/README-ES.md` at line 3, The missing alt attribute on several <img> tags
in docs/README-ES.md is causing markdownlint MD045 failures; update each <img>
element (e.g., the one at the top that currently reads <img
src="../media-resources/app_icon.png" width="200" /> and the others at the
listed locations) to include a meaningful alt="..." attribute describing the
image (e.g., alt="App icon" or appropriate descriptive text) so every <img> tag
has an alt value for accessibility and linting.
| <div align="center"> | ||
| </br> | ||
| <img src="../composeApp/src/commonMain/composeResources/drawable/app-icon.png" width="200" /> | ||
| <img src="../media-resources/app_icon.png" width="200" /> |
Contributor
There was a problem hiding this comment.
Missing alt attributes in changed image tags.
Please add concise alt text on these lines to satisfy accessibility and markdownlint MD045.
Also applies to: 24-24, 27-27, 30-30, 36-36, 71-75, 111-111, 116-116
🧰 Tools
🪛 markdownlint-cli2 (0.21.0)
[warning] 3-3: 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 `@docs/README-FR.md` at line 3, Add concise alt attributes to the changed <img>
tags (e.g., the <img src="../media-resources/app_icon.png" width="200" />
instance and the other image tags referenced at lines 24, 27, 30, 36, 71-75,
111, and 116) to satisfy accessibility and markdownlint MD045; update each tag
to include a meaningful alt="..." value that describes the image (for example
alt="App icon" for app_icon.png) while preserving existing src and width
attributes.
| <div align="center"> | ||
| </br> | ||
| <img src="../composeApp/src/commonMain/composeResources/drawable/app-icon.png" width="200" /> | ||
| <img src="../media-resources/app_icon.png" width="200" /> |
Contributor
There was a problem hiding this comment.
Please add alt text to changed images.
Line 3 and other listed image tags are missing alt, which is an accessibility regression and markdownlint MD045 violation.
Also applies to: 24-24, 27-27, 30-30, 70-75, 111-111, 115-115
🧰 Tools
🪛 markdownlint-cli2 (0.21.0)
[warning] 3-3: 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 `@docs/README-HI.md` at line 3, Multiple <img> tags (e.g., <img
src="../media-resources/app_icon.png" width="200" /> and the other image tags
around lines mentioned) are missing alt attributes causing accessibility/MD045
failures; update each image tag in README-HI.md to include a meaningful alt text
(for example alt="App icon" for app_icon.png and descriptive alt text matching
each image's content) so every <img ... /> becomes <img ... alt="..."/>
throughout the file (apply same fix to the other changed image tags noted).
| <div align="center"> | ||
| </br> | ||
| <img src="../composeApp/src/commonMain/composeResources/drawable/app-icon.png" width="200" /> | ||
| <img src="../media-resources/app_icon.png" width="200" /> |
Contributor
There was a problem hiding this comment.
Add missing alt attributes on changed images.
Changed images at Line 3, Line 24, Line 71+ still miss alt, which fails MD045 and hurts accessibility.
Also applies to: 24-24, 27-27, 36-36, 71-75, 111-111
🧰 Tools
🪛 markdownlint-cli2 (0.21.0)
[warning] 3-3: 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 `@docs/README-IT.md` at line 3, Update all <img> tags in README-IT.md that
currently lack alt attributes (for example the tag '<img
src="../media-resources/app_icon.png" width="200" />' and the other changed
image tags) by adding meaningful alt text for informative images or an empty alt
(alt="") for purely decorative images so they pass MD045; use concise Italian
descriptions for content images (e.g., alt="Icona dell'app" or similar) and
alt="" for decorative icons.
Comment on lines
+47
to
+50
| <a href="https://trendshift.io/repositories/15655" target="_blank"><img src="https://trendshift.io/api/badge/repositories/15655" alt="OpenHub-Store%2FGitHub-Store | Trendshift" style="width: 250px; height: 54px;" width="250" height="55" /> | ||
|
|
||
| <a href="https://hellogithub.com/en/repository/rainxchzed/Github-Store" target="_blank"> | ||
| <img src="https://abroad.hellogithub.com/v1/widgets/recommend.svg?rid=rainxchzed%2FGithub-Store&claim_uid=&theme=dark" alt="Featured｜HelloGitHub" style="width: 250px; height: 54px;" width="250" height="54" /> | ||
| <a href="https://hellogithub.com/en/repository/OpenHub-Store/GitHub-Store" target="_blank"> | ||
| <img src="https://abroad.hellogithub.com/v1/widgets/recommend.svg?rid=OpenHub-Store%2FGitHub-Store&claim_uid=&theme=dark" alt="Featured｜HelloGitHub" style="width: 250px; height: 54px;" width="250" height="54" /> |
Contributor
There was a problem hiding this comment.
Close the Trendshift anchor tag before the next link.
Line 47 opens an <a> that is never closed before Line 49 starts another <a>, producing invalid HTML and unreliable rendering/click behavior.
🔧 Suggested fix
- <a href="https://trendshift.io/repositories/15655" target="_blank"><img src="https://trendshift.io/api/badge/repositories/15655" alt="OpenHub-Store%2FGitHub-Store | Trendshift" style="width: 250px; height: 54px;" width="250" height="55" />
+ <a href="https://trendshift.io/repositories/15655" target="_blank"><img src="https://trendshift.io/api/badge/repositories/15655" alt="OpenHub-Store%2FGitHub-Store | Trendshift" style="width: 250px; height: 54px;" width="250" height="55" /></a>🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/README-PL.md` around lines 47 - 50, The Trendshift anchor opened with
'<a href="https://trendshift.io/repositories/15655"...>' is not closed before
the next anchor '<a
href="https://hellogithub.com/en/repository/OpenHub-Store/GitHub-Store"...>';
fix by inserting a closing </a> immediately after the Trendshift image tag (the
<img src="https://trendshift.io/api/badge/repositories/15655" ... />) so each
link is its own properly closed anchor.
| <div align="center"> | ||
| </br> | ||
| <img src="../composeApp/src/commonMain/composeResources/drawable/app-icon.png" width="200" /> | ||
| <img src="../media-resources/app_icon.png" width="200" /> |
Contributor
There was a problem hiding this comment.
Provide alt attributes for changed <img> tags.
Line 3 and the other listed image lines are missing alt text, so this still fails accessibility/lint checks (MD045).
Also applies to: 24-24, 27-27, 30-30, 36-36, 71-75, 112-112, 116-116
🧰 Tools
🪛 markdownlint-cli2 (0.21.0)
[warning] 3-3: 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 `@docs/README-TR.md` at line 3, Add meaningful alt attributes to all <img> tags
that currently lack them (e.g. the tag shown as <img
src="../media-resources/app_icon.png" width="200" /> and the other image tags
referenced in the comment) so they pass MD045 accessibility checks; update each
tag to include an appropriate alt="..." value that describes the image content
(for decorative images use alt=""). Ensure every modified <img> tag across the
file has a non-empty alt attribute or explicit empty alt for purely decorative
images.
| <div align="center"> | ||
| </br> | ||
| <img src="../composeApp/src/commonMain/composeResources/drawable/app-icon.png" width="200" /> | ||
| <img src="../media-resources/app_icon.png" width="200" /> |
Contributor
There was a problem hiding this comment.
Add missing alt attributes for changed images.
These changed image lines still omit alt text (MD045). Please add concise localized or neutral alt labels.
Also applies to: 25-25, 28-28, 37-37, 72-76, 112-112
🧰 Tools
🪛 markdownlint-cli2 (0.21.0)
[warning] 3-3: 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 `@docs/README-ZH.md` at line 3, Add concise alt attributes to the changed <img>
tags (e.g., the tag with src="../media-resources/app_icon.png" and the other
image tags noted in the comment) so they no longer violate MD045; update each
img element to include a brief localized or neutral alt text (for example a
Chinese label like "应用图标" or a neutral "app icon") and ensure all instances
mentioned (the app_icon.png line and the other changed image lines) are updated
consistently.
| <div align="center"> | ||
| </br> | ||
| <img src="./composeApp/src/commonMain/composeResources/drawable/app-icon.png" width="200" /> | ||
| <img src="media-resources/app_icon.png" width="200" /> |
Contributor
There was a problem hiding this comment.
Add alt text to changed images for accessibility compliance.
Several changed <img> tags (e.g., Line 3, Line 24, Line 72) are missing alt, which triggers MD045 and reduces screen-reader usability.
Suggested fix pattern
-<img src="media-resources/app_icon.png" width="200" />
+<img src="media-resources/app_icon.png" alt="GitHub Store app icon" width="200" />
-<img src="https://img.shields.io/github/downloads/OpenHub-Store/GitHub-Store/total?..."/>
+<img src="https://img.shields.io/github/downloads/OpenHub-Store/GitHub-Store/total?..." alt="Total downloads badge"/>
-<img src="media-resources/screenshots/mobile/01.jpg" width="18%" />
+<img src="media-resources/screenshots/mobile/01.jpg" alt="Mobile screenshot 1" width="18%" />Also applies to: 24-24, 27-27, 36-36, 72-76, 112-112
🧰 Tools
🪛 markdownlint-cli2 (0.21.0)
[warning] 3-3: 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 3, Several <img> tags in README.md (for example the image
with src "media-resources/app_icon.png" and other images around lines noted in
the review) are missing alt attributes and trigger MD045; update each <img>
element to include a meaningful alt="..." describing the image (e.g., alt="App
icon" for media-resources/app_icon.png) so screen readers can convey the
content, ensuring all instances mentioned (lines ~3, 24, 27, 36, 72-76, 112) get
appropriate alt text that is concise and descriptive.
| **GitHub Store** has reached **48,000+ active users** and **5,500+ GitHub stars** — and it's **100% free** with no ads, no tracking, and no premium features. | ||
|
|
||
| I built and maintain this entirely on my own while finishing high school. Your support (even $3) helps me: | ||
| GitHub Store is built and maintained by high school student. Your support helps him: |
Contributor
There was a problem hiding this comment.
Fix minor wording issues in support section.
Line 293 is missing an article (“a high school student”), and Line 311 reads awkwardly (“tech and non-tech buddies”).
Suggested wording
-GitHub Store is built and maintained by high school student. Your support helps him:
+GitHub Store is built and maintained by a high school student. Your support helps him:
-- 📢 **Sharing with friends** — spread the word to other developers and tech and non-tech buddies!
+- 📢 **Sharing with friends** — spread the word to developers and non-technical friends!Also applies to: 311-311
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@README.md` at line 293, Update the two awkward phrases in the README: change
"GitHub Store is built and maintained by high school student." to "GitHub Store
is built and maintained by a high school student." and replace "tech and
non-tech buddies" with a clearer phrase such as "technical and non‑technical
friends" (locate the exact occurrences by searching for those strings in the
README and update them accordingly).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary by CodeRabbit
Documentation
Chores