-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
feat(WorkListUINext): New Study List (WorkListUINext) #6005
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
Open
jbocce
wants to merge
178
commits into
OHIF:master
Choose a base branch
from
dan-rukas:feat/OHIF-1534-studylist
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
178 commits
Select commit
Hold shift + click to select a range
f4cd0b0
Moving prototype location
dan-rukas 8017488
Panel prototypes
dan-rukas 1557235
Add Table component and prototype
dan-rukas f401c25
Component theme updates
dan-rukas 48ec1b7
More visual design edits
dan-rukas 64bac48
Updated Panel prototype with table
dan-rukas 74ca5bc
Separated patient list data into a separate file
dan-rukas eed1b3d
Panel orientation switch and scrolling
dan-rukas 027b40e
Table scrolling fixes
dan-rukas 196d681
DataTable prototype
dan-rukas 3f034e3
Added pinned table header
dan-rukas 6b0739d
Split panel states into separate files
dan-rukas 0cab691
Added thumbnail panel content
dan-rukas 1ec30f2
Random number of thumbnails
dan-rukas 770b6f0
Updated header layout
dan-rukas 06a3aa3
Removed DataTable and added directly to Study List
dan-rukas 027f95c
Filters prototype working
dan-rukas 098d9c2
Table header and other fixes
dan-rukas 985c8e5
Updated panel
dan-rukas 6bc294d
Added action button to rows
dan-rukas 960f329
Missing text fix
dan-rukas 6c0f35a
Rebuilt prototype
dan-rukas 7ef09d4
Fixed ScrollArea in new prototype
dan-rukas 84b9441
Created a more reusable DataTable component
dan-rukas 8bfe788
Moving all related prototype files
dan-rukas 965bdd0
DataTable component updates
dan-rukas d76f52e
Updated prototype documentation
dan-rukas d3b9124
Added temporary logo for prototype
dan-rukas d1b9480
Spacing fixes in panels
dan-rukas 4eceb18
Spacing fixes
dan-rukas e2c2c84
Removed bottom panel
dan-rukas c67e1a6
Update package.json after rebase
dan-rukas a156c89
Netlify deployable
dan-rukas 000ec09
Initial Summary component
dan-rukas 9cc1c74
Updated component
dan-rukas cf6eab8
Updated Summary component
dan-rukas 3061c62
Color updates and info icon
dan-rukas c1ac954
Added workflow buttons to preview
dan-rukas 0a33f4d
Responsive fixes for buttons
dan-rukas bbad027
Added Panel toggle to toolbar
dan-rukas 8863591
Table responsive updates
dan-rukas 816e06b
Table patient column sizing
dan-rukas d61fa45
Workflow buttons match Modality selection
dan-rukas 3cedf08
Added default mode selection
dan-rukas ea968ac
Content fixes
dan-rukas 1b5c103
Add a settings dialog with workflow select
dan-rukas fc8ab4a
Fix workflow button padding
dan-rukas 8e4bc53
UX copy update
dan-rukas bc1f876
StudyList compound component
dan-rukas 3491ef2
Added StudyList component
dan-rukas 638982f
Prototype updates
dan-rukas 634192f
Remove legacy prototype files
dan-rukas 3b544e8
Remove legacy files
dan-rukas 9b7e46b
Icons from ui-next, removed tmp assets
dan-rukas 487b6e8
Migrated PatientSummary to ui-next
dan-rukas 090663d
Moved more Icons to ui-next, removed tmp assets
dan-rukas 21e8101
Updated StudyList architecture to be flexible
dan-rukas 5850b70
Simplified row action button
dan-rukas 0ae2bc7
Pagination added to table
dan-rukas a970301
Pagination fixes
dan-rukas d30fddd
Added Popover for settings instead of Dialog
dan-rukas 44d08b7
Remove default workflow button in Patient summary
dan-rukas 49d8243
Other modes are selectable with a default active
dan-rukas 5871531
Updated default workflow design and feedback
dan-rukas 6479b25
Filters always visible when scrolling
dan-rukas a22d324
Rename for SettingsPopover
dan-rukas e8cff76
Added hint text for other workflows
dan-rukas d3c3833
Default mode double click launches viewer
dan-rukas 6f19946
PatientSummary updates
dan-rukas 616e645
PatientSummary flexible Title and Subtitle
dan-rukas 7d9eb4d
Small tweaks
dan-rukas 955fef4
Moved StudyList components
dan-rukas 27522da
playground clean-up
dan-rukas 76eeb9b
StudyList clean-up
dan-rukas 9d6505a
use props instead of slots for StudyListLayout
dan-rukas 4dda733
Layout renaming, Updated docs, Component structure
dan-rukas 0ad858b
Doc updates, Popover fix
dan-rukas f71e6bc
Added second prototype with real data
dan-rukas d519011
Keep action menu visible regardless of column visibility
dan-rukas d2f399e
More button visuals, Instances header icon
dan-rukas 40a5046
Fix action menu tooltip
dan-rukas f399d27
Fix: table header and filters resize correctly
dan-rukas e4b1ec1
Added rows per page in Pagination, Assets
dan-rukas 2605447
Settings added to panel close state
dan-rukas 4d83eb6
Added List View to Preview Panel
dan-rukas 2f284f2
Use Viewer StudyBrowser controls
dan-rukas fe7c447
Added no description feedback in table
dan-rukas d0c677d
Added "No Default" to default workflow menu
dan-rukas c09836c
Removed additional backgrounds
dan-rukas 70b3b8f
SettingsPopover with subcomponents
dan-rukas 61dbd12
Added settings popover to prototype with working modals
dan-rukas e445a9c
Updated docs
dan-rukas 312f5f1
SettingsPopover updates
dan-rukas 458ee6f
StudyListLayout updates
dan-rukas 7332e7e
Added real Icons for sorting
dan-rukas f76161c
Default sorting in prototype and width updates
dan-rukas 66b9b0d
Visual fixes to preview panel
dan-rukas 5939d87
visual fix spacing for patient summary
dan-rukas bb38da1
Remove second real data prototype
dan-rukas 1dfb2c5
Add InputMultiSelect and more to Study List
dan-rukas 38592b9
Updated DataTable compound component
dan-rukas 06dfa8f
More DataTable refactoring. Made DataTable and its components more fl…
jbocce ceee28b
More code cleanup and removal. Specifically, all items related to the…
jbocce a2245e7
Restored template.html file and removed unused dependency.
jbocce ca74e24
Moved study date and time formatting to study list columns.
jbocce c674f14
Replace the previous workflow system with the new StudyListWorkflowPr…
jbocce ce23982
Removed overuse of generics.
jbocce 4ae6527
More refactoing.
jbocce 07348da
Integrate study list into OHIF. This is a prototype of the study list…
jbocce b5d37b3
Sync study list state with URL and session storage.
jbocce 2a7b046
Fix netlify deploy error.
jbocce 0a06dfd
Preserve datasources query parameter.
jbocce f19a1d5
New Data Source Configuration design and assets
dan-rukas 302d549
Renamed StudyListNext2 to WorkListUINext.
jbocce c6f5fb2
Removed failing playwright worklist test as it no longer applies to t…
jbocce a2c909f
Fix broken restore of pagination persistence when it occurs with sort…
jbocce a292d59
A better fix for initial restore of pagination persistence.
jbocce 859eb39
Added logo white labelling.
jbocce bdda4f1
Added OIDC logout menu item.
jbocce bffd6ff
Added upload and DICOM data source configuration components.
jbocce 40ba4b0
Add investigational use dialog to worklist ui next.
jbocce a0f5396
Add loading indicator for WorkListUINext and avoid showing empty stat…
jbocce 917d9da
Add date range filter to StudyList.
jbocce b331578
Size date range filter to match the height of the other filters.
jbocce a3ab54a
Remove left padding on StudyList preview panel content.
dan-rukas 19fcfe9
Truncate long workflow button labels with ellipsis.
dan-rukas 12e2b59
Restyle StudyList View button to ghost variant with chevron.
dan-rukas fd065c5
Revised study date column and DateRange component
dan-rukas a6a90f1
DateRange allows editing of placeholders
dan-rukas 9d141fd
Add SeriesPlaceholder icon
dan-rukas 03247f5
Refine thumbnail rendering for study list preview.
jbocce 969b3cc
Append viewport query parameter to thumbnail endpoint path for better…
jbocce 1df0db6
Updated empty thumbnail design
dan-rukas 18de75b
Add customization for the study list's preview content.
jbocce 06b7118
Remove obsolete tests.
jbocce 1842920
Fix preview container layout and series list scroll area.
jbocce 984bd18
Cleanup PreviewPatientSummary. Remove StudyList README.
jbocce 44bcfbb
Cleanup some config files.
jbocce 44bd463
Add RWV to thumbnailNoImageModalities.
jbocce 2879ba7
Use thumbnail fetch strategy for netlify.
jbocce bd2dca0
Simplify PreviewPatientSummary to match actual usage
dan-rukas 0b92e2c
Simplify InputMultiSelect to match actual usage
dan-rukas ed59216
Simplify SettingsPopover to match actual usage
dan-rukas 89f1aa9
Append accept=image/jpeg to the thumbnail endpoint path for bulk data…
jbocce c04a059
Faster diff
wayfarer3130 dd050cc
Merge branch 'master' into feat/OHIF-1534-studylist
wayfarer3130 ece9e4e
Merge branch 'master' into feat/OHIF-1534-studylist
wayfarer3130 01a6c19
Batch thumbnail fetching so studies with many series don't saturate t…
jbocce 0756987
Add ability to abort thumbnail fetches in the study list.
jbocce 5770270
Merge remote-tracking branch 'origin/master' into feat/OHIF-1534-stud…
jbocce 2c2de40
Merge remote-tracking branch 'dan/feat/OHIF-1534-studylist' into feat…
jbocce bb52537
Rename WorkListUINext to WorkList and WorkList to LegacyWorkList.
jbocce bb9f798
Remove unused dataTotal prop from WorkList.
jbocce 1223d35
Customize the WorkList columns.
jbocce 9816a2c
Move StudyListSettingsPopover and SidePanelPreview to their own files.
jbocce 744255b
Customize the preview content in the study list.
jbocce f427f9f
Customize the study list settings popover content.
jbocce e195bc1
Document the about modal customization.
jbocce baab339
Refactor the data source wrapper to use a shallow equal ignoring arra…
jbocce c906142
Merge remote-tracking branch 'origin/master' into feat/OHIF-1534-stud…
jbocce 03ec8ac
Fix patient-name filter and centralize the study-list URL key contract.
jbocce fc6a332
Hoist DEFAULT_DATA to module scope to avoid re-allocating it each ren…
jbocce 32c592e
Recover from search failures: ensure loading state resets, prevent re…
jbocce 5a9d7a3
Deselect rows that are no longer on the current page.
jbocce cf0d410
Auto-hide columns based on container width.
jbocce 18c06ea
Make URL parameter lookup case-insensitive.
jbocce 1b9d300
Revoke preview thumbnail blob URLs on cleanup.
jbocce 035a728
Move the queryLimit to a data source level configuration.
jbocce c8c480c
Refactor the thumbnail retrieval to a separate file.
jbocce 3d28941
Add documentation for the maxNumRequests.thumbnail setting.
jbocce d4670f9
Extract useSeriesFetch hook from SidePanelPreview.
jbocce 4315f79
Refactor useStudyListQuery from DataSourceWrapper.
jbocce 3d90577
Add test for shallowEqualIgnoringArrayOrder.
jbocce a43a4c1
Consolidate date and time formatting for both study lists.
jbocce f261021
Update date picker components to react-day-picker v9 and date-fns v4
dan-rukas 743caed
defaultColumns is now a constant instead of a function.
jbocce 70cfe86
Document data source query limit and paging behavior changes.
jbocce 9414ff2
Merge remote-tracking branch 'origin/master' into feat/OHIF-1534-stud…
jbocce File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
Oops, something went wrong.
Oops, something went wrong.
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.
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.
I introduced this setting for the (potentially) odd case where someone was using the existing
bulkDataRetrievereliably withthumbnailRendering=thumbnailorthumbnailRendering=rendered. However, my investigations have shown that there are the following problems whenbulkDataRetrieveis used...orthanc
/renderedendpointgoogle cloud healthcare
?accept=image/jpegdcm4chee
?accept=image/jpegwhich then conflicts with google?viewport=addition to the url