Skip to content

Conversation

@curlymorphic
Copy link
Contributor

Proposed fix for #1150 Preset Browser: division between factory and user files fails if factory has folders

image

Simply added a counter to the Directory class. that is incremented for every child directory added.

Then used this counter to calc the position in tree of --Factory Presets --

@tresf
Copy link
Member

tresf commented Dec 29, 2014

Nice, simple and sweet. I'll let @diizy merge this one.

@tresf
Copy link
Member

tresf commented Jan 10, 2015

Safe initialization of the new variable, good counter handling. Merging.

tresf added a commit that referenced this pull request Jan 10, 2015
Proposed fix for 1150 Preset Browser: division between factory and user ...
@tresf tresf merged commit e61a0d1 into LMMS:master Jan 10, 2015
@tresf
Copy link
Member

tresf commented Apr 6, 2015

@curlymorphic can you take a look at #1925? It looks very closely related to the code in this PR (probably not caused by anything you did, just reaching out for help) :)

@curlymorphic
Copy link
Contributor Author

sure :)

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.

2 participants