Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Sort tags when displaying ; use AND for search results #2720

Merged
merged 2 commits into from
Oct 19, 2016

Conversation

ngotchac
Copy link
Contributor

Fixes #2719

@ngotchac ngotchac added A0-pleasereview 🤓 Pull request needs code review. M5-ui labels Oct 19, 2016
@GitCop
Copy link

GitCop commented Oct 19, 2016

There were the following issues with your Pull Request

  • Commit: 88eb724
    • Your subject line is longer than 72 characters

Guidelines are available at https://github.com/ethcore/parity


This message was auto-generated by https://gitcop.com

.map(searchValue => {
return values
.filter(value => value.indexOf(searchValue) >= 0)
.length > 0;
Copy link
Contributor

Choose a reason for hiding this comment

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

Use Array.prototype.some here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah, right!

Copy link
Contributor

@derhuerst derhuerst left a comment

Choose a reason for hiding this comment

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

looks very readable!

@GitCop
Copy link

GitCop commented Oct 19, 2016

There were the following issues with your Pull Request

  • Commit: 88eb724
    • Your subject line is longer than 72 characters

Guidelines are available at https://github.com/ethcore/parity


This message was auto-generated by https://gitcop.com

@jacogr jacogr added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Oct 19, 2016
@derhuerst derhuerst merged commit cf17041 into master Oct 19, 2016
@ngotchac ngotchac deleted the ng-ui-tags-fixes branch October 19, 2016 11:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants