Details
-
Type:
Task
-
Status: Closed (View Workflow)
-
Priority:
Major
-
Resolution: Done
-
Affects Version/s: None
-
Fix Version/s: 10.2.0
-
Labels:None
-
Story Points:3
-
Epic Link:
-
Sprint:Spring 4, Spring 5, Spring 6
Description
Investigate how to do the below improvements and implement them in the Genome Dynamic Search UI:
- Enable sorting of the column names, for example alphabetically sorting by "Scientific Name".
- Move the "Clear Search" button to the top of the app to the right of the search bar.
- Increase the default number of rows visible to 100 (Make the rows of the table scrollable).
Attachments
Issue Links
- relates to
-
-
- Closed
-
Activity
After investigating, found that changing this to a new menu item might take a couple of weeks, so we will focus on the actual loading functionality first and then consider this again. Closing this ticket, will do the other UI changes in a new ticket after the loading functionality is implemented. Also, the changes that are part of this ticket are reviewed.
Notes from our discussion:
- Consider adding a new menu called "Genome Search" in the top menu bar.
- In this, we might add a menu item for Search to add searching functionality for our existing genomes.
- When users install the external data provider apps (like ucsc-genark), a new menu item will be added, eg, "GenArk Genomes"
- Upon clicking that menu item, a new window will be opened that loads the dynamic search UI for the external data provider similar to ProtAnnot.
Raised PR for the changes in IGB dynamic search code: https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/1067. For the changes in ucsc-genark app, as we aren't pushing to loraine lab yet, I have pushed it to my workspace.
PR is merged. New installers are built and deployed to BioViz.org early access section.
Ready for final testing.
Testing main branch on Mac.
- Clear Search button works, located next to search bar.
- Default number of rows set to 100.
- Sorting by column names alphabetically working, but I have one suggestion.
- The sorting is splitting the results into upper case and lower case such that all of the results that are uppercase are shown alphabetically, then all of the lowercase results are shown alphabetically. Would be better to ignore case when sorting alphabetically.
Nowlan Freese Thanks for pointing that out, changed the sorting to case-insensitive sorting now. Please review and let me know if there are any other issues.
Sorting is now case-insensitive.
Closing ticket.
add the total no.of genomes and the page number out of the total pages, change the Previous and next buttons to arrows. Remove load button and have single load.
Investigate adding this as a new menu item and opening it as a window.