search

brackets-archive / bracketsIssues

Archive of issues in brackets.
0 stars 0 forks source link

[CLOSED] Fix issue #993 #1008

Open core-ai-bot opened 3 years ago

core-ai-bot commented 3 years ago

Issue by RaymondLim Thursday Jun 07, 2012 at 23:04 GMT Originally opened as https://github.com/adobe/brackets/pull/1014

We were not initializing closed parent directories when saving the states of nodes in our project tree and I'm now initializing these directories to an empty array.

RaymondLim included the following code: https://github.com/adobe/brackets/pull/1014/commits

core-ai-bot commented 3 years ago

Comment by jasonsanjose Friday Jun 08, 2012 at 01:10 GMT

Nice find.

I think instead of trying to fill in empty arrays for depths that have no open folders, we should probably just skip .jstree-open nodes that aren't visible by changing line 223 to $(".jstree-open:visible"). This way, we'll never end up at the point where depth[3] is in preferences but depth[0] isn't.

core-ai-bot commented 3 years ago

Comment by RaymondLim Friday Jun 08, 2012 at 01:30 GMT

Nice suggestion! I was not aware of using the :visible selector to skip the entire directory if it is invisible.