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
Implement hash tag completion #3196
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
overall this is really cool and i like it!
return [null, null]; | ||
} | ||
|
||
word = word.trim().toLowerCase().slice(1); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why remove this?
will it make the matching case sensitive? do we want that?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought the hashtag would distinguish between uppercase and lowercase letters.
I found it to be all lowercase so fixed it.
resolve conflicts. |
6d9fe94
to
646dbb5
Compare
646dbb5
to
4685a2b
Compare
localStorage.setItem('hash_tag_history', data); | ||
} catch (e) { | ||
//ignore | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Storing large amounts of data in LocalStorage is inefficient. I'd advise using IDB instead via a library like Dexie. You can also use it for range queries, meaning you don't need to constantly re-parse a big string of JSON and then search through it using JavaScript.
@orekyuu ping |
Closing due to inactivity. |
This PR implements completion of hashtags.
The tooted hashtag is included in the candidate.
Candidates are stored localStorage.