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

fixed #1508 : Root folders show nothing in recent projects list after the "-" (no path or drive name) #1601

Closed
wants to merge 2 commits into from

Conversation

torifat
Copy link

@torifat torifat commented Sep 8, 2012

I don't own a windows so didn't able to check it in windows. Also there are no tests available for this module.

…after the "-" (no path or drive name)

Signed-off-by: Rifat Nabi <to.rifat@gmail.com>
* 'master' of git://github.com/adobe/brackets: (22 commits)
  Use achored regexp
  Fix unit tests when run from symlinked directory
  Remove the local assignment to endQuote and directly use the quote character.
  Fix alt text for French version of Getting Started
  missing translation in CMD_LINE_DOWN
  Misc. improvements
  Automatically wrap the selected attribute value with double quotes if the begin quote is missing. Also update some unit tests for this functionality change.
  Merge branch 'master', remote-tracking branch 'origin' into rlim/code-hint-issues
  fix case in nls/strings.js module.exports object
  Replace "Brackets" with "{APP_NAME}", sync with nls/root/strings.js
  Typo: "forige" -> "forrige"
  Better usage messages. Check for XP.
  added no_content_bg.svg and modified the styles to display it.
  Windows scripts for setting up/restoring dev environment.
  Change about h2 margin-bottom to -4px
  Add tools directory.
  tweak about h2 spacing
  remove unused string
  working set files are not guaranteed to be openable, so check for failure
  Sync with root/strings.js
  ...
@ghost ghost assigned redmunds Sep 10, 2012
@RaymondLim
Copy link
Contributor

@torifat

Your changes fix the issue that I mentioned in #1508, but there is a related issue that @njx mentioned in the comment and that issue is not fixed yet. The related issue can be reproduced if you select the entire mac drive as your project folder. In that case, nj wants to show the project name as "/".

@ghost ghost assigned RaymondLim Sep 10, 2012
@torifat
Copy link
Author

torifat commented Sep 10, 2012

@RaymondLim Ok, I'll send a revised pull request 😄

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.

None yet

3 participants