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.

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

Add "Open Brackets Source" functionality #8859

Merged
merged 1 commit into from Aug 29, 2014

Conversation

Projects
None yet
4 participants
@MarcelGerber
Contributor

MarcelGerber commented Aug 25, 2014

This is for #8795. It adds Debug > Open Brackets Source, which opens the Brackets source code in Brackets.
It's the easiest possible solution - it just opens the symlink dir (like C:\Program Files (x86)\Brackets\dev on Windows).
The menu entry is only enabled in development builds and not in experimental (release) builds.

Since I heard of several problems with symlinks, it's no problem to me if you don't accept this PR.

@ingorichter

This comment has been minimized.

Show comment
Hide comment
@ingorichter

ingorichter Aug 27, 2014

Contributor

Looks good on OSX. I'm going to check on my Windows VM later today.

Contributor

ingorichter commented Aug 27, 2014

Looks good on OSX. I'm going to check on my Windows VM later today.

@Mark-Simulacrum

This comment has been minimized.

Show comment
Hide comment
@Mark-Simulacrum

Mark-Simulacrum Aug 28, 2014

Contributor

Opens fine on Ubuntu 14.04 (64-bit).

Contributor

Mark-Simulacrum commented Aug 28, 2014

Opens fine on Ubuntu 14.04 (64-bit).

@ingorichter

This comment has been minimized.

Show comment
Hide comment
@ingorichter

ingorichter Aug 29, 2014

Contributor

Looks good to me on Windows too. Thanks @MarcelGerber!

Contributor

ingorichter commented Aug 29, 2014

Looks good to me on Windows too. Thanks @MarcelGerber!

ingorichter added a commit that referenced this pull request Aug 29, 2014

Merge pull request #8859 from MarcelGerber/debug-brackets-source
Add "Open Brackets Source" functionality

@ingorichter ingorichter merged commit e9c5936 into adobe:master Aug 29, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details

@MarcelGerber MarcelGerber deleted the MarcelGerber:debug-brackets-source branch Aug 29, 2014

@peterflynn

This comment has been minimized.

Show comment
Hide comment
@peterflynn

peterflynn Oct 2, 2014

Member

I just realized this is fairly risky because it will hit #6692. Put up PR #9399 with a quick fix.

Member

peterflynn commented Oct 2, 2014

I just realized this is fairly risky because it will hit #6692. Put up PR #9399 with a quick fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment