-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Splitting -- Placeholder folders (and files) in user directory for Ladspa, VSTs, themes and templates #1807
Comments
ty for cleaning up my original post :) |
I am not against a new tab, but I want to point out that you could possibly make the window scrollable if you think some settings belong to for example the path tab |
Thats a nice idea. It would give us room to expand with ease, and have mimimal impact, on both the user experience, and current code base. |
True... we could just put them at the bottom. In regards to the artwork, if you make a new button, please make it as ugly as possible. It makes people cringe and then they make a better one. |
Perfect! Haha.
I think the scroll area is just fine in this case. I think if we put the less used ancillary ones at the very bottom it is actually more effective then putting them on an unrelated Misc tab. 👍 |
@tresf reminds me of a saying or something from somewhere that I don't
quite remember. The general idea is that to get the answer to a question,
post an incorrect answer. This prompts people to correct you and gives you
the answer much faster than asking the question itself!
|
Small progress. I have wrapped the Path options in a scroll area, and adjusted sizes to suit.
It's so un obvious I don't know what ones can have multiple selections, or how. So i gess the questions are
|
Can:
Should:
Will when we add support for it:
Doesn't but may eventually:
Can't:
I would recommend a single folder icon for "set directory value" and a different icon (with a plus or multiple folder icon) for "add additional directory value". -Tres |
When we are referring to multiple selections, are we meaning the default locations + 1 user location, I currently cant find a way to set more than 1 LADSPA directory.
Would clicking on the "plus" icon, add an additional row to the table, with the heading duplicated? i.e adding a second vst directory would result in this list? working dir |
If you cannot by separating the paths with semicolon I guess it is not supported? |
Neither could #1170 ;) |
well we have a standard. we use comma separated lists. now a plus sign makes more sense. I think i might do a couple of mock up's for this, im having trouble visualizing this on the ui. |
@tresf 👍 |
Having experimented, currenlty if you click on the folder icon next to LADSPA, it adds it to a cvs list. should we keep this current behaviour, or add the plus button? |
Keep, but we should make it clear what is happening. I suggest we use the plural (PATHS or DIRECORIES) and make a modified browse button with a Plus indicator. If you'd rather not fiddle with the artwork, an ugly mockup should be sufficient until someone has time to make a new pixmap. 👍 -Tres |
Paths or Directories? |
Changed The Icon to show Directories that can have multiple values, currenly thats just LADSPA plugings. I will leave the creation of the user plugin directorys, until the plugin phase of this project. |
FoldersIf i suggest a file structure here, at least we have something to work from. lmms Are there any comments on this, what have I missed? |
The Besides that, the GUI looks great to me. As far as establishing places for this stuff moving forward, I'd really like @Umcaruje @Sti2nd @mikobuntu @grejppi @cubician's opinions. Once we pick these folder names, we're bound to stick with them for year to come, so opinions matter here... :) |
@tresf @curlymorphic I will have a good think about the folder names and the file structure, but for now yes I agree the "+" icon could do with being more noticeable and I would like the Plugins to actually have their own tab, this I will need to get out pen and paper and have a good think about , but basically " anything that is part of the sound creation should not be in the same tab as themes, installation directories, background artwork etc" imo ;) |
@curlymorphic"stops coding until design is completed" |
@curlymorphic " curlymorphic looks at the issue tracker and works on another bug " haha ;) |
@curlymorphic decides to atleast finish connecting the gig directory up before moving on |
Dave, thanks for staying on top of this. Please rebase the PR and I think we can merge it in. |
Please ignore the above question. :( |
The Directories are now all correctly added. The new locations are used in there relavant place, (sf2 player uses the sf2 folder), The new folders have been added to the path dialog in a scroll area. The only outstanding item is @mikobuntu redesign of the dialog, I feel maybe this should be a separate issue? If anyone wants to look
@tresf is it ok / desirable for me to post a test windows build here?' |
Please feel free to do to this for any items that you feel aren't getting the proper user feedback. It certainly helps, and then you can tag the Windows users. Just remember that this is a |
@curlymorphic if you are not sure where to upload the Windows builds to, I would recommend you upload them to https://github.com/curlymorphic/lmms/releases |
note. lmms will used currently save folders if you already have them set ( if your reading this im 99.999% sure you do ) so you may want to rename your |
download link leads to a 404 error |
@musikBear sorry. Maybe I didn,t set up the release properly. I think its done now |
@curlymorphic its fine now. i will install and come back 👍 ..Back :p Opend PR Are there any other tests we want done? |
thanks for doing this testing :)
this is based on master, that is still set to 1.1.0, I am assuming that once this forks into the 1.2 branch that will change.
At the moment yes, but that will get addressed by #1837
could be worth checking the sf2 player opens the correct folder, same for gig, please. I can finally feel a PR comming up :) |
👍 |
will do that later -only thing though -I dont have any 'gig' files, and i do not know where to get any.. so :| |
@musikBear cheers for the effort, The tests my end work ok, so im going to open a PR. |
@curlymorphic -Just ping me if you like more tests. Looking fwd to the PR 👍 |
@musikBear did you manage to download and test a -Tres |
Harh harh ha.. u 👹 |
Very similar to soundfonts. If you don't appreciate soundfonts, you won't appreciate Giga banks either.
I linked one for you that was in MB, not GB. #1807 (comment) |
@curlymorphic how are we with test results on this? Should we merge? -Tres |
So far this has only been tested by myself and @musikBear , My test releases seem to not get too much attention. I would say merge it, |
Tremendous Dave. This puts a big smile on my face. 🚀 |
:) |
This issue is to keep track of progress in #1541 as it covers a lot or areas. I am planing on breaking it down into the following areas
I plan on keeping this initial post updated
The text was updated successfully, but these errors were encountered: