Skip to content

Conversation

@gaelcool
Copy link

@gaelcool gaelcool commented Jan 20, 2026

Added a page for the waybar config in spanish (I am a native speaker). It was fun- think I learned a lot. Also removed some mentions of hyde-ctl to hyde-shell which works better for me and seems to be the preffered option throughout the documentation.

P.S: Thanks for considering these changes-

Summary by CodeRabbit

  • Documentation
    • Added comprehensive Spanish documentation for Waybar configuration.
    • Simplified Spanish installation guide, now recommending hyde-shell usage.
    • Updated theming guide: replaced older command examples with hyde-shell and clarified interactive theme/wallpaper workflows (rofi).
    • Minor wording improvement in the configuration guide for clearer step instructions.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 20, 2026

📝 Walkthrough

Walkthrough

Documentation updates refine English wording, replace hydectl usages with hyde-shell in theming guides, add a comprehensive Spanish Waybar configuration guide, and simplify the Spanish installation guide's second option to recommend hyde-shell. No code or public API changes.

Changes

Cohort / File(s) Summary
English Waybar doc
src/content/docs/en/configuring/waybar.md
Minor wording change in Step 6: "Finally, assign..." → "Now you can assign...". No code or config block changes.
English theming gallery
src/content/docs/en/theming/gallery.md
Replaced hydectl references with hyde-shell across import/select/update and wallpaper commands; updated command syntax/examples and headings to reference hyde-shell and rofi where applicable.
Spanish Waybar doc (new)
src/content/docs/es/configuring/waybar.md
Added full Spanish Waybar configuration guide: layouts, menus, modules, styles, includes, transient/generated files, group shapes, examples, HyDE integration, and step-by-step layout/group usage.
Spanish installation (edited)
src/content/docs/es/getting-started/installation.md
Simplified Option 2: removed prior HyDE-CLI caution and Hyde-install guidance; added concise recommendation to use hyde-shell and link to its manual.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 A hop through docs, both crisp and new,
Commands now whisper hyde-shell true,
Waybar en español, laid out with care,
Small wording changes, breezy as air,
I nibble the README and leave a bright clue. ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title accurately describes the main change: adding Spanish translation of waybar configuration documentation, which is the primary file addition in this PR.

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

✨ Finishing touches
  • 📝 Generate docstrings

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

@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: 2

🤖 Fix all issues with AI agents
In `@src/content/docs/es/configuring/waybar.md`:
- Around line 8-20: Remove the duplicate `user-style.css` entry in the
configuration tree inside the Waybar docs (the code block that lists
~/.config/waybar/). Edit the `configuring/waybar.md` code block so
`user-style.css` appears only once and the tree ends with a single terminal
entry; ensure no other filenames are accidentally duplicated (e.g., confirm
`theme.css`, `style.css` remain once).
- Around line 122-123: The link fragment for the radio group description is
wrong: replace the existing link target in the line containing
"[grupos](`#groups-grupos`)" so it points to the actual heading anchor
"#group-class-clase-grupal-para-estilizado" (i.e., change the fragment portion
only), ensuring the display text "grupos" remains unchanged; locate the string
"[grupos](`#groups-grupos`)" and update it to use the correct fragment
"#group-class-clase-grupal-para-estilizado".

@gaelcool gaelcool changed the title feat: translate waybar configuration to Spanish docs: translate waybar configuration to Spanish Jan 24, 2026
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