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

[RFC] New foreground setting "strikethrough" #239

Merged
merged 1 commit into from
Jan 2, 2018

Conversation

geier
Copy link
Contributor

@geier geier commented Jun 21, 2017

After looking through the code again, I'm rather sure that this should be the way to go, see #236.

@and3rson
Copy link
Collaborator

and3rson commented Jan 2, 2018

Hi! Thanks for the contribution.

We're now trying to keep urwid alive and are reviewing all open pull requests.

I've recently fixed the build process. In order to properly re-run the build, can you please update your forked repo branch with fresh base upstream?

Thanks!

If you are unsure if your terminal supports this, try the following:

echo -e "\e[9mstrikethrough\e[0m"
@coveralls
Copy link

coveralls commented Jan 2, 2018

Coverage Status

Coverage decreased (-0.02%) to 75.892% when pulling 6933bed on geier:strikethrough into ab6bfbc on urwid:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 75.879% when pulling 6933bed on geier:strikethrough into ab6bfbc on urwid:master.

@geier
Copy link
Contributor Author

geier commented Jan 2, 2018

Done! And thank you for maintaining urwid!

@and3rson
Copy link
Collaborator

and3rson commented Jan 2, 2018

I can confirm that this works. Thanks!

@and3rson and3rson merged commit 6f94735 into urwid:master Jan 2, 2018
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.

None yet

3 participants