Skip to content

Adjusted order of entry's context menu#193

Merged
phoerious merged 2 commits intokeepassxreboot:developfrom
ryanrolds:context-menu-order
Jan 24, 2017
Merged

Adjusted order of entry's context menu#193
phoerious merged 2 commits intokeepassxreboot:developfrom
ryanrolds:context-menu-order

Conversation

@ryanrolds
Copy link
Copy Markdown
Contributor

@ryanrolds ryanrolds commented Jan 24, 2017

Changes context menu entry order so that commonly used actions are at the top.

Description

Reordered the context menu.

Motivation and Context

The mostly commonly used actions should be at the top.

How Has This Been Tested?

Built on a Ubuntu 16.04 workstation and hand tested the context menu

Types of changes

  • ✅ Bug fix (non-breaking change which fixes an issue)

Checklist:

  • ✅ I have read the CONTRIBUTING document. [REQUIRED]
  • ✅ My code follows the code style of this project. [REQUIRED]
  • ✅ All new and existing tests passed. [REQUIRED]

@TheZ3ro
Copy link
Copy Markdown
Contributor

TheZ3ro commented Jan 24, 2017

I think the perform-autotype action should be right after the CopyAttribute action, leaving Edit Clone Delete and New at the bottom

Copy link
Copy Markdown
Contributor

@TheZ3ro TheZ3ro left a comment

Choose a reason for hiding this comment

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

Looks good. 👍

@phoerious phoerious added this to the v2.2.0 milestone Jan 24, 2017
@phoerious phoerious merged commit 42c0815 into keepassxreboot:develop Jan 24, 2017
antongulenko pushed a commit to antongulenko/keepassxc that referenced this pull request Apr 18, 2017
antongulenko pushed a commit to antongulenko/keepassxc that referenced this pull request May 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants