-
Notifications
You must be signed in to change notification settings - Fork 488
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
7081 kebab file options icon btn #7353
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From a quick look, I think this would be ok to move to QA but there are conflicts in src/main/webapp/filesFragment.xhtml (and two pages in the guides) between this branch and the preview tools branch. So, I think we should let preview tools get merged before this gets to QA.
# Conflicts: # doc/sphinx-guides/source/developers/geospatial.rst # doc/sphinx-guides/source/user/find-use-data.rst # src/main/webapp/filesFragment.xhtml
Tested, this appears consistent across elements and works. |
Thanks @kcondon - let's wait on merging until we can investigate. @scolapasta can you take a quick look and see if there's anything obvious here? |
@djbrooke @scolapasta It was a shift+reload issue. Can merge at any time but maybe tomorrow. |
What this PR does / why we need it:
This PR adds a new fragment for edit buttons (and one for related popups) so that the UI code can be reused in multiple locations - currently, as of this PR used on: FilePage and DatasetPage, both for the kebabs and the selected files edit button.
Which issue(s) this PR closes:
Closes #7081
Special notes for your reviewer:
This is the infrastructure for this - some of the options are still specific and have logic, for example, to only render on the file page (e.g. replace and provenance). There are open issues to add those to DatasetPage (and one for file tags to be added to both kebabs and file page). Eventually, we will likely want to use this as well for the edit files page / fragment.
One thing I did for the new fragments is add a section at the top (with a ui:remove) that lists the different parameters that can be passed and/or are needed for usage of the fragment.
Suggestions on how to test this:
Make sure the correct edit functions appear and work in the 3 locations: File Page, Dataset Page kebab, and Dataset Page edit button for selected files.
Does this PR introduce a user interface change? If mockups are available, please link/include them here:
Yes. Any needed mockups should be in the issue.
Is there a release notes update needed for this change?:
no
Additional documentation: