canonical / canonical/vanilla-framework

[Component amendment]: Notifications, enable lists in notification message

Open
#5,496 4 comments 0 reactions 0 assignees View on GitHub
Priority: Medium Tier: Global WG: Validated
Dominant language
HTML
Stars
983
Forks
216
Avg merge
3d 13h
Merged PRs (30d)
3

Description

## Component/pattern to amend

[Notification](https://vanillaframework.io/docs/patterns/notification)

## Visual

![Image](https://github.com/user-attachments/assets/462928fa-29fe-4293-bf2a-53d859055ba9)

## Context

The Notification component doesn't expect lists, and while it works, it generates invalid markup, because it only expects text.
I'd like to display error notifications for payment issues, that could have multiple reasons or ways to resolve, breaking up these for the users would improve readability. I wanted to know if there are any guidance on the preferred size of notification messages.

Contributor guide

Open the contributing guide

Research direction

Start with the Notification pattern documentation at vanilla-framework.io/docs/patterns/notification and inspect the component entry point and its existing markup guidance. Confirm how lists should be represented without invalid markup, and document or demonstrate the preferred notification message size when the change is complete.

Written by the indexing model from the issue text.

Assessment

Tech stack
html
Domain
frontend
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.