Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Clean up native menu item creation #2709

Merged
merged 3 commits into from
Feb 1, 2013
Merged

Clean up native menu item creation #2709

merged 3 commits into from
Feb 1, 2013

Conversation

gruehle
Copy link
Member

@gruehle gruehle commented Jan 30, 2013

Fixes #2615

Re-organize and simplify the creation of native menu items. This also has the side effect of setting the initial enabled and checked states for menu items.

@ghost ghost assigned redmunds Jan 30, 2013
@redmunds
Copy link
Contributor

This looks good, but investigating Travis Error.

@jasonsanjose
Copy link
Member

FYI travis-ci/travis-ci#617

Looks like exactly what we're seeing with the git clone failure. According to the comments, it's a Github issue. Not sure there's anything we can do about it besides creating a new pull request and maybe a new branch?

The chain of events is weird since build #39 did pass but for some reason build #40 got kicked off but there were no following commits that would have triggered it.

@redmunds
Copy link
Contributor

redmunds commented Feb 1, 2013

Travis is happy. Merging.

redmunds added a commit that referenced this pull request Feb 1, 2013
Clean up native menu item creation
@redmunds redmunds merged commit 873cea7 into master Feb 1, 2013
@redmunds redmunds deleted the glenn/issue-2615 branch February 1, 2013 18:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Debug > Run Tests menu item should be disabled in installer builds
3 participants