Skip to content

feat: add One Piece and One Piece Zoro themes#107

Merged
pfeerick merged 13 commits into
EdgeTX:mainfrom
louiswaelkens2:new
Jun 21, 2026
Merged

feat: add One Piece and One Piece Zoro themes#107
pfeerick merged 13 commits into
EdgeTX:mainfrom
louiswaelkens2:new

Conversation

@louiswaelkens2

@louiswaelkens2 louiswaelkens2 commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

All Submissions:

  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Theme Submissions:

Have you included all the required elements?

  1. Have you included all the required elements in your theme?
  2. Are your screenshots in the correct resolution?

Summary by CodeRabbit

  • New Features
    • Added new “One Piece” theme with a comprehensive color palette for improved visual customization across the interface.
    • Added new “One Piece Zoro” theme configuration with its own metadata and matching color palette for consistent styling.

@coderabbitai

coderabbitai Bot commented Jun 18, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Adds THEMES/ONE_PIECE/theme.yml and THEMES/ONE_PIECE_ZORO/theme.yml, two new theme definitions. Each file includes summary metadata (name, author, info) and a colors map with 13 hex color entries for primary, secondary, focus, edit, active, warning, disabled, and quick-menu states.

Changes

One Piece Theme Variants

Layer / File(s) Summary
One Piece theme definitions
THEMES/ONE_PIECE/theme.yml, THEMES/ONE_PIECE_ZORO/theme.yml
Two new theme YAML files with metadata and matching colors palette structure (13 hex constants each).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description check ✅ Passed The PR description follows the template structure and confirms completion of all required checklist items for theme submissions, including required elements and screenshot resolution.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
Title check ✅ Passed The title accurately summarizes the main change: adding two theme files (One Piece and One Piece Zoro) to the repository.

✏️ 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.

@pfeerick pfeerick changed the title New Theme feat: add One Piece theme Jun 19, 2026
@pfeerick

Copy link
Copy Markdown
Member

@louiswaelkens2 Firstly, I really like the look of that background... very nice indeed!

There was no 320x240 (small screen) or 320x480 (portrait) backgrounds, so I got ChatGPT to do a content aware resize of the images to generate something, and added via e47b924. If you are happy with this, this can be merged as is. Otherwise, if you have a difference image or preference, we can go with that also. Entirely up to you.

@louiswaelkens2

Copy link
Copy Markdown
Contributor Author

Looks perfect to me, thanks for the nice comment. It would be an honor to have my design on the edgetx themes github.

@pfeerick pfeerick changed the title feat: add One Piece theme feat: add One Piece and One Piece Zoro themes Jun 21, 2026
@pfeerick
pfeerick merged commit bf3d14f into EdgeTX:main Jun 21, 2026
8 checks passed
@coderabbitai coderabbitai Bot mentioned this pull request Jul 3, 2026
3 tasks
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