Problem/Motivation

The contact form lists the recipients but it uses a comma separated list. This in not accessible or semantic.

Proposed resolution

Use the item list with comma list style.

Remaining tasks

Review
Commit

User interface changes

The contact listing uses an item list for recipients.

API changes

None

Data model changes

None

CommentFileSizeAuthor
#3 commas.png83.61 KBstefan.r
#2 2557577.2.patch2.42 KBalexpott
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alexpott created an issue. See original summary.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented
Status: Active » Needs review
Issue tags: +SafeMarkup
FileSize
2557577.2.patch2.42 KB
stefan.r’s picture

stefan.r CreditAttribution: stefan.r commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community
FileSize
commas.png83.61 KB

The checkPlain is covered by automated tests, manually tested the comma list and it looks great:

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.0.x, thanks!

  • catch committed c9072a4 on 8.0.x 
    Issue #2557577 by alexpott: Use comma separated item list in contact...
andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost at Skilld commented
Related issues: +#267492: Update storage of contact form email recipients

Status: Fixed » Closed (fixed)

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