Skip to content
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

Move dialog - Show tree structure inside the content selector #4413

Closed
alansemenov opened this issue Feb 14, 2017 · 2 comments
Closed

Move dialog - Show tree structure inside the content selector #4413

alansemenov opened this issue Feb 14, 2017 · 2 comments
Assignees

Comments

@alansemenov
Copy link
Member

alansemenov commented Feb 14, 2017

By default, content items inside the content selector should be tree-structured with the same order as in the Content Grid. When the filter applied, the selector should switch to flat structure (like now).

The core of this task is to implement a new class DropdownTreeGrid which will then be used inside the combobox (if certain config parameter is specified).

image

image

@alansemenov alansemenov modified the milestone: Sprint 194 Feb 14, 2017
@alansemenov
Copy link
Member Author

Great job! Some things to fix:

  1. The first time I open the dropdown, scrolling is not working. If I collapse and expand again, it works fine.
  2. If I search by keyword, then select a node, then click "x" to delete selection, the keyword remains in the field. We should clear it after clicking "x".
  3. Would be convenient to auto-expand the tree down to selected item, and highlight selected item in the dropdown. This will help a lot if I want to move the item one level up, for example, or to a sibling folder.

@sshklianko
Copy link
Contributor

sshklianko commented Feb 28, 2017

Could not reproduce 1. even with browserstack emulation mode/
2. Is implemented now.
3. I did not add it yet - let's create separate task for that as this task has already taken lots of time.

alansemenov added a commit that referenced this issue Feb 28, 2017
alansemenov added a commit that referenced this issue Feb 28, 2017
…g of nodes inside the dropdown. Small styling fixes. (#4413)
@srs srs removed this from the Sprint 194 milestone Mar 7, 2017
@srs srs closed this as completed Mar 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants