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

AddComponentCommand should use the active project name as a default folder name #4016

Closed
mansellan opened this issue May 16, 2018 · 0 comments
Closed

AddComponentCommand should use the active project name as a default folder name #4016

mansellan opened this issue May 16, 2018 · 0 comments
Labels
difficulty-01-duckling Issue where no particularly involved knowledge of the internal API is needed. edge-case Marks a bug as an edge case that would be nice to fix, but that likely doesn't affect normal use. feature-code-explorer good first issue Want to contribute? That's a good place to start! has-workaround There is some way of working around this limitation / bug that is confirmed to work up-for-grabs Use this label in conjunction with a difficulty level label, e.g. difficulty-02-ducky

Comments

@mansellan
Copy link
Member

mansellan commented May 16, 2018 •
edited

MCVE:

  1. Open Excel VBE on an empty workbook
  2. Using PE, rename VBAProject to 'Test'
  3. In CE, without selecting any node, Click Add Module from the toolbar
  4. Module will be created in folder 'VBAProject', not 'Test'

image

Absent a selected node, the default folder should be based on the name of the active project. Spotted because it's more apparent in VB6, which has a different (and varying) default project name. In VBA this is edge-case.
@mansellan mansellan added feature-code-explorer has-workaround There is some way of working around this limitation / bug that is confirmed to work edge-case Marks a bug as an edge case that would be nice to fix, but that likely doesn't affect normal use. difficulty-01-duckling Issue where no particularly involved knowledge of the internal API is needed. up-for-grabs Use this label in conjunction with a difficulty level label, e.g. difficulty-02-ducky good first issue Want to contribute? That's a good place to start! labels May 16, 2018
Imh0t3b added a commit to Imh0t3b/Rubberduck that referenced this issue Jul 10, 2018
@Imh0t3b
Issue rubberduck-vba#4016 AddComponentCommand should use the active p…
6ce76af 
…roject name as a default folder name
@Imh0t3b Imh0t3b mentioned this issue Jul 10, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
difficulty-01-duckling Issue where no particularly involved knowledge of the internal API is needed. edge-case Marks a bug as an edge case that would be nice to fix, but that likely doesn't affect normal use. feature-code-explorer good first issue Want to contribute? That's a good place to start! has-workaround There is some way of working around this limitation / bug that is confirmed to work up-for-grabs Use this label in conjunction with a difficulty level label, e.g. difficulty-02-ducky
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mansellan