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

feat(tag flyout menu): flyout menu implementation #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

AAPK-code
Copy link
Collaborator

No description provided.

Copy link
Collaborator

@Klimonov Klimonov left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@dmitryueno dmitryueno left a comment

Choose a reason for hiding this comment

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

All looks good to me, but I think we also need to have some kind of event, which will fire once flyout menu option was clicked!

@dmitryueno dmitryueno self-requested a review September 17, 2020 17:34
Copy link
Collaborator

@dmitryueno dmitryueno left a comment

Choose a reason for hiding this comment

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

LGTM!

Klimonov added a commit that referenced this pull request Sep 24, 2020
Klimonov added a commit that referenced this pull request Sep 28, 2020
Klimonov added a commit that referenced this pull request Oct 1, 2020
Klimonov added a commit that referenced this pull request Oct 1, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants