Skip to content

Conversation

@aymanbagabas
Copy link
Member

@aymanbagabas aymanbagabas commented Mar 25, 2025

This drops the dependency on termenv and replaces it with lipgloss/v2. It breaks the API by dropping termenv colorprofiles and removing the WithColorProfile option.

This drops the dependency on termenv and replaces it with lipgloss/v2.
It breaks the API by dropping termenv colorprofiles and removing the
`WithColorProfile` option.
This drops muesli/reflow in favor of custom IndenWriter and
PaddingWriter. It also uses cellbuf.Wrap instead of wordwrap.NewWriter
to keep track of style and link changes when wrapping.

Fixes: #149
Related: #237
@aymanbagabas aymanbagabas marked this pull request as ready for review March 27, 2025 15:26
Copy link
Member

@andreynering andreynering left a comment

Choose a reason for hiding this comment

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

This will probably add some conflicts to #406, but I can handle those.

aymanbagabas and others added 5 commits March 27, 2025 20:50
* refactor(ansi): drop reflow and use cellbuf.Wrap

This drops muesli/reflow in favor of custom IndenWriter and
PaddingWriter. It also uses cellbuf.Wrap instead of wordwrap.NewWriter
to keep track of style and link changes when wrapping.

Fixes: #149
Related: #237

* fix(ansi): remove unused overline style

* chore: update testdata
Signed-off-by: Carlos Alexandro Becker <[email protected]>
This adds support for OSC 8 hyperlinks. It enables wrapped urls to be
clickable and hoverable in the terminal.
This purify the Glamour API by removing the auto style feature and making
the dark style the default. Glamour no longer needs to query the terminal
background to determine the style to use. Querying the terminal is now
part of Lip Gloss v2.
@andreynering
Copy link
Member

@aymanbagabas FYI, I just merged the table links work into this branch.

Copy link

@timscharfenort8 timscharfenort8 left a comment

Choose a reason for hiding this comment

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

Good work

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.

7 participants