Minor style updates, fixing the "combined" steps provider #112
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.
Overview
This takes a few of the style updates I had applied separately in #110 so I could release them more quickly. It also fixes the "combined" steps provider; currently it errors if you don't have garmin or fitbit enabled for your study. So it now checks the data collection status.
The style updates are mostly tweaking the default border radius to 12px and tweaks the default dropshadow based on some of the designs @kyledehovitz-ce has been putting out lately; so things can stay consistent / aligned.
It also changes the line-height to 1.2 instead of 1.2em, so it's applied consistently across the page (I forgot how this worked; 1.2em just sets it statically whereas 1.2 is relative to font size of the element) as this fixes an acute issue with titles in the view builder.
Security
REMINDER: All file contents are public.
Describe briefly what security risks you considered, why they don't apply, or how they've been mitigated.
Checklist
Testing
Documentation
Consider "Squash and merge" as needed to keep the commit history reasonable on
main
.Reviewers
Assign to the appropriate reviewer(s). Minimally, a second set of eyes is needed ensure no non-public information is published. Consider also including: