Skip to content
New issue

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

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

Already on GitHub? # to your account

github-files-filter : strikethrough applied filter names #68

Merged
merged 1 commit into from
Jan 12, 2019

Conversation

darkred
Copy link
Collaborator

@darkred darkred commented Jan 11, 2019

In continuation to your commit in #46 i.e. changing the capitalization as you toggle the filters,
my suggested addition is to also add strikethrough to the toggled filters (like in my PR #59 for 'Toggle Issue Comments') The reason is because I was still finding it difficult to distinguish which filters are disabled.

Screenshot comparison:

Before

1

After

2

 

I hope you like my suggestion, @Mottie 🙂

@Mottie Mottie merged commit fe8027b into master Jan 12, 2019
@Mottie
Copy link
Owner

Mottie commented Jan 12, 2019

Hi @darkred!

Yeah, it looks good. Thanks!

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

Successfully merging this pull request may close these issues.

2 participants