HP-255 HEAL UI updates with adaptations to ecosystems UI #876
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
williamhaley
force-pushed
the
feat/heal
branch
3 times, most recently
from
30b0306
to
4892a7a
Compare
williamhaley
changed the title
mfshao
reviewed
Jul 8, 2021
mfshao
reviewed
Jul 8, 2021
|
@mfshao Yeah, for sure. I'm waiting for a cloud-automation fix to pass and then I'll push the latest MDS updates and then I'll build out this branch to QA |
Nice thanks! There is a huge queue in quay for portal images 🤦 |
* Quickly swap out access selector for column filter * Add Export to Workspace button * Studies are not selectable unless user has access * Restore border to study description * Add export buttons * Post to manifestservice when Export to Workspace button is clicked * Fix bug with config variable scope * Update mock config with file manifests * Implement 'Download Manifest' button * [WIP] Support content_type: link * Add content_type: link * Redirect to workspace page on Export to Workspace click * Configure whether Disco page is behind login * Revert breaking config change * Fix bug preventing multiple files from being selected - Steps to reproduce (heal preprod): - Search for 'jcoin' and select that study - See 1 study selected - Search for 'bacpac' and select that study - Expect 2 studies selected, get 1 study selected - Cause: table selection onChange handler sets selected resources to be whatever's currently selected in the table. If search or filtering has removed a selected row in the table, the onChange handler will happily drop it, as even though it was selected it's no longer in the table. - Fix: ~~Update table selection onChange handler to work by deselecting / selecting a single row at a time.~~ Actually it turns out there's a one-line config change that also fixes the issue :angry-haha: * Remove old code and fix errors * Fix tests * Fix style issues * Adv search open/close * Merge master * [WIP] Filter sliding animation and stick to viewport * [WIP] Filtering logic * Set filter state * Filter based on selections * Combine filters with AND * Hide locks if authz field is missing * Hide Adv Search Filters button if not enabled * Warn instead of crashing if adv search filters field is missing * Update style: search bar * Update font * Update style: adjust font sizes and margins to compress header * Update style: advanced search button * Update style: advanced search button and export to workspace button * Tweak header style to match prototype * [WIP] Update table style to match prototype * Warn instead of crash if malformed filterField * Fix eslint * Update table style to match prototype - Hide border between row and study desc row - Negative margin hack to move study desc row up - Fix bottom border - Remove hover state * Make table scrollable when advanced search filters are open * Enable search over fields not present in table * [WIP] Study description page pops in from right * [WIP] Improve study description page - [x] Study desc page pops in from right - [x] Field styles match prototype - [ ] Permalink button matches prototype - [ ] Tags match prototype - [ ] Study description is expandable - [ ] Close button matches prototype * [WIP] Improve study description page - [x] Study desc page pops in from right - [x] Field styles match prototype - [ ] Permalink button matches prototype - [ ] Tags match prototype - [ ] Study description is expandable - [ ] Close button matches prototype * [WIP] Improve study description page - [x] Study desc page pops in from right - [x] Field styles match prototype - [ ] Permalink button matches prototype - [x] Tags match prototype - [ ] Study description is expandable - [ ] Close button matches prototype * [WIP] Improve study description page - [x] Study desc page pops in from right - [x] Field styles match prototype - [ ] Permalink button matches prototype - [x] Tags match prototype - [ ] Study description is expandable - [ ] Close button matches prototype - [x] Handle display of long fields * Clean up unused code * New study description page - [x] Study desc page pops in from right - [x] Field styles match prototype - [x] Tags match prototype - [x] Close button matches prototype - [x] Permalink button matches prototype - [x] Handle display of long fields - TODO Study description is expandable * Fix table row highlight visual bug - Use a CSS hack (::after element behind the expanded row, which has a transparent background) * Fix visual bug: pagination not visible * Fix visual bug: top of third study description line visible in table * change label 'Unaccessible' to 'No Access' * change n/a logic * change label to 'Access Required' * fix unit test * remove bhc from testSchema * fix stylelint * heal bundle without home page * Delete portal_config.md * fix * fix * fix unit test * fix: adv filter overflow * Mitigate table row CSS bug in Safari - In Safari, table rows overhang the bottom. - Seems to be caused by absolutely positioned ::after element -- in Chrome, it seems that `bottom: -35px` puts the ::after element right below the element it's attached to, but on Safari, `bottom: -35px` puts the ::after element on the bottom of the *container* element (the `table body`!) - Mitigated by changing table background to the same gray as the rows, and hiding the ::after elements at the bottom of the <tbody> element behind the pagination element. * fix: row color * fix: empty component * fix: css * fix: css * fix: more css * feat: tag scroll * fix lint * feat: scroll tag reset * feat: sort tag * chore: rebase * chore: pkg * chore: 8 tags * chore: no more beta * fix: search crash * fix: css * fix: checkbox Co-authored-by: Michael Ingram <mpingram@uchicago.edu> Co-authored-by: haraprasadj <haraprasadj@uchicago.edu>
* Quickly swap out access selector for column filter * Add Export to Workspace button * Studies are not selectable unless user has access * Restore border to study description * Add export buttons * Post to manifestservice when Export to Workspace button is clicked * Fix bug with config variable scope * Update mock config with file manifests * Implement 'Download Manifest' button * [WIP] Support content_type: link * Add content_type: link * Redirect to workspace page on Export to Workspace click * Configure whether Disco page is behind login * Revert breaking config change * Fix bug preventing multiple files from being selected - Steps to reproduce (heal preprod): - Search for 'jcoin' and select that study - See 1 study selected - Search for 'bacpac' and select that study - Expect 2 studies selected, get 1 study selected - Cause: table selection onChange handler sets selected resources to be whatever's currently selected in the table. If search or filtering has removed a selected row in the table, the onChange handler will happily drop it, as even though it was selected it's no longer in the table. - Fix: ~~Update table selection onChange handler to work by deselecting / selecting a single row at a time.~~ Actually it turns out there's a one-line config change that also fixes the issue :angry-haha: * Remove old code and fix errors * Fix tests * Fix style issues * Adv search open/close * Merge master * [WIP] Filter sliding animation and stick to viewport * [WIP] Filtering logic * Set filter state * Filter based on selections * Combine filters with AND * Hide locks if authz field is missing * Hide Adv Search Filters button if not enabled * Warn instead of crashing if adv search filters field is missing * Update style: search bar * Update font * Update style: adjust font sizes and margins to compress header * Update style: advanced search button * Update style: advanced search button and export to workspace button * Tweak header style to match prototype * [WIP] Update table style to match prototype * Warn instead of crash if malformed filterField * Fix eslint * Update table style to match prototype - Hide border between row and study desc row - Negative margin hack to move study desc row up - Fix bottom border - Remove hover state * Make table scrollable when advanced search filters are open * Enable search over fields not present in table * [WIP] Study description page pops in from right * [WIP] Improve study description page - [x] Study desc page pops in from right - [x] Field styles match prototype - [ ] Permalink button matches prototype - [ ] Tags match prototype - [ ] Study description is expandable - [ ] Close button matches prototype * [WIP] Improve study description page - [x] Study desc page pops in from right - [x] Field styles match prototype - [ ] Permalink button matches prototype - [ ] Tags match prototype - [ ] Study description is expandable - [ ] Close button matches prototype * [WIP] Improve study description page - [x] Study desc page pops in from right - [x] Field styles match prototype - [ ] Permalink button matches prototype - [x] Tags match prototype - [ ] Study description is expandable - [ ] Close button matches prototype * [WIP] Improve study description page - [x] Study desc page pops in from right - [x] Field styles match prototype - [ ] Permalink button matches prototype - [x] Tags match prototype - [ ] Study description is expandable - [ ] Close button matches prototype - [x] Handle display of long fields * Clean up unused code * New study description page - [x] Study desc page pops in from right - [x] Field styles match prototype - [x] Tags match prototype - [x] Close button matches prototype - [x] Permalink button matches prototype - [x] Handle display of long fields - TODO Study description is expandable * Fix table row highlight visual bug - Use a CSS hack (::after element behind the expanded row, which has a transparent background) * Fix visual bug: pagination not visible * Fix visual bug: top of third study description line visible in table * change label 'Unaccessible' to 'No Access' * change n/a logic * change label to 'Access Required' * fix unit test * remove bhc from testSchema * fix stylelint * heal bundle without home page * Delete portal_config.md * fix * fix * fix unit test * fix: adv filter overflow * Mitigate table row CSS bug in Safari - In Safari, table rows overhang the bottom. - Seems to be caused by absolutely positioned ::after element -- in Chrome, it seems that `bottom: -35px` puts the ::after element right below the element it's attached to, but on Safari, `bottom: -35px` puts the ::after element on the bottom of the *container* element (the `table body`!) - Mitigated by changing table background to the same gray as the rows, and hiding the ::after elements at the bottom of the <tbody> element behind the pagination element. * fix: row color * fix: empty component * fix: css * fix: css * fix: more css * feat: tag scroll * fix lint * feat: scroll tag reset * feat: sort tag * chore: rebase * chore: pkg * chore: 8 tags * chore: no more beta * fix: search crash * fix: css * fix: checkbox * feat: button redirect to login * update * fix: tests * fix: travis * update lock Co-authored-by: Michael Ingram <mpingram@uchicago.edu> Co-authored-by: haraprasadj <haraprasadj@uchicago.edu>
* feat: new pending icon * update tooltip text
This was referenced Jan 10, 2022
This was referenced Mar 21, 2022
This was referenced Mar 29, 2022
Closed
Closed
This was referenced Apr 30, 2022
This was referenced Jun 23, 2022
This was referenced Dec 19, 2023
Open
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Jira Ticket: HP-255
Deployed at: https://qa-bloodpac.planx-pla.net/discovery and https://qa-heal.planx-pla.net/
New Features
Improvements
pendingDependency updates
Deployment changes
discoveryConfig.features.exportToWorkspaceBETAhas been renamed todiscoveryConfig.features.exportToWorkspaceGEN3_BUNDLEoptionhealintroduced for buliding a HEAL bundle