bcgov / nr-theme

Common repo for FSA theme
Apache License 2.0
2 stars 0 forks source link

Decide on spacing for labels & content when showing read-only text #83

Open jackwingsam opened 1 year ago

jackwingsam commented 1 year ago

SPAR lead the way on designing the style/components for displaying read-only text that is important to display to users. CLIENT subsequently followed the same style.

A good time to review and commit to or re-do the standards for read-only content.

jackwingsam commented 10 months ago

Example from SPAR that is ok but somewhat difficult to parse through quickly

Image

jackwingsam commented 10 months ago

Example 1 from CLIENT that is spaced awkwardly.

Image

Example 2 from CLIENT that looks easy to parse and consume/review.

Image

jackwingsam commented 10 months ago

@alkolot to try some alternatives on Team Alliance/CLIENT and report back.

https://carbondesignsystem.com/patterns/read-only-states-pattern/#formatting

jackwingsam commented 9 months ago

Won't try and standardize for whole FDS program. Everyone application to find their own appropriate to show read-only text.

alkolot commented 9 months ago

We've resolved the read only text in Client to be more readable. The bigger issue I see with the SPAR screen is the amount of repeated (and serving no purpose) text.