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

Re organizing of the user LMMS directory #1908

Merged
merged 6 commits into from
Apr 13, 2015
Merged

Re organizing of the user LMMS directory #1908

merged 6 commits into from
Apr 13, 2015

Conversation

curlymorphic
Copy link
Contributor

Have updated the users LMMS directory to contain the following folders

lmms
____projects
____templates
____samples
________gig
________sf2
____presets
____themes
____plugins
________ladspa
________vst

Added the extra folders to the settings dialog , Added a Scroll area to the dialog to allow the extra space needed.

Updated the icon where multiple Folders are allowed.
ArtWork renamed to themes

Opening of correct folders where needed, SF2 player opens the sf folder, GIG player opens the gig folder.

fixes #1807

Wrapped the path selection widgets in a Scroll Area, to allow for
the addition of addition paths.

Where multipue paths are allowed the icon has been changed to refect this.

Reordered the list.
Added option to set gig folder in setup dialog
Gig player now opens at this location
added add_folder.png icon to use for adding addition folders.
this may need attention by others

changed layout to correctly display scroll bar
added a default path for sf
create folder
add folder to setup dialog
sf player now uses this location
the user vst folder nowbecome the default vstfolder. this is configurable
in the setup dialog.

The user ladspa folder is added to the list of folder to search
@@ -425,6 +441,11 @@ void ConfigManager::loadConfigFile()
QDir().mkpath( userProjectsDir() );
QDir().mkpath( userSamplesDir() );
QDir().mkpath( userPresetsDir() );
QDir().mkpath( userGigDir() );
QDir().mkpath( userSf2Dir() );
QDir().mkpath( vstDir() );
Copy link
Member

Choose a reason for hiding this comment

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

Noticed the naming convention of this vstDir() vs. usersVstDir()... also many Windows users historically have the vstDir() set to a restricted area (such as Program Files/Steinberg/). Does mkpath(...) fail silently if the area can't be written to?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The directories are only created once, so there will be a user Lmms/plugin/vst folder created when the the user lmms folder in initially created.

If/ when the user sets it to another location ie Program Files/Steinberg/ no attempt is made to create or write to that location.

Copy link
Member

Choose a reason for hiding this comment

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

If/ when the user sets it to another location ie Program Files/Steinberg/ no attempt is made to create or write to that location.

Ok. :)

@tresf
Copy link
Member

tresf commented Mar 30, 2015

This isn't your fault, but we have some naming convention conflicts in our code, i.e. usersVstDir() vs. openVSTDir(). This may be the time to re-factor this. What do you think?

-Tres

@tresf
Copy link
Member

tresf commented Mar 30, 2015

@curlymorphic this is great. Since it is a significant change, I want a few set of eyes before merging it. @badosu, @lukas-w any objections?

@curlymorphic
Copy link
Contributor Author

This isn't your fault, but we have some naming convention conflicts in our code, i.e. usersVstDir() vs. openVSTDir(). This may be the time to re-factor this. What do you think?

will have a look at this later today

Added a userVstDir function for consistancy
use the userVstDir() function to create new folder
@curlymorphic
Copy link
Contributor Author

This isn't your fault, but we have some naming convention conflicts in our code, i.e. usersVstDir() vs. openVSTDir(). This may be the time to re-factor this. What do you think?

Added userVstDir() to configManager.h for consistancy.

@Spekular
Copy link
Member

Spekular commented Mar 31, 2015 via email

@tresf
Copy link
Member

tresf commented Mar 31, 2015

where do "actual" (wav) samples go in this? It seems they go under
samples

Yes, samples.

@tresf
Copy link
Member

tresf commented Mar 31, 2015

Maybe there ought to be a subfolder specifially for
audio samples as well? Sorry if I'm late to the discussion.

Most people already put them in subfolders, you may be overthinking this..

@Spekular
Copy link
Member

Spekular commented Mar 31, 2015 via email

@tresf
Copy link
Member

tresf commented Mar 31, 2015

Then don't. What do you propose? samples/samples? I think too many root folders is much more confusing than a few subfolders.

@curlymorphic
Copy link
Contributor Author

I think too many root folders is much more confusing than a few subfolders.

I agree, we have to be careful to still give users there freedom.

@Spekular
Copy link
Member

Spekular commented Mar 31, 2015 via email

@Sti2nd
Copy link
Contributor

Sti2nd commented Mar 31, 2015

Nah, leave it. SF2 and GIG was decided as samples. Samples is audio, no difference :)

@tresf
Copy link
Member

tresf commented Apr 13, 2015

Merging per organization feedback and researching #1807. 👍

Updated release notes:

If we encounter issues, we'll open new bugs. :)

tresf added a commit that referenced this pull request Apr 13, 2015
Re organizing of the user LMMS directory
@tresf tresf merged commit d516c74 into LMMS:master Apr 13, 2015
@curlymorphic curlymorphic deleted the pathDialog branch July 5, 2015 18:56
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.

4 participants