You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe.
The current method for passing the Wish renderer to the various styles in huh is clunky, in that you call one of the theme functions (like huh.ThemeBase()), then go back and replace the renderer anywhere there are colors, as shown:
In addition to being awkward, it also means that any new styles added to huh must be added to the code that calls ThemeBase() and then swaps out styles.
Describe the solution you'd like
A new set of functions that accept a renderer should be added (huh.ThemeBaseWithRenderer(), huh.ThemeCharmWithRenderer, etc), and then the current functions can simply pass lipgloss.DefaultRenderer() to the new functions, such as:
Is your feature request related to a problem? Please describe.
The current method for passing the Wish renderer to the various styles in huh is clunky, in that you call one of the theme functions (like
huh.ThemeBase()
), then go back and replace the renderer anywhere there are colors, as shown:huh/examples/ssh-form/main.go
Lines 78 to 93 in dad7190
In addition to being awkward, it also means that any new styles added to huh must be added to the code that calls
ThemeBase()
and then swaps out styles.Describe the solution you'd like
A new set of functions that accept a renderer should be added (
huh.ThemeBaseWithRenderer()
,huh.ThemeCharmWithRenderer
, etc), and then the current functions can simply passlipgloss.DefaultRenderer()
to the new functions, such as:The text was updated successfully, but these errors were encountered: