Skip to content
This repository was archived by the owner on May 12, 2026. It is now read-only.

added the context menu to the list view#1036

Merged
kazup01 merged 1 commit into
BoostIO:masterfrom
mslourens:context-menu
Oct 30, 2017
Merged

added the context menu to the list view#1036
kazup01 merged 1 commit into
BoostIO:masterfrom
mslourens:context-menu

Conversation

@mslourens

Copy link
Copy Markdown
Contributor

Fixes #1034

I copied the code from the NoteItem context menu and added that to the SimpleItem. I couldn't find any tests for this behavior, so I left it for now.

@kazup01 kazup01 self-requested a review October 30, 2017 15:34

@kazup01 kazup01 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@kazup01 kazup01 merged commit 73b5546 into BoostIO:master Oct 30, 2017
@kazup01

kazup01 commented Oct 30, 2017

Copy link
Copy Markdown
Member

Merged. Thank you for your contribution @mslourens 🎉

@mslourens mslourens deleted the context-menu branch October 30, 2017 16:20
@kohei-takata kohei-takata mentioned this pull request Nov 25, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

3 participants