Skip to content

copy on system buffer selected item name #132

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

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

copy on system buffer selected item name #132

wants to merge 6 commits into from

Conversation

MaksimDrobchak
Copy link
Contributor

@MaksimDrobchak MaksimDrobchak commented Feb 22, 2019

added opportunity to copy selected items to system buffer


const { items, selectedItems } = props;
this.state = {
selectedItems,
items,
filteredItems: items
};

this.refCmponent = React.createRef();
Copy link
Contributor

Choose a reason for hiding this comment

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

please remove

Copy link

@johanzilber johanzilber left a comment

Choose a reason for hiding this comment

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

Should we add it to readme?

@MaksimDrobchak
Copy link
Contributor Author

@johanzilber
Sure, done.

@liorheber
Copy link
Collaborator

Hi,

the problem with this solution is that it mandates that an item has a label which is not the case.
You should have the ability to customize the copy method (with documentation) otherwise you break the component for some.
Additionally, I would consider giving the ability yo toggle the feature off for people who are not yet ready to migrate to this feature.

@MaksimDrobchak
Copy link
Contributor Author

@liorheber
Sure, done.

# 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.

6 participants