Skip to content

Add iceberg theme - #244

Merged
shugo merged 2 commits into
mainfrom
add-iceberg-theme
Apr 3, 2026
Merged

Add iceberg theme#244
shugo merged 2 commits into
mainfrom
add-iceberg-theme

Conversation

@shugo

@shugo shugo commented Apr 3, 2026

Copy link
Copy Markdown
Owner

Port of cocopon/iceberg.vim with both dark and light variants.

Port of cocopon/iceberg.vim with both dark and light variants.

Co-Authored-By: Claude Opus 4.6 <[email protected]>

Copilot AI left a comment

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.

Pull request overview

Adds a new built-in Iceberg theme (dark + light variants) to Textbringer by porting the palette and face mappings from cocopon/iceberg.vim.

Changes:

  • Introduce iceberg theme definition with separate :dark and :light palettes.
  • Define programming/UI/completion/dired faces for the theme and set default foreground/background colors.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread lib/textbringer/themes/iceberg.rb Outdated
p.color :green, hex: "#668e3d", ansi: "green" # PreProc/Special guifg
p.color :purple, hex: "#7759b4", ansi: "magenta" # Constant guifg
p.color :orange, hex: "#eac6ad", ansi: "yellow" # Search guibg
p.color :search_fg, hex: "#85512c", ansi: "yellow" # Search guifg

Copilot AI Apr 3, 2026

Copy link

Choose a reason for hiding this comment

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

In the light palette, both :orange and :search_fg resolve to the same ANSI color ("yellow"). When the terminal is in ANSI tier (<256 colors), :isearch will end up with yellow foreground on yellow background, making matches unreadable. Consider changing the ANSI mapping for either :search_fg (e.g., to "black"/"brightblack") or :orange so that :isearch always has sufficient contrast in ANSI tier.

Suggested change
p.color :search_fg, hex: "#85512c", ansi: "yellow" # Search guifg
p.color :search_fg, hex: "#85512c", ansi: "black" # Search guifg

Copilot uses AI. Check for mistakes.
Change search_fg ansi from "yellow" to "black" in the light palette
to avoid same-color foreground/background in ANSI tier.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@shugo
shugo enabled auto-merge April 3, 2026 07:49
@shugo
shugo merged commit 9d8d760 into main Apr 3, 2026
7 checks passed
@shugo
shugo deleted the add-iceberg-theme branch April 3, 2026 07:51
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