Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support strikethrough text style #580

Closed
Tyriar opened this issue Mar 3, 2017 · 11 comments · Fixed by #3386
Closed

Support strikethrough text style #580

Tyriar opened this issue Mar 3, 2017 · 11 comments · Fixed by #3386
Assignees
Labels
Milestone

Comments

@Tyriar
Copy link
Member

Tyriar commented Mar 3, 2017

I notice these aren't implemented:

@Tyriar Tyriar added the type/enhancement Features or improvements to existing features label Mar 3, 2017
@Tyriar Tyriar added this to the 2.4.0 milestone Mar 3, 2017
@parisk parisk modified the milestone: 2.4.0 Mar 7, 2017
@lucat1
Copy link
Contributor

lucat1 commented Jul 20, 2017

Is this implemented as of today?
The css is here, but for me it's not working inside of vim, macOS

@Tyriar
Copy link
Member Author

Tyriar commented Jul 20, 2017

It's not implemented, what CSS are you referring to?

@lucat1
Copy link
Contributor

lucat1 commented Jul 21, 2017

Sorry for the CSS part, I thought to have seen it there

Are you planning on implementing this soon or is this a low priority task?

@Tyriar
Copy link
Member Author

Tyriar commented Jul 21, 2017

Those are for the ones that are implemented.

This is low priority and is blocked by #756 which is reimplementing how the character attributes are stored.

@mofux
Copy link
Contributor

mofux commented Oct 9, 2017

Here is a nice script to test the different styles:

https://gist.githubusercontent.com/mxmerz/92e97cd27857a9ba787b/raw/c16c216d11c3fa6e7f0254e2d5f485164cddfb96/ANSI-Test.sh

@lucat1
Copy link
Contributor

lucat1 commented Oct 10, 2017

Cool! 😍

@Tyriar Tyriar added help wanted type/feature type/enhancement Features or improvements to existing features and removed type/enhancement Features or improvements to existing features type/feature labels Apr 4, 2018
@flybayer
Copy link
Contributor

flybayer commented Apr 26, 2018

Oh man, I want this so bad (italic).
It looks like this is no longer blocked by anything, correct?

@Tyriar
Copy link
Member Author

Tyriar commented May 4, 2018

Italics fixed in #1422

@Tyriar Tyriar changed the title Add support for dim, italics and strikethrough text styles Support strikethrough text style Jun 1, 2018
@felixfbecker
Copy link

Strike through would be really great. Git supports it as a diff style: git config --global diff.color.old "red strike"

PowerGit uses it for gone branches:

image

@Tyriar
Copy link
Member Author

Tyriar commented May 11, 2019

If anyone wants to help out with this you will want to copy how underline works in the DOM (DomRendererRowFactory.ts) and canvas renderers (TextRenderLayer.ts), for the latter a new method like fillBottomLineAtCells will need to be done which draws a line through the middle.

@Tyriar
Copy link
Member Author

Tyriar commented Jul 9, 2021

DOM and canvas renderer support is merged thanks to @silamon, webgl is still outstanding. It's a little harder there since we don't do underline yet either #2251

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants