Skip to content

Remove display: contents from metadata layout - #17

Merged
raunaqgupta merged 1 commit into
mainfrom
05-15-2026-remove-display-contents
May 16, 2026
Merged

raunaqgupta merged 1 commit into
mainfrom
05-15-2026-remove-display-contents

Conversation

@raunaqgupta

Copy link
Copy Markdown
Owner

Summary

  • Flatten the metadata <dl> so <dt>/<dd> pairs render as direct grid items of .cooklang-meta-list
  • Drop the .cooklang-meta-item wrapper and the display: contents rule, since display: contents historically had inconsistent accessibility-tree support

Test plan

  • Render a recipe with multiple metadata entries and confirm the key/value two-column grid layout is unchanged
  • Verify the title still renders prominently above the list

🤖 Generated with Claude Code

Flatten the metadata <dl> so <dt>/<dd> pairs are direct grid items,
removing the need for a wrapper div with display: contents (which
has historically had patchy accessibility tree support).

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@raunaqgupta
raunaqgupta merged commit 80549f6 into main May 16, 2026
2 checks passed
@raunaqgupta
raunaqgupta deleted the 05-15-2026-remove-display-contents branch May 16, 2026 04:48
@raunaqgupta raunaqgupta mentioned this pull request May 16, 2026
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.

1 participant