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.

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

workbench: renamed "folder" to "sub-directory" #678

Merged
merged 2 commits into from Jan 18, 2018
Merged

workbench: renamed "folder" to "sub-directory" #678

merged 2 commits into from Jan 18, 2018

Conversation

lpaulsen93
Copy link
Contributor

This PR changes the naming convention in the workbench plugin from "folder" to "sub-directory".

@b4n, @elextr: please give some feedback as you two participated in the mail thread about the naming convention for directories and folders.

@elextr
Copy link
Member

elextr commented Jan 18, 2018

LGBI. I didn't check the code part in detail, and I personally would only have changed the user facing parts, not the internal names, but you have done it now, so thats fine.

A couple of minor unrelated fixes to the readme noted.

workbench/README Outdated
@@ -85,7 +85,7 @@ These are the available items:
if you right clicked inside of a project. After selecting it a dialog
will be opened. Chosse the directory which shall be added. After that
Copy link
Member

Choose a reason for hiding this comment

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

Choose

workbench/README Outdated
@@ -101,18 +101,18 @@ These are the available items:
**Directory settings**
Select this item to change the directory settings. It is only available
if you right clicked inside of a project directory. In the directory
settings you can set a filter which controls the files and folders
settings you can set a filter which controls the files and sub-directories
that shall be displayed or not.
Copy link
Member

Choose a reason for hiding this comment

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

More idiomatic to say "will" in this context.

@lpaulsen93
Copy link
Contributor Author

@elextr: fixed README also, thanks for reporting the typos.

@frlan frlan merged commit 67ba90d into geany:master Jan 18, 2018
@lpaulsen93 lpaulsen93 deleted the wb-dir-names branch January 19, 2018 06:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants