Skip to content
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

CARDS-2609: Configurable actions on user dashboard and related views #1853

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

acrowthe
Copy link
Contributor

@acrowthe acrowthe commented Dec 4, 2024

  • Add support for UI extensions to configure what actions should be allowed
    • Adds support for disableDeletion, disableMoving and disableCreation to form and subject clinician UIs where applicable
    • Adds support for disableActions and disableExpansion to subject and form dashboard views
  • Add support for seperate admin extensions for the clinician UIs
    • This allows seperate action configurations for admin and non-admin views

Within built in CARDS runmodes, these configurations are unused. This means that no UI changes should be detected when running CARDS in any built in runmode.

To Test:
As these configurations are unused, all UIs within this branch should remain untouched. Instructions on what should be tested to ensure this can be found in the related SPARC pull request. That PR also contains instructions and configurations for testing when functionality has been changed.

- Add support for UI extensions to configure what actions should be allowed
  - Adds support for disableDeletion, disableMoving and disableCreation to
    form and subject clinician UIs where applicable
  - Adds support for disableActions and disableExpansion to subject and form
    dashboard views
- Add support for seperate admin extensions for the clinician UIs
  - This allows seperate action configurations for admin and non-admin views

Within built in CARDS runmodes, these configurations are unused.
This means that no UI changes should be detected when running CARDS in any built in runmode.
Fix disableCreation configuration not being passed through properly
Add minimum height for live table data rows
- Improves appearance for rows with no avatars or icon buttons
@@ -65,6 +65,9 @@ const liveTableStyle = theme => ({
float : "right"
}
},
dataRow: {
height: "3em"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wouldn't min-height be better in case the content requires 2 lines?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

min-height doesn't work on table rows which is what this is applied to. height acts like min-height with the table row expanding vertically if needed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants