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

Added names of elements in context menu #16449

Merged
merged 2 commits into from
Aug 29, 2023

Conversation

Ruslan-Aleev
Copy link
Collaborator

What does it do?

Added names of elements in tree. Now it takes less clicks to find out the key / id of an element in the tree + uniformity in the interface.

  1. Added a source name to context menu.

Before:
src_tree_1

After:
src_tree_name

  1. Added names of elements (directory and file) to their context menu. Now it is clear for which directory / file the context menu is called.

Before:
dir_tree_1
file_tree_1

After:
dir_tree_2
file_tree_2

Why is it needed?

UX improvements

Related issue(s)/PR(s)

#15922

@cla-bot cla-bot bot added the cla-signed CLA confirmed for contributors to this PR. label Jun 18, 2023
@Ruslan-Aleev Ruslan-Aleev added type-frontend Issues related to UI/UX issues, mostly about styles and frontend implementations on JavaScript. pr/review-needed Pull request requires review and testing. labels Jun 18, 2023
@Ruslan-Aleev Ruslan-Aleev force-pushed the 3.x-trees-names branch 3 times, most recently from e501716 to 4f83dca Compare August 7, 2023 14:07
@Ibochkarev Ibochkarev added pr/ready-for-merging Pull request reviewed and tested and ready for merging. and removed pr/review-needed Pull request requires review and testing. labels Aug 8, 2023
@opengeek opengeek added this to the v3.1.0 milestone Aug 29, 2023
@opengeek opengeek merged commit 56bdb13 into modxcms:3.x Aug 29, 2023
6 checks passed
@Ruslan-Aleev Ruslan-Aleev deleted the 3.x-trees-names branch October 16, 2023 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed CLA confirmed for contributors to this PR. pr/ready-for-merging Pull request reviewed and tested and ready for merging. type-frontend Issues related to UI/UX issues, mostly about styles and frontend implementations on JavaScript.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants