[WIP] Adding Tree2D to the UI sub-module

[antrikshmisri]

This PR adds Tree2D UI to the ui sub-module.

[pep8speaks]

Hello @antrikshmisri! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

• In the file docs/tutorials/02_ui/viz_tree_actor_modifier.py:

Line 152:1: E305 expected 2 blank lines after class or function definition, found 1

• In the file fury/ui/elements.py:

Line 3512:80: E501 line too long (84 > 79 characters)

Comment last updated at 2021-08-24 18:39:08 UTC

[codecov]

Codecov Report

Merging #460 (62e47fa) into master (0f97a6d) will increase coverage by 0.07%.
The diff coverage is 89.08%.

❗ Current head 62e47fa differs from pull request most recent head d568836. Consider uploading reports for the commit d568836 to get more accurate results

@@            Coverage Diff             @@
##           master     #460      +/-   ##
==========================================
+ Coverage   88.53%   88.60%   +0.07%     
==========================================
  Files          31       31              
  Lines        6575     6968     +393     
  Branches      787      838      +51     
==========================================
+ Hits         5821     6174     +353     
- Misses        535      564      +29     
- Partials      219      230      +11     

Impacted Files	Coverage Δ
fury/ui/elements.py	88.63% <88.37%> (+0.09%)	⬆️
fury/ui/tests/test_elements.py	84.52% <91.39%> (+1.29%)	⬆️
fury/ui/containers.py	84.03% <0.00%> (+0.22%)	⬆️

[skoudoro]

Hi @antrikshmisri,

I have few questions:

• is it normal to be able to select (highlight) multiple elements. if yes, can we disable it easily?
• When you click on the text, you can not select, can you allow the same behavior as when we click outside the text (same comment for the icon)
• the file test.py should be an additional tutorial so you will need to add an explanation and put it in the right folder

[antrikshmisri]

Hey @skoudoro ,

• is it normal to be able to select (highlight) multiple elements. if yes, can we disable it easily?

I think it totally depends on the use case, some should allow multiselecting and other shouldn't. Nonetheless, I think this feature should be added. All the changes along with this feature will be added with the latest commit.

[antrikshmisri]

Hey @skoudoro , all the comments have been addressed as well as the multiselect feature has also been added.

[antrikshmisri]

Hey @skoudoro, all the proposed changes have been added. PTAL.

[skoudoro]

Hi @antrikshmisri, Before starting to review this PR, Can you fix all the conflict, rebase it and address the small pep8 above?

Thank you

[ganimtron-10]

Hey @antrikshmisri ,
Can you please rebase this PR?

[ganimtron-10]

Closing this PR here and continuing at #821 as we got no response from author.

[antrikshmisri]

Sorry @ganimtron-10 for not being active, I am a bit tied up right now. Let me know if you need any context/help.