Skip to content

Improve display of Process List in email notifications#6976

Merged
aduth merged 1 commit intomainfrom
aduth-email-process-list
Sep 16, 2022
Merged

Improve display of Process List in email notifications#6976
aduth merged 1 commit intomainfrom
aduth-email-process-list

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented Sep 16, 2022

Why: So that the Process List as displayed in email messages is consistent with the design system reference implementation.

It's assumed this is a side-effect of the revisions in #6736, where an increase in default email line-height of 1.3 to 1.5 caused an offset of the appearance of the text of the list item number. Explicitly assigning the line-height ensures correct math.

Screenshots:

Before After
process-list-before process-list-after

**Why**: So that the Process List as displayed in email messages is consistent with the design system reference implementation.

It's assumed this is a side-effect of the revisions in #6736, where an increase in default email line-height of 1.3 to 1.5 caused an offset of the appearance of the text of the list item number. Explicitly assigning the line-height ensures correct math.

changelog: Bug Fixes, Email Templates, Improve appearance of Process List in email template
@aduth aduth requested review from a team and sheldon-b September 16, 2022 15:12
Copy link
Contributor

@eileen-nava eileen-nava left a comment

Choose a reason for hiding this comment

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

Looks good to me. 👍🏻

@aduth aduth merged commit 71b42f5 into main Sep 16, 2022
@aduth aduth deleted the aduth-email-process-list branch September 16, 2022 19:48
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