Skip to content

Conversation

alfy91
Copy link

@alfy91 alfy91 commented Mar 11, 2019

This commit adds two new properties: maxLoadUsers and enableHighlight.
With the property maxLoadUser the component limits a number of elements to show. MaxLoadUser is an integer property, and when it sets at -1, it shows all elements.
With the property enableHighlight, a function adds to a div with contenteditable=true a span with class=mentioned and with text the value.

@fritx fritx self-assigned this Mar 13, 2019
@fritx
Copy link
Owner

fritx commented Mar 13, 2019

@piscitelli91 hey, thanks for you PR, great features, will try to merge ASAP

members: {
type: Array,
default: () => []
},
Copy link
Owner

Choose a reason for hiding this comment

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

@piscitelli91 I'm confused why the prop of members is deleted 😕 it will break the members passing in

Copy link
Author

Choose a reason for hiding this comment

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

You are right, i knought that the pull request send my branch at the date of pull request and not for future commits (I'm working on new functions that I will discuss with you).

I will create a new branch until this commit: 9196ffe
Sorry for mistake :)

Copy link
Owner

Choose a reason for hiding this comment

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

@piscitelli91 that would be cool, otherwise, I'm also trying to merge with fix if possible

computed: {
atItems () {
return this.at ? [this.at] : this.ats
return this.ats.map(at => at.symbol)
Copy link
Owner

@fritx fritx Mar 14, 2019

Choose a reason for hiding this comment

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

Obviously, change like this is something breaking, against all the versions before this, and will influenced all the users who use this library.

@agam-sb
Copy link

agam-sb commented May 18, 2022

Hey, is the highlighting feature added for mentioned user?
Thanks

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 this pull request may close these issues.

3 participants