* fix: 🐛 Make Plugin switch part of ToolbarModule
Closes: #1241
* Move setCornerstoneViewport command into cornerstone extension
* Exposing extensionManager into extension's get modules functions
* Remove Plugin switch
* Updating set cornerstonelayout to work on VIEWER context
* Creating a customComponent for 2D MPR button
* Use extensionManager to show or not buttons
Adding isHidden option into toolbarModules
* Make buttons hidden depending on extension registration
* Fix unit test after changes on extensionManager
* Remove workaround for extensionManager
* Use studyMeatadataManager and remove TODO's
* Remove storeContext not needed
* Moving Exit 2D MPR into cornerstone extension with VTK context
* Remove PluginSwitch files
* Fix E2E tests
* Adding cypress screenshots into gitignore
* Remove extensionManager from getModuleFn functions and moving CustomComponent for VTK button to VTK extension
Co-authored-by: Danny Brown <danny.ri.brown@gmail.com>
* E2E Tests: Save Measurements Functionality
* Moving test file to PWA only. Script-tag doesn't have the required Study
* Removing data-cy selector from cornerstone toolbar
* feat: 🎸 Configuration so viewer tools can nix handles
Adds a key to cornerstone configuration that allows to toggle on/off
handle rendering
Closes: #1223
* Update extension docs
* Update doc
* docs: include hideHandles configuration in our configuration docs
* chore: default to hiding handles for annotations
* chore: switch to drawHandlesOnHover to mimic legacy viewer functionality
* Flip logic for drawHandlesOnHover; reverse of hideHandles
* Don't hide the handles of ellipse or rectangle roi
* invert check
Co-authored-by: Danny Brown <danny.ri.brown@gmail.com>
* Move DICOMSR to @ohif/core
* refactor: 💡 Move DICOMSR code to @ohif/core project
We've just merged support for DICOM Structured Reports. Today, support
for this feature lives in the @ohif/viewer project in it's lib folder.
Ideally, it should be moved to @ohif/core and given a clear public API
and unit tests.
Closes: #1282
* Update imports
* CR Update: Update imports
* feat: 🎸 Expose extension config to modules
Currently, only the preRegistration hook receives the extension's
configuration as a parameter. Providing getModuleFn's with the
extension's configuration, and all lifecycle/modules with the
application's configuration as rootConfig should open the doors to more
configurable extensions.
Closes: #1268
* CR Update: Pass extension and service config through extension manager preinit/getmodule
* CR Update: Remove appConfig from servicesManager
* CR Update: Remove appconfig variable
* ci: update executor and browser flag for circleci orbs
* ci: browser flag when running percy/ci tests
* ci: limit ourselves to the executors defined by the cypress orb for now
* fix: Set VTK viewport as active by interaction
* chore: upgrade cypress to latest
* Revert "fix: Set VTK viewport as active by interaction"
This reverts commit b6e174c2def9d8f688a54d32053679788b9aaf76.
* ci: add record flag for master cypress runs
* ci: simplify cypress orb tasks now that cypress@3.8 uses electron w/ chromium 78
* ci: remove video false flag
* ci: drop browser flag
* ci: store video artifacts
* fix: Set VTK viewport as active by interaction
* chore: upgrade cypress to latest
* Revert "fix: Set VTK viewport as active by interaction"
This reverts commit b6e174c2def9d8f688a54d32053679788b9aaf76.
* ci: add record flag for master cypress runs
* Tests for routes: PT/CT json, studies, series, frames, metadatas
* Fix based on reviews and JSON file for PT/CT Study added
* chore: revert to remote JSON file
* chore: delete PTCTStudy.json file
We can circle back to this approach when/if we have issues. No need to hold-up this PR. My fault on this one.
* E2E Tests: Download Image File Modal
* CI fix
* Fixes for PR comments
* Small change just to check CI behavior
* Fix for CI failure: comparing classes instead of image src
* Fixed Download button test and blocked Download Dialog event
* Deleting file ConnectedViewportDownloadForm.js
This file was not supposed to be created by this PR. It was already deleted by another user on Master branch.
* Removing wrong import line
* Fixes based on PR comments
* Fixes based on PR comments
* test: remove assignment to window
* feat: 🎸 DICOM SR STOW on MeasurementAPI
Closes: #758
* feat: 🎸 Adding the dataExchance from DICOM SR
* feat: 🎸 Fix some issues and get the retrieve measurements working
* feat: 🎸 Preventing measurement tool helpers to broke because the data is not Number (it comes as string from dcmjs)
* feat: 🎸 Adding a button to start testing SATE functionality
* feat: 🎸Small fixes into the code
* feat: 🎸 MAke sure we get latest SR in the study, even on lazyLoad mode
* Revert "feat: 🎸 Preventing measurement tool helpers to broke because the data is not Number (it comes as string from dcmjs)"
This reverts commit 4b3a1efb5810487d2319826fd59c03d779882b5e.
* feat: 🎸 Make sure after sync we update the viewports and also that annotation tools are set to passive, so it can be drawn in the viewports
* feat: 🎸 Remove console logs and start using OHIF.log
* feat: 🎸 Refactor measurement table to use Snackbar service of messages and make sure we only show save button if server type is dicomWeb
* feat: 🎸 Refactor to remove mocked wadoRoot and use server
* Fix to expose error once we can't save the SR
* Update DCMJS into latest dev version
* Update dcmjs with latest fixes
* Fix PR callouts
* Logging error on MeasurementReport creation
* Preventing from breaking on non supported tools and also warning that some toolType is not sopported
* Small error message update
* Expose better the errors and when we have unsopported tools into the measurements
* Renaming variable
* Expose dcmjs tool support check
* Change the way we warn the unsupportedTools
* Adding style to the button
* Small change on button style
* Fix git conflicts
* Fix after merge with latest master
* Fix after merge with master, btn was not clickable
* Fixing callouts
* Adding util functions into files and refactor with JSDocs
* Refactoring code, adding some basic JSDocs
* Bump dcmjs with latest version and fixes
* JSDocs and small refactors
* yarn lock file after dcmjs update
* Remove unused parameters
* Registering a plain copy of server into measurementAPI
* Small refactor
* Start changing from toolType to toolName
* Rename a function to get instanceMetadata
* Adding error if server is not DICOMWeb on retrieve
* Move snackbar outside of UI component and use UINotificationService
* Refactor and JSDocs
* Small fixes on JSDocs (style)
* Remove unused import
* Improve JSDocs on MeasurementApi
* Remove refactor leftover
* Improve JSDocs and propTypes
* JSDocs and small refactors
* Fix ExtensionManager bug and add test to bandaid
* Add tools configuration to extension manager preinit
* Fix reducing of configs
* Merge internal with external configs
* Merge internal with external configs
* Remove dialog from init in measurementstable
* Testing injected configuration
* New way to set config
* Add new prop to dialog provider to allow disabling last position
* Remove code from preinit in cornerstone
* Add new prop to dialog provider to allow disabling last position
* Add centralize to dialogs
* Reorder dialogs when adding them
* Fix draggable styles (cursor)
* Remove repositioning methods from labelling flow and remove overlay from labelling manager
* Fix empty array being set in bringToFront
* Add new command to update table and pass commands manager to modules/preinit hook
* Ad UIContextMenu service / factory
* Use new contextmenu service in measurementspanel extension
* Use dialogs for arrow annotate in default
* Remove positioning funcionality from tool context menu
* Add context menu service
* Pass commandsModule to extension
* Update edit description dialog and simple dialog to position relative
* Remove style code from labelling flow and manager
* Remove eventdata from labelling
* Remove labelling code from measurement init
* Add commandsmanager to provider
* Update contextmenu provider and service
* Add touchstart and mouseclick to hide contextmenu
* Hide labelling if click/touch
* Remove labelling and context menu dead code
* Fix undefined bug if ViewerMain grid has no children
* Fix broken prop on context menu
* Update commandsmodule based on master
* Fix broken configuration
* Update script tag config
* Remove cornerstone from toolcontextmenu
* Remove cornerstone from toolcontextmenu
* Split labelling and context menu providers
* Split labelling and context menu providers
* Update test
* Destructure extensions into new array
* CR Update: Move default arrow config to cornerstone instead of default
* CR Update: Fix app configuration props structure
* CR Update: Fix app configuration prop in script tag and extract commands manager from providers
* CR Update: Create custom providers to use commandsManager
* CR Update: Use services directly in measurementspanel
* CR Update: Pass components to providers
* CR Update: Remove position from dialog
* CR Update: fix dialog prop check
* CR Update: Fix comments
* CR Update: Update documentation
* CR Update: Add test default configuration
* CR Update: Add default empy array to extensions
* CR Update: Update i18n configuration all ot match current function configuration
* CR Update: Add defaults to injected dependencies in configuration and extension configuration
* CR Update: Add defaults to configuration with no args
* Update documentation
* CR Update: Add default for tools
* CR Update: Update config object to i18n
* CR Update: spread defaults
* CR Update: Add tool configuration example to cornerstone extension
* CR Update: Add tool configuration to netlify (testing)
* CR Update: Remove netlify config for tools
* CR Update: Rollback changes to i18n to be fixed later
* CR Update: Update documentation and pass whole cornerstone config object instead of tools key
SEE: https://www.conventionalcommits.org/en/v1.0.0/#commit-message-with-both-and-breaking-change-footer
BREAKING CHANGE: modifies the exposed react <App /> components props. The contract for providing configuration for the app has changed. Please reference updated documentation for guidance.
* docs: process docs to include UX Stories requirement
* docs: include note regarding different environments
* Services init
* Remove unused canny logos
* docs: add ModalService diagram
* docs: GIF of notification
* docs: add ui-services page
* docs: simplify ui services call out in the general services docs
* docs: tips and tricks for UI services
* docs: moar pages
* docs: dialog gif
* docs: services in summary/sidebar
* docs: gif examples at top of dialog and notification pages
* docs: add UIModal gif
* docs: details for Dialog Service
* docs: include usage information for ui modal service
* docs: detailed information about our UI Notification Service
* docs: services diagram
* docs: remove unused links
* docs: services example and image
* docs: stubbing extension pages
* docs: UX Stories --> User Cases
* chore: eslint fix
* docs: extensions overview + diagram
* docs: update extensions in sidebar
* docs: extension skeleton and registration info
* docs: copy of maintained extensions table partial in the extensions folder
* docs: swap out old extensions structure for new
* Organize viewports reducers
* Organize viewports actions
* Add local state to store dom node and remove hack
* Comment usage of dom in vtk
* Fix set of enabledElements
* Fix warning in html viewport
* Update docs for state
* Add commandsmanager to commandsmodule
* refactor: Shift ConnectedViewportDownloadForm to extension-cornerstone (#1224)
* refactor: 💡 Shift ConnectedViewportDownloadForm to its own proj
Shift ConnectedViewportDownloadForm to its own project and use local
state to get enabledElement
Closes: #1206
* Refactor connected component to cornerstone component
* Remove builtin behaviour from toolbarrow
* Organize viewports reducers
* Organize viewports actions
* Add local state to store dom node and remove hack
* Comment usage of dom in vtk
* Fix set of enabledElements
* Fix warning in html viewport
* Update docs for state
* Add commandsmanager to commandsmodule
* fix: study list translations
* Don't render until translations are ready
* Try to wait for translations to load
* Use null to render "nothing"
* Try toggling useSuspense off up a layer
* logging
* Remove useSuspense false flags
* DO NOT OPEN A PR WITH IT.
Experimental changes only.
Try to solve issues with useTranslation hooks
#Please enter the commit message for your changes. Lines starting
* Remove unecessary changes
* feat: 🎸 useMedia hook to not use one more prop for upd state vl
* docs: Add license scan report and status (#1161)
Signed-off-by: fossabot <badges@fossa.io>
* fix: 🐛 Fix for JS breaking on header (#1164)
* feat: 🎸 Code review and refact
Revised code based on PRs(variable alias,...). Changed hook for
useMedia. Now, it provides two hooks: one to get displayMediaSize and
other to get Entity(component, objects...) based on displayMediaSize.
Implemented a similar solution for state manager to store
mediaQueryList(s) and displaySize for app.
* feat: 🎸 Fine tunning on mediaQuery value, fixed issue about it
* fix: 🐛 Fixed issue and refactoring
Fixed js exception and also refactoring userPreferences components to
functional component
* fix: 🐛 Code review. Localstorage fix minor bugs
* fix: 🐛 Ensure hotkey lower case always
* fix: translation switcher
* chore(release): publish [skip ci]
- @ohif/extension-vtk@0.53.6
- @ohif/ui@0.62.1
- @ohif/viewer@2.8.2
* Add new modal service
* Change serviceManager prop to servicesManager
* CR Update: fix casing and add required proptypes to providers
* CR Update: Improve ohifmodal proptypes
* CR Update: Fix typo in extensionmanager
* CR Update: add default props to service and check service in provider
* Refactor modal provider to better use its own state
* ci: don't build our master branch (#1177)
* ci: don't build our master branch
* Add netlify-cli as a dev dependency
* ci: trying a sheltered merge flow for promotions
* Use modal instead of modal context
* Ci/promotable builds (#1179)
* ci: don't build our master branch
* Add netlify-cli as a dev dependency
* ci: trying a sheltered merge flow for promotions
* ci: try building with QUICK_BUILD flag
* Change modal children order
* Ci/promotable builds (#1180)
* ci: don't build our master branch
* Add netlify-cli as a dev dependency
* ci: trying a sheltered merge flow for promotions
* ci: try building with QUICK_BUILD flag
* Try using ~/repo prefix in command
* Ci/promotable builds (#1181)
* ci: don't build our master branch
* Add netlify-cli as a dev dependency
* ci: trying a sheltered merge flow for promotions
* ci: try building with QUICK_BUILD flag
* Try using ~/repo prefix in command
* ci: make sure netlify command is available
* Ci/promotable builds (#1182)
* ci: don't build our master branch
* Add netlify-cli as a dev dependency
* ci: trying a sheltered merge flow for promotions
* ci: try building with QUICK_BUILD flag
* Try using ~/repo prefix in command
* ci: make sure netlify command is available
* ci: use sudo for global command
* Fix OHIFModal proptypes
* Ci/promotable builds (#1183)
* ci: don't build our master branch
* Add netlify-cli as a dev dependency
* ci: trying a sheltered merge flow for promotions
* ci: try building with QUICK_BUILD flag
* Try using ~/repo prefix in command
* ci: make sure netlify command is available
* ci: use sudo for global command
* Inline personal access token w/ env var
* Ci/promotable builds (#1184)
* ci: don't build our master branch
* Add netlify-cli as a dev dependency
* ci: trying a sheltered merge flow for promotions
* ci: try building with QUICK_BUILD flag
* Try using ~/repo prefix in command
* ci: make sure netlify command is available
* ci: use sudo for global command
* Inline personal access token w/ env var
* ci: workaround for sudo limitations
* ci: restore release workflow (#1185)
* ci: don't build our master branch
* Add netlify-cli as a dev dependency
* ci: trying a sheltered merge flow for promotions
* ci: try building with QUICK_BUILD flag
* Try using ~/repo prefix in command
* ci: make sure netlify command is available
* ci: use sudo for global command
* Inline personal access token w/ env var
* ci: workaround for sudo limitations
* ci: restore release workflow
* Pass services to each module, improve tests
* Add servicesManager test and registerServices method
* Fix key warning of snackbar elements
* Remove netlify-cli; we';ll install this on CI server
* Update staging and prod netlify site IDs
* Clean up NPM_PUBLISH step
* Clean up DOCS_PUBLISH step
* Clean up Deploy workflow
* Custom executor to override cypress config
* Spacing
* Use an existing docker hub image
* Switch to npx instead of digging into npm bin location
* Remove e2e test before prod deploy
* Add workflow images
* docs: continous integrationn
* Add default props to modal
* chore(release): publish [skip ci]
- @ohif/extension-cornerstone@1.5.1
- @ohif/extension-vtk@0.53.7
- @ohif/core@1.9.1
- @ohif/ui@0.62.2
- @ohif/viewer@2.8.3
* chore(release): publish [skip ci]
- @ohif/extension-vtk@0.53.8
- @ohif/core@1.10.0
- @ohif/ui@0.62.3
- @ohif/viewer@2.8.4
* ci: Redirect site traffic to index.html if file is not resolved
* ci: fix typo
* fix: 🐛 Code review. Remove 'global state' for displaySize
* fix: 🐛 Code review. Ref back to useMedia and pass value down
Code review. Ref back to useMedia and pass value down (components)
instead of creating a specialized hook to tied any component on it
* fix: translation switcher
* fix
* cleanup
* add missed translation
* fix: 🐛 Code review
* fix: 🐛 Fix unit tests
* fix: 🐛 Fix unit test
* fix: 🐛 Code merge solve conflicts. Missing files changes
* Merge from master. Missing files changes
* feat: 🎸 Code review. Fix issues with DatePicker
Fixed at least the minimum issue with datePicker and update some content
on every translation changed
* fix: 🐛 Code review. Style fix on modal
* fixes in general preferences after merge
* fix
* export default i18n language
* translation fixes
* increase snackbar zindex
* initialize language
* WIP propagate props<>state fixes
* fix state propagation
* remove hotkey error when reset default
* fix datePicker
* i18n updates
* fix i18n strings
* fix reset default -- set right input value
* fix default language
* fix e2e for user preferences
* small fix datePicker props
* remove i18n from window -- debugging purposes only
* lint fixes
* Testing dialog
* Refactor modal and add dialog component to simple dialog
* Use existent dialog
* Expect event from getText callback (annotation)
* Bump cornerstone-tools version
* Use simple dialog
* bump cornersotne version
* CR Update: Update dialgo method names and default values
* CR Update: Bump overlay and show only if theres at least one overlay prop set to true
* CR Update: Use percentage over view height in draggable area
* Fix broken test vtk checks WWWC tool
* Comment test (non-deterministic)
* fix: study list translations
* Don't render until translations are ready
* Try to wait for translations to load
* Use null to render "nothing"
* Try toggling useSuspense off up a layer
* logging
* Remove useSuspense false flags
* DO NOT OPEN A PR WITH IT.
Experimental changes only.
Try to solve issues with useTranslation hooks
#Please enter the commit message for your changes. Lines starting
* Remove unecessary changes
* feat: 🎸 useMedia hook to not use one more prop for upd state vl
* docs: Add license scan report and status (#1161)
Signed-off-by: fossabot <badges@fossa.io>
* fix: 🐛 Fix for JS breaking on header (#1164)
* feat: 🎸 Code review and refact
Revised code based on PRs(variable alias,...). Changed hook for
useMedia. Now, it provides two hooks: one to get displayMediaSize and
other to get Entity(component, objects...) based on displayMediaSize.
Implemented a similar solution for state manager to store
mediaQueryList(s) and displaySize for app.
* feat: 🎸 Fine tunning on mediaQuery value, fixed issue about it
* chore(release): publish [skip ci]
- @ohif/extension-vtk@0.53.6
- @ohif/ui@0.62.1
- @ohif/viewer@2.8.2
* Add new modal service
* Change serviceManager prop to servicesManager
* CR Update: fix casing and add required proptypes to providers
* CR Update: Improve ohifmodal proptypes
* CR Update: Fix typo in extensionmanager
* CR Update: add default props to service and check service in provider
* Refactor modal provider to better use its own state
* ci: don't build our master branch (#1177)
* ci: don't build our master branch
* Add netlify-cli as a dev dependency
* ci: trying a sheltered merge flow for promotions
* Use modal instead of modal context
* Ci/promotable builds (#1179)
* ci: don't build our master branch
* Add netlify-cli as a dev dependency
* ci: trying a sheltered merge flow for promotions
* ci: try building with QUICK_BUILD flag
* Change modal children order
* Ci/promotable builds (#1180)
* ci: don't build our master branch
* Add netlify-cli as a dev dependency
* ci: trying a sheltered merge flow for promotions
* ci: try building with QUICK_BUILD flag
* Try using ~/repo prefix in command
* Ci/promotable builds (#1181)
* ci: don't build our master branch
* Add netlify-cli as a dev dependency
* ci: trying a sheltered merge flow for promotions
* ci: try building with QUICK_BUILD flag
* Try using ~/repo prefix in command
* ci: make sure netlify command is available
* Ci/promotable builds (#1182)
* ci: don't build our master branch
* Add netlify-cli as a dev dependency
* ci: trying a sheltered merge flow for promotions
* ci: try building with QUICK_BUILD flag
* Try using ~/repo prefix in command
* ci: make sure netlify command is available
* ci: use sudo for global command
* Fix OHIFModal proptypes
* Ci/promotable builds (#1183)
* ci: don't build our master branch
* Add netlify-cli as a dev dependency
* ci: trying a sheltered merge flow for promotions
* ci: try building with QUICK_BUILD flag
* Try using ~/repo prefix in command
* ci: make sure netlify command is available
* ci: use sudo for global command
* Inline personal access token w/ env var
* Ci/promotable builds (#1184)
* ci: don't build our master branch
* Add netlify-cli as a dev dependency
* ci: trying a sheltered merge flow for promotions
* ci: try building with QUICK_BUILD flag
* Try using ~/repo prefix in command
* ci: make sure netlify command is available
* ci: use sudo for global command
* Inline personal access token w/ env var
* ci: workaround for sudo limitations
* ci: restore release workflow (#1185)
* ci: don't build our master branch
* Add netlify-cli as a dev dependency
* ci: trying a sheltered merge flow for promotions
* ci: try building with QUICK_BUILD flag
* Try using ~/repo prefix in command
* ci: make sure netlify command is available
* ci: use sudo for global command
* Inline personal access token w/ env var
* ci: workaround for sudo limitations
* ci: restore release workflow
* Pass services to each module, improve tests
* Add servicesManager test and registerServices method
* Fix key warning of snackbar elements
* Remove netlify-cli; we';ll install this on CI server
* Update staging and prod netlify site IDs
* Clean up NPM_PUBLISH step
* Clean up DOCS_PUBLISH step
* Clean up Deploy workflow
* Custom executor to override cypress config
* Spacing
* Use an existing docker hub image
* Switch to npx instead of digging into npm bin location
* Remove e2e test before prod deploy
* Add workflow images
* docs: continous integrationn
* Add default props to modal
* chore(release): publish [skip ci]
- @ohif/extension-cornerstone@1.5.1
- @ohif/extension-vtk@0.53.7
- @ohif/core@1.9.1
- @ohif/ui@0.62.2
- @ohif/viewer@2.8.3
* chore(release): publish [skip ci]
- @ohif/extension-vtk@0.53.8
- @ohif/core@1.10.0
- @ohif/ui@0.62.3
- @ohif/viewer@2.8.4
* ci: Redirect site traffic to index.html if file is not resolved
* ci: fix typo
* fix: 🐛 Code review. Remove 'global state' for displaySize
* fix: 🐛 Code review. Ref back to useMedia and pass value down
Code review. Ref back to useMedia and pass value down (components)
instead of creating a specialized hook to tied any component on it
* Merge from master. Missing files changes
* feat: 🎸 Code review. Fix issues with DatePicker
Fixed at least the minimum issue with datePicker and update some content
on every translation changed
* Test Cases for User Preferences
* CI fix
* Fix for CI
* Fix based on PR review comments
* Fix for script-tag issue on CI
* Fix for new selectors on Preferences Modal and CI failure
* feat: 🎸 Filter by url query param for seriesInstnaceUID
* fix: Set SR viewport as active by interaction (#1118)
* fix: Set SR viewport as active by interaction
* quick fix
* (eslint) add "before" as global variables
* add data-cy
* add data-cy
* create E2E test
* (E2E) create custom command to set layout size
* remove .only e2e
* remove throttle for onScroll
* feat: 🎸 Code review in progress
Code review. Move retrieveMEtadata load to separate folders. Some minor
code clean up
* feat: 🎸 Code review. Missing changes from previous commit
* feat: 🎸 Code review missing changes from previous commit
When sorting, Criteria for instance must use instanceNumber and not
instancesNumber
* feat: 🎸 Code review. Add more jsdoc info
* feat: 🎸 Code review. Prettify changed code
* feat: 🎸 Instead of filtering do promote. WIP
* feat: 🎸 Fix minor issue. Allow promote or filter
* feat: 🎸 Changing component to functional component
* Merge from master Part1/2
Conflicts solved:
both added: core/src/studies/services/wado/retrieveMetadataLoader.js
both added: core/src/studies/services/wado/retrieveMetadataLoaderAsync.js
both added: core/src/studies/services/wado/retrieveMetadataLoaderSync.js
both modified: viewer/src/connectedComponents/ViewerRetrieveStudyData.js
both modified: viewer/src/routes/ViewerRouting.js
* Merge process from master Part 1/2
Missing files from previous commit
* feat: 🎸 Add cancelable promises to cut async methods
* feat: 🎸 Missing changes from previous merge process
* feat: 🎸 Missing changes from previous merge process
* Visual testing for Cornerstone and VTK
* Fix for CI
* Review fix: Removed cornerstone function and added snapshot description
* Added snapshot description
* Fix for CI
* playing around
* Change extension init and preRegistration signature
* Clear test code
* Update core index test
* Fix test
* Tricking the bug
* Renaming file back to trick some weird bug
* feat: 🎸 Update hotkeys and user preferences modal
This feature fix incompatibility with existent hotkeys component and
adds user preferences modal back
Closes: #923
* Update preferences structure in store
* Hide window level section of user preferences
* Update modal to reflect current hotkey value
* Clone object with hotkeys before passing to manager
* CR Update: Extract hotkeys manager format code to manager
* Fix broken cypress test
* Use new modal provider
* Rename hotkeyDefinitions in hotkeyspreferences and use array as representation
* Update study test and remove unused styles
* Study list tests: Patient Name, MRN, Modality, Description
* Fixed field selector
* Desktop tests
* Fix for study list
* Added tablet tests and small refactor on aliases
* Fix based on reviews
* feat: 🎸 New modal provider
A new modal provider/context component to unify modal related code and
simplify modal use
Closes: #1086
* refactor(ModalContext): Keep current modal lib
* Update modal provider version according to PR #1116
* Update modal provider props to use inner props
* Add custom class prop
* CR Update: Refactor provider to extract specific modal
* Fix modal import
* CR Update: Move from spread to object assign
* CR Update: Add proptypes, use classnames dependency over interpolation and rename modal
* fix: 🎸 Update AboutModal to use new modal provider (#1116)
* feat: 🎸 Update AboutModal to use new modal provider
This feature updates the AboutModal component and Header to use the new
modal context (modal provider)
* feat(ModalProvider): Add title prop to provider
* fix(ModalContext): Add defaults on hide
* CR Updates: Refactor modal provider and fix import
* CR Update: Expose props and flat modal optons
* Use customclassname instead of static
* Add cypress tag
* test: set right viewport when layout is decreased (#1146)
* chore(release): publish [skip ci]
- @ohif/viewer@2.3.6
* fix: 🐛 Minor issues measurement panel related to description (#1142)
* chore(release): publish [skip ci]
- @ohif/extension-vtk@0.52.28
- @ohif/ui@0.58.5
- @ohif/viewer@2.3.7
* fix: 🐛 Upgrade react-vtkjs-viewport to fix volume orientation (#1143)
* chore(release): publish [skip ci]
- @ohif/extension-vtk@0.52.29
- @ohif/viewer@2.3.8
* CR Update: Extract and rename AboutModal to AboutContent
* feat: 🎸 New modal provider (#1110)
* feat: 🎸 New modal provider
A new modal provider/context component to unify modal related code and
simplify modal use
Closes: #1086
* refactor(ModalContext): Keep current modal lib
* Update modal provider version according to PR #1116
* Update modal provider props to use inner props
* Add custom class prop
* CR Update: Refactor provider to extract specific modal
* Fix modal import
* CR Update: Move from spread to object assign
* CR Update: Add proptypes, use classnames dependency over interpolation and rename modal
* chore(release): publish [skip ci]
- @ohif/extension-vtk@0.52.30
- @ohif/ui@0.59.0
- @ohif/viewer@2.4.0
* CR Update: Keep translation name
* CR Update: Reposition cy tag of about modal in the correct level
* feat: 🎸 Update DownloadModal to use new modal provider (#1119)
* feat: 🎸 Update AboutModal to use new modal provider
This feature updates the AboutModal component and Header to use the new
modal context (modal provider)
* feat(ModalProvider): Add title prop to provider
* feat(DownloadModal): Refactor to use new modal provider
* Update modal provider props
* Use translation for DownloadDialog
* CR Update: Use useTranslation hook
* CR Update: Extract downloadDialog and rename to downloadContent
* CR Update: Fix typo
* CR Update: Rename download content to viewport download form
* feat: 🎸 Filter by url query param for seriesInstnaceUID
* fix: Set SR viewport as active by interaction (#1118)
* fix: Set SR viewport as active by interaction
* quick fix
* (eslint) add "before" as global variables
* add data-cy
* add data-cy
* create E2E test
* (E2E) create custom command to set layout size
* remove .only e2e
* remove throttle for onScroll
* feat: 🎸 Code review in progress
Code review. Move retrieveMEtadata load to separate folders. Some minor
code clean up
* feat: 🎸 Code review. Missing changes from previous commit
* feat: 🎸 Code review missing changes from previous commit
When sorting, Criteria for instance must use instanceNumber and not
instancesNumber
* feat: 🎸 Code review. Add more jsdoc info
* feat: 🎸 Code review. Prettify changed code
* feat: 🎸 Code review
Changed case for seriesInstanceUID. Use qido to filter (async). Fallback
to no params and same api method(async)
* fix: #1075 Returning to the Study List before all series have finished loading throws multiple errors in console
* fix: 🐛 Addressing review feedback from issue #1075
Adding support for cancelable pattern
Closes: #1075
* fix: 🐛 update unit test for exports
Closes: #1075
* feat: 🎸 New modal provider
A new modal provider/context component to unify modal related code and
simplify modal use
Closes: #1086
* refactor(ModalContext): Keep current modal lib
* Update modal provider version according to PR #1116
* Update modal provider props to use inner props
* Add custom class prop
* CR Update: Refactor provider to extract specific modal
* Fix modal import
* CR Update: Move from spread to object assign
* CR Update: Add proptypes, use classnames dependency over interpolation and rename modal
* fix: get adapter store picker to show
* fix: error caused by DICOMWeb "empty object" for patientName
* formatting
* refactor: kill the pills
* Indicate modalities is missing if not provided
* Consistent modality display
* Make sure modalities styles are applied
* revert config
* fix: 🐛 Fix ghost shadow on thumb
Updated react-dnd package for the lates. Refactored thumbnail component
Closes: 1087
* feat: 🎸 Code review
Set flex instead of float. Add cypress tag to thumb container
* feat: 🎸 Code review. Revised style when no description
* init
* use workbox-window
* Try to accomodate SW hosted at non root url
* v5 updated namespaces
* Test w/ no prompt
* add icons to manifest
* remove scope for now
* add icon tile information to entrypoint
* See if we can skirt by without a duplicate install
* shift client claims to skip waiting message
* Remove prepended slash
* comment out cache handler for now
* Simplify service-worker to skip waiting and install when a new worker is detected
* Use generated icons
* Update rollbar config
* fix: Set SR viewport as active by interaction
* quick fix
* (eslint) add "before" as global variables
* add data-cy
* add data-cy
* create E2E test
* (E2E) create custom command to set layout size
* remove .only e2e
* remove throttle for onScroll
* feat: 🎸 Load spinner when selecting gcloud store. Add key on td
* feat: 🎸 Code review. Translate loading study status
* feat: 🎸 Code review. Minor code refactoring
* fix: add missing study and index declerations
* Newer version of react-dates (calendar input component)
* Add useMedia hook
* Add useDebounce hook
* Export hooks for consumption
* Pull search filter into own component
* StudyListWithData to StudyListRoute
* Rename pagination component
* Rename Toolbar
* CustomeDateRangePicker to plain ol' css
* Remove unused styles
* Misc
* Magic to make our "OR" filters work
* Play w/ Mobile styles
* Fix pagination pushing out max width for viewport
* Tidy up small and medium templates
* Remove unecessary nesting
* Better mobile header and logo styles
* Fix studyList font size
* Remove unused code
* hookup on click
* Fix table search selectors
* Cypress uses medium width
* Remove irrelevant test
* Fix primary sort
* Fix sort for our smaller displays
* fix study list sometimes longer than specified rowsPerPage
* Simplify date picker styles
* restore study date filters
* Fix about table styling
* fix take value for search
* Fix header sizing for viewer and list route
* Remove dead code
* Restore isLoading
* Update platform/ui/src/components/studyList/StudyList.styl
Co-Authored-By: Rodrigo Antinarelli <rodrigoantinarelli@gmail.com>
* Attempt first usage of translations w/ hooks
* Pull from react-i18n library
* Fit presets on to a single line
* add no results message
* Show error message in studyList table
* Update input selectors for e2e tests
* Make sure messages span all columns
* fix test selector for table results
* Skip hover for result rows
* fix imports
* Date filtering w/ config
* Default export
* Pushed to PageComponent
* Remove DropZone
This previously only worked for null/undefined studies. I'm unable to trigger the state by testing the data. This "might" have happened when using a fresh/empty PACS; but even an empty array of studies would not show
* Try workaround for percy snapshot
* Return our modified document
* test: some dark magic to make it possible to take screenshots of WebGL canvases
* Apply transformation directly to test DOM
* investigate: window:before:load
* Register flag in correct place; add experimental-webgl check
* Remaining canvas fixes
* Remove comment
* E2E tests: Stack Scroll, Right-click to zoom, Middle-click to Pan
* About modal tests and added some cypress selectors
* Checking app version number
* Removed accidental import
* Review fixes
* feat: 🎸 Load spinner when selecting gcloud store. Add key on td
* feat: 🎸 Code review. Translate loading study status
* feat: 🎸 Code review. Minor code refactoring
* fix: add missing study and index declerations
* test: Improviment on VTK beforeEach hook (#1057)
* feat: 🎸 Some improvements on table cell and react lifecycle
* feat: 🎸 Prevent component update on a not searchData changing
* feat: 🎸 Some cypress improv to pass ci
* feat: 🎸 Code reviewe and also fixing list for gcloud
* feat: 🎸 Code review. Add spinner on first load of store
* feat: 🎸 Code review. Centering loading list spinner icon
* Adding Percy Snapshot on VTK, PDF and Microscopy tests
* Small improvement to always click on first search result
* Wait for search result
* Added timouts all over the code to avoid intermitted failures in CI
* Increasing Default Timeout and other configs
* Cleaning up the code
* try force enabling gpu
* Try custom executor with newer version of chrome
* Fix based on review comments
* Increased minimum thumbnails expected on VTK test
* Fix config validation
* Make sure we're using the chrome browser
* feat: 🎸 Progressive volume loading for vtk viewport
Adds progressive volume loading for vtkjs viewport whilst the frames are
streamed from PACs and rebuilt in the volume.
Closes: closes#1051
* Update to react-vtkjs-viewport 0.3.0
* Test cases for extensions: VTK, HTML and PDF
* Test Cases: Delete measurement, Relabel and Description through Contextmenu
* Fix: PDF, HTML and VTK extensions
* CI: running into different folders for PWA and Script-tag tests
* CI: running into different folders for PWA and Script-tag tests
* CI: running into different folders for PWA and Script-tag tests
* Fix for VTK tests and Cornerstone Eraser tool
* Removed VTK alias from Cornerstone aliases initialization
* Temporary fix for Eraser tool
* Test workaround
* Use juno for vtk
* Shift VTK tests to PWA folder
* feat: 🎸 Only allow reconstruction of datasets that make sense
Only allow reconstruction of datasets which are imaging data, that have
frames in the same orientation, with the same size and make sense to be
reconstructed in 3D.
Closes: #561
* feat: Implement a 'Exit 2D MPR' button in the toolbar to act as a toggle when '2D MPR' button is clicked
* Move logic to exit MPR and toggling into ConnectedPluginSwitch and PluginSwitch. Also, discard the ConnectedExitPluginSwitch and ExitPluginSwitch components.
* Update ToolbarRow.js
* show package bundle progress
* Ability to run "quick builds" for faster feedback on status checks (e2e)
* ci: speed up PR checks
* specify quick build value
* Fix for test scripts that were failing in CI
* Test cases for tools: Length and Angle
* Test cases implemented: Scroll series, Levels, Pan, Reset, Zoom
* Added test cases for tools: Cine, More, Layout, Invert, Rotate, FlipH, FlipV
* Fixes made according to comments on PR review
* Fixes for failing scenarios: Layout and Cine
* Test cases implemented: Cornerstone tools Clear and Eraser; Measurement panel Relabel and Description
* Fix for CI failure
* Better css chunk filenames depending on context
* skip minimize flag to speed up builds that are timing out (short-term fix)
* make sure our custom TerserJS config is actually applied
* Use SKIP_MINIMIZE flag for netlify and demo builds
* fix: 🐛 set current viewport as active when switching layouts
check if current viewport index is less than the current layout lenght
and set 0 if so or keep current selected cell index as active viewport
Closes: 999
* chore: 🤖 commenting out docker layer caching in the short-term
commenting out docker layer caching in the short-term
* refactor: 💡 refactor layout matrix index conditional
* Fix for test scripts that were failing in CI
* Test cases for tools: Length and Angle
* Test cases implemented: Scroll series, Levels, Pan, Reset, Zoom
* Added test cases for tools: Cine, More, Layout, Invert, Rotate, FlipH, FlipV
* Fixes made according to comments on PR review
* Fixes for failing scenarios: Layout and Cine
