Closed andaleliz closed 10 months ago
@mtcA6 we forgot to make a ticket for this during planning last week. I've added it to the current sprint.
I reviewed the feedback from team reviews, and want to drop some comments here to acknowledge and capture what I changed, especially since @pamela-drouin is going to design intent while I'm out. Thank you @pamela-drouin @mattmarino-adhoc @Samara-Strauss @mtcA6 @tpharrison and @adamwhitlock1 for sharing your feedback with me!
<ol>
, <ul>
, or <dl>
, and if that impacts the copy/paste-ability of the text.+5 more...
treatment.<hr>
, and making the duty status an h4 in option A, and reducing the heading size for details to an h5. For option B, I opted for the <hr>
and bolding the duty status as regular body text. It looked weird to me to have it styled as a heading with such minimal other information exposed on the page.I feel like we're in a great place for design intent + research, and am closing this ticket for now since most folks are out and I won't be able to change anything if design intent happens while I'm away. I think we can address further changes in a new ticket.
Background
We have some outstanding feedback from the team captured in Sketch comments that needs to be revisited and potentially incorporated into the design prior to design intent.
Tasks
AC