Problem/Motivation

UI Kit update https://github.com/civictheme/uikit/issues/642 introduces a new variable to set the vertical spacing on messages. This should be brought back to the status message template to correctly space Drupal system messages.

CommentFileSizeAuthor
#11 status message vertical spacing.png677.87 KBdanielgry

Comments

alan.cole created an issue. See original summary.

alan.cole’s picture

https://github.com/civictheme/monorepo-drupal/pull/1393

1. Set the vertical_spacing on the message component for system messages.

alan.cole’s picture

Status: Active » Needs review
richardgaunt’s picture

Assigned: Unassigned » danielgry
richardgaunt’s picture

richardgaunt’s picture

richardgaunt’s picture

  • 28007528 committed on 1.x
    Issue #3530263: Support message component updates in 642 (#1393)
    
fionamorrison23’s picture

danielgry’s picture

StatusFileSize
new677.87 KB

Verified and tested on latest dev version

Testing Results:
1. Verified that system messages no longer have default outer margin applied
2. Confirmed the 'Both' type of vertical spacing is applied to status messages
3. Verified status message message displays correctly across different viewports

Screenshots:
image

danielgry’s picture

Assigned: danielgry » richardgaunt
Status: Needs review » Reviewed & tested by the community
fionamorrison23’s picture

Version: 1.11.0 » 1.12.0
Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

fionamorrison23’s picture

Assigned: richardgaunt » Unassigned

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.