Skip to content

docs: more icons in readme#1687

Merged
amezin merged 1 commit into
masterfrom
readme-bells-and-whistles
May 31, 2026
Merged

docs: more icons in readme#1687
amezin merged 1 commit into
masterfrom
readme-bells-and-whistles

Conversation

@amezin

@amezin amezin commented Jan 23, 2026

Copy link
Copy Markdown
Member

No description provided.

@coderabbitai

coderabbitai Bot commented Jan 23, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: ffaece8b-620e-4fbd-9265-5f0df7aa25ea

📥 Commits

Reviewing files that changed from the base of the PR and between 76d3cb8 and 58f087d.

📒 Files selected for processing (1)
  • README.md
📜 Recent review details
⏰ Context from checks skipped due to timeout of 900000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (5)
  • GitHub Check: ci / bundle
  • GitHub Check: ci / lint
  • GitHub Check: Analyze (actions)
  • GitHub Check: Analyze (python)
  • GitHub Check: Analyze (javascript-typescript)
🔇 Additional comments (1)
README.md (1)

19-19: ⚡ Quick win

Dev build badge: query=0.updated_at renders correctly (no change needed)
The Dev build badge SVG from README.md line 19 loads without any obvious “invalid/error/n/a/unknown/not found” markers, so query=0.updated_at is being accepted by shields.io.


📝 Walkthrough

Summary by CodeRabbit

  • Documentation
    • Updated README badges: added an AUR badge and removed the Weblate languages badge.
    • Replaced the Dev build badge with a new shields.io definition (updated query and label).
    • Switched Weblate badge to shields.io “weblate progress” and adjusted footer badge links accordingly.

Walkthrough

Updated the README badge block: added an AUR badge, removed the Weblate languages badge, changed the Dev build shields.io definition, and replaced the hosted Weblate widget reference with a shields.io weblate progress badge; corresponding footer badge links were adjusted.

Changes

README Badge Updates

Layer / File(s) Summary
Badge area and reference updates
README.md
Added an AUR badge and removed the Weblate languages badge from the header; updated the “Dev build” shields.io badge definition and link; replaced the hosted Weblate widget reference with a shields.io weblate progress badge and adjusted footer badge links accordingly.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No pull request description was provided by the author, making it impossible to assess whether any description relates to the changeset. Add a description explaining the changes made, such as which badges were added/removed and why these updates improve the README.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'docs: more icons in readme' accurately reflects the main change: adding badges/icons (AUR badge and Weblate shield) to the README file.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch readme-bells-and-whistles

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.

@amezin amezin force-pushed the readme-bells-and-whistles branch 4 times, most recently from 1940ef9 to bd757ef Compare January 29, 2026 12:34
@amezin amezin force-pushed the readme-bells-and-whistles branch from bd757ef to 0de5e68 Compare January 30, 2026 00:57
@amezin amezin force-pushed the readme-bells-and-whistles branch 5 times, most recently from 2e5ec1e to b81e87f Compare May 31, 2026 13:15
@amezin amezin marked this pull request as ready for review May 31, 2026 13:33

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

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 current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@README.md`:
- Line 24: Update the README tagline string "Another Drop Down Terminal
Extension for GNOME Shell" to use hyphenation for consistency by changing "Drop
Down" to "Drop-Down" so it reads "Another Drop-Down Terminal Extension for GNOME
Shell"; ensure only the tagline text is modified and preserve surrounding
punctuation and capitalization.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 4d7d32e1-76b3-497c-aeb6-c60065ce7c2d

📥 Commits

Reviewing files that changed from the base of the PR and between e3229c7 and b81e87f.

📒 Files selected for processing (1)
  • README.md
📜 Review details
🧰 Additional context used
🪛 LanguageTool
README.md

[grammar] ~24-~24: Use a hyphen to join words.
Context: ...ension-ddterm?logo=weblate Another Drop Down Terminal Extension for GNOME Shell ...

(QB_NEW_EN_HYPHEN)

🔇 Additional comments (1)
README.md (1)

7-10: LGTM!

Also applies to: 13-23, 26-26

Comment thread README.md Outdated
[Weblate languages badge]: https://hosted.weblate.org/widget/gnome-shell-extension-ddterm/language-badge.svg
[Weblate status badge]: https://img.shields.io/weblate/progress/gnome-shell-extension-ddterm?logo=weblate

Another Drop Down Terminal Extension for GNOME Shell

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 | ⚡ Quick win

Use hyphenation in the tagline for consistency and readability.

“Drop Down” should be “Drop-Down” in this sentence.

✏️ Proposed fix
-Another Drop Down Terminal Extension for GNOME Shell
+Another Drop-Down Terminal Extension for GNOME Shell
📝 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
Another Drop Down Terminal Extension for GNOME Shell
Another Drop-Down Terminal Extension for GNOME Shell
🧰 Tools
🪛 LanguageTool

[grammar] ~24-~24: Use a hyphen to join words.
Context: ...ension-ddterm?logo=weblate Another Drop Down Terminal Extension for GNOME Shell ...

(QB_NEW_EN_HYPHEN)

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@README.md` at line 24, Update the README tagline string "Another Drop Down
Terminal Extension for GNOME Shell" to use hyphenation for consistency by
changing "Drop Down" to "Drop-Down" so it reads "Another Drop-Down Terminal
Extension for GNOME Shell"; ensure only the tagline text is modified and
preserve surrounding punctuation and capitalization.

@amezin amezin force-pushed the readme-bells-and-whistles branch from b81e87f to 76d3cb8 Compare May 31, 2026 14:24

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

♻️ Duplicate comments (1)
README.md (1)

24-24: ⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Use hyphenation in the tagline for consistency and readability.

"Drop Down" should be "Drop-Down" in the tagline.

✏️ Proposed fix
-Another Drop Down Terminal Extension for GNOME Shell
+Another Drop-Down Terminal Extension for GNOME Shell
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@README.md` at line 24, Update the README tagline to use hyphenation for
consistency: change the string "Drop Down Terminal Extension for GNOME Shell" to
"Drop-Down Terminal Extension for GNOME Shell" so the tagline reads with a
hyphenated compound adjective.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Duplicate comments:
In `@README.md`:
- Line 24: Update the README tagline to use hyphenation for consistency: change
the string "Drop Down Terminal Extension for GNOME Shell" to "Drop-Down Terminal
Extension for GNOME Shell" so the tagline reads with a hyphenated compound
adjective.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: c1f2d73e-6d1d-4291-b310-76ecd64afd02

📥 Commits

Reviewing files that changed from the base of the PR and between b81e87f and 76d3cb8.

📒 Files selected for processing (1)
  • README.md
📜 Review details
⏰ Context from checks skipped due to timeout of 900000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (5)
  • GitHub Check: ci / lint
  • GitHub Check: ci / dist
  • GitHub Check: Analyze (javascript-typescript)
  • GitHub Check: Analyze (actions)
  • GitHub Check: Analyze (python)
🧰 Additional context used
🪛 LanguageTool
README.md

[grammar] ~24-~24: Use a hyphen to join words.
Context: ...ension-ddterm?logo=weblate Another Drop Down Terminal Extension for GNOME Shell ...

(QB_NEW_EN_HYPHEN)

🔇 Additional comments (4)
README.md (4)

13-13: LGTM!

Also applies to: 18-19


20-21: LGTM!


15-15: LGTM!

Also applies to: 22-22


8-8: README: Logo path is valid
data/com.github.amezin.ddterm.svg exists in the repository, so the README header logo should render correctly.

@amezin amezin force-pushed the readme-bells-and-whistles branch from 76d3cb8 to 58f087d Compare May 31, 2026 16:05
@amezin amezin merged commit 58f087d into master May 31, 2026
14 checks passed
@amezin amezin temporarily deployed to github-pages May 31, 2026 16:23 — with GitHub Actions Inactive
@amezin amezin deleted the readme-bells-and-whistles branch June 7, 2026 12:26
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.

1 participant