[ZEPPELIN-846] Fix Long Notebook Dropdown

[corneadoug]

What is this PR for?

This PR is fixing the bug by implementing a new layout for the Notebook Dropdown menu.
Reason being that setting a maximum height and having a scrollbar doesn't mix with having bootstrap submenus.

In term of dropdown layout, I think we have 2 options:

• The one I implemented, a tree view similar to the one in the home page, I can probably uniform both of them later. One possible problem is the dropdown width getting bigger and bigger as we indent at each submenu.
• The second one would not have any indent, but use a background color to show elements in the same folder (A grey that goes clearer as we go in the submenus)

Reworking the whole notebook list and dropdown will probably need a lot more work later (especially in term of refactoring and code cleaning), so I'm not sure where the scope of this PR should stop, especially with the release coming. I think the layout I implemented is fine to solve the immediate problem, and it is always possible to switch to the layout of option 2 later if we think it is best.

What type of PR is it?

Hot Fix

Todos

• - Rebase & Double Check the filtering and Notebook ordering after [ZEPPELIN-1047] Add filter to handle upper case #1066

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-846

How should this be tested?

Just play with the folder structure and check the notebook dropdown

Screenshots (if appropriate)

Questions:

• Does the licenses files need update? No
• Is there breaking changes for older versions? No
• Does this needs documentation? No

[AhyoungRyu]

@corneadoug Awesome! I tested and it works really well 👍

[Leemoonsoo]

@corneadoug Tested and works well. Further work doesn't need to be addressed in this PR I think.

By the way, CI build is failing with error

INFO] src/components/expandCollapse/expandCollapse.directive.js
[INFO]   line 16  col 3   Missing "use strict" statement.
[INFO]   line 20  col 55  Strings must use singlequote.
[INFO]   line 21  col 54  Strings must use singlequote.
[INFO]   line 22  col 52  Strings must use singlequote.
[INFO]   line 24  col 46  Strings must use singlequote.
[INFO]   line 25  col 38  Strings must use singlequote.
[INFO]   line 31  col 10  Missing semicolon.
[INFO]   line 19  col 13  '$' is not defined.
[INFO]   line 20  col 18  '$' is not defined.
[INFO]   line 21  col 17  '$' is not defined.
[INFO]   line 22  col 17  '$' is not defined.
[INFO]   line 24  col 17  '$' is not defined.
[INFO]   line 25  col 19  '$' is not defined.
[INFO] 
[INFO] ✖ 13 problems

[corneadoug]

@Leemoonsoo Just fixed that and a few HTML problems when minified.
Jshint didn't run during dev mode which is a bit fishy.

[minahlee]

@corneadoug Can you rebase please?

[corneadoug]

@minahlee Rebased and ready

[minahlee]

Awesome!! Works like a charm. Merge if there is no more discussion