Skip to content

limit show elements in list and highlight of user mentioned - #91

Open
alfy91 wants to merge 4 commits into
fritx:devfrom
alfy91:dev
Open

limit show elements in list and highlight of user mentioned#91
alfy91 wants to merge 4 commits into
fritx:devfrom
alfy91:dev

Conversation

@alfy91

@alfy91 alfy91 commented Mar 11, 2019

Copy link
Copy Markdown

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

fritx commented Mar 13, 2019

Copy link
Copy Markdown
Owner

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

Comment thread src/At.vue
members: {
type: Array,
default: () => []
},

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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

Comment thread src/At.vue
computed: {
atItems () {
return this.at ? [this.at] : this.ats
return this.ats.map(at => at.symbol)

@fritx fritx Mar 14, 2019

Copy link
Copy Markdown
Owner

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

agam-sb commented May 18, 2022

Copy link
Copy Markdown

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