Skip to content

Fix line-break issue in MiradorLink card novel titles#285

Open
csbailey5t wants to merge 2 commits into
mainfrom
fix/card-title-linebreak
Open

Fix line-break issue in MiradorLink card novel titles#285
csbailey5t wants to merge 2 commits into
mainfrom
fix/card-title-linebreak

Conversation

@csbailey5t

Copy link
Copy Markdown
Collaborator

Summary

  • Adds a <br /> between "for" and the novel title span in MiradorLink.astro so the title always starts on its own line
  • Prevents long titles like "David Copperfield" from wrapping mid-word inside the card

Closes #273

Test plan

  • Check each novel landing page (Bleak House, David Copperfield, Hard Times, Little Dorrit) — novel title should appear on its own line below "Explore Dickens's Working Notes for"
  • Verify at mobile viewport widths as well

🤖 Generated with Claude Code

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@vercel

vercel Bot commented May 17, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ddnp Ready Ready Preview, Comment May 17, 2026 7:27pm
ddnp-staging Ready Ready Preview, Comment May 17, 2026 7:27pm

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@csbailey5t

Copy link
Copy Markdown
Collaborator Author

@annamgibson and @grenerad I edited so that the novel title always on its own line. I tried just pushing doing a break after "for," but then the second line was sometimes really long and cumbersome, so trying this instead. Let me know what you think.

@annamgibson

annamgibson commented May 19, 2026

Copy link
Copy Markdown
Contributor

Reviewed -- the initial line break works well, but it's creating a long final line that isn't well padded in the box.
@csbailey5t Can we edit the text so that it reads:

View Dickens's working notes for
novel title in Mirador

Changes:

  • Change "explore" to "view" to avoid repetition
  • Lowercase working notes
  • cut "and read our Editorial Annotations
  • move "in Mirador" up to the same line as the novel title

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.

Fix line-break issue across lines in cards on WN landing pages

2 participants