Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Tool[s] menu reorganised and renamed #2430

Merged
merged 1 commit into from
Feb 2, 2023
Merged

Tool[s] menu reorganised and renamed #2430

merged 1 commit into from
Feb 2, 2023

Conversation

lucas-wilkins
Copy link
Contributor

Description

  1. Renamed "Tool" menu to "Tools"
  2. Some re-ordering of menu contents

Fixes #2412

How Has This Been Tested?

I ran it and the menu looked like I specified in QtDesigner

Review Checklist (please remove items if they don't apply):

  • Functionality has been tested
  • Windows installer (GH artifact) has been tested (installed and worked)
  • MacOSX installer (GH artifact) has been tested (installed and worked)
  • User documentation is available and complete (if required)

@lucas-wilkins lucas-wilkins added the Enhancement Feature requests and/or general improvements label Feb 2, 2023
@lucas-wilkins lucas-wilkins linked an issue Feb 2, 2023 that may be closed by this pull request
Tool menu re-organisation #2412
Closed
@lucas-wilkins lucas-wilkins marked this pull request as ready for review February 2, 2023 15:31
rozyczko
rozyczko approved these changes Feb 2, 2023
View reviewed changes
Copy link
Member

@rozyczko rozyczko left a comment

Choose a reason for hiding this comment

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

code looks good, GUI changes make sense.

@krzywon krzywon merged commit e06a210 into main Feb 2, 2023
@krzywon krzywon deleted the 2412-tool-menu-re-organisation branch February 2, 2023 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@krzywon krzywon krzywon approved these changes

@rozyczko rozyczko rozyczko approved these changes

@smk78 smk78 Awaiting requested review from smk78

Assignees
No one assigned
Labels
Enhancement Feature requests and/or general improvements
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Tool menu re-organisation
3 participants
@lucas-wilkins @rozyczko @krzywon