Skip to content

remove Sci-fi theme#111

Merged
rubiin merged 1 commit intoHyDE-Project:masterfrom
zerythx:master
Apr 14, 2026
Merged

remove Sci-fi theme#111
rubiin merged 1 commit intoHyDE-Project:masterfrom
zerythx:master

Conversation

@zerythx
Copy link
Copy Markdown
Contributor

@zerythx zerythx commented Apr 14, 2026

Description

This PR removes the Sci-Fi theme from hyde-gallery.
The original repository for the theme was deleted or made private by the mantainer.

The PR will fix issues when:

*Updating themes on HyDE/scripts/install.sh
*Theme import with theme.import.py --select

Linked Issues

Additional context

  • PR title follows naming conventions (e.g., feat(theme):, fix(theme):)
  • PR description clearly explains the purpose of the changes
  • Related issues are linked (if any)
  • Screenshots are included
  • Screenshot size smaller than 4 MB (https://www.iloveimg.com/compress-image for compression)
  • Affected components/pages are listed
  • No unrelated changes are included

Summary by CodeRabbit

Updates

  • Removed the Sci-fi theme from the gallery
  • Updated preview images for multiple themes including Abyssal-Wave, BlueSky, Catppuccin Mocha, Code Garden, Decay Green, Edge Runner, Gruvbox Retro, Material Sakura, Nightbrew, Paranoid Sweet, Solarized Dark, and others
  • Adjusted theme numbering in the gallery to reflect changes

the original repository was deleted or made private by the maintainer
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 14, 2026

📝 Walkthrough

Walkthrough

The changes remove the "Sci-fi" theme from the gallery database and documentation. Theme numbering is adjusted accordingly, and gallery preview image references are updated across multiple theme entries.

Changes

Cohort / File(s) Summary
Gallery Configuration
README.md, hyde-themes.json
Removed "Sci-fi" theme entry and its documentation section. Renumbered subsequent themes (#55#63). Updated preview image filenames for multiple themes (Abyssal-Wave, BlueSky, Catppuccin Mocha, Code Garden, Decay Green, Edge Runner, Gruvbox Retro, Material Sakura, Nightbrew, Paranoid Sweet, Solarized Dark, and others).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

  • feat(theme): add Chisa theme #110: Modifies the same gallery files (README.md and hyde-themes.json) but adds the "Chisa" theme and updates previews/numbering instead of removing.

Suggested reviewers

  • Maroc02
  • saber-88

Poem

🐰 A Sci-fi dream fades to the stars,
One less theme in our gallery of art,
But the previews shine with fresh new light,
Numbers recount from wrong to right,
Hop hop, the gallery hops on! 🌟

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and concisely summarizes the main change: removing the Sci-fi theme from the gallery. It accurately reflects the primary objective of the pull request.
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

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
Contributor

@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 754: Update the description text that currently reads "Solarized Dark
based theme for HyDE" to use a hyphenated compound modifier; replace it with
"Solarized-Dark-based theme for HyDE" to improve readability in the README
description.
🪄 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: CHILL

Plan: Pro

Run ID: a454420f-d60a-4bec-9a22-a4cb9955f2e9

📥 Commits

Reviewing files that changed from the base of the PR and between bce0406 and 7d300b2.

⛔ Files ignored due to path filters (3)
  • Sci-fi/ss1.png is excluded by !**/*.png
  • Sci-fi/ss2.png is excluded by !**/*.png
  • Sci-fi/ss3.png is excluded by !**/*.png
📒 Files selected for processing (2)
  • README.md
  • hyde-themes.json
💤 Files with no reviewable changes (1)
  • hyde-themes.json

Comment thread README.md

**By:** [rishav12s](https://github.com/rishav12s)

**Description:** Solarized Dark based theme for HyDE
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Hyphenate the compound modifier in description text.

Line 754 should use a hyphenated form for readability (e.g., “Solarized-Dark-based theme for HyDE”).

✍️ Suggested wording tweak
-**Description:** Solarized Dark based theme for HyDE
+**Description:** Solarized-Dark-based theme for HyDE
🧰 Tools
🪛 LanguageTool

[grammar] ~754-~754: Use a hyphen to join words.
Context: ...hav12s) Description: Solarized Dark based theme for HyDE [![Solarized Dark ...

(QB_NEW_EN_HYPHEN)

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

In `@README.md` at line 754, Update the description text that currently reads
"Solarized Dark based theme for HyDE" to use a hyphenated compound modifier;
replace it with "Solarized-Dark-based theme for HyDE" to improve readability in
the README description.

@rubiin rubiin merged commit c58bc44 into HyDE-Project:master Apr 14, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants