Skip to content

Conversation

@Isabella-Mitchell
Copy link
Contributor

@Isabella-Mitchell Isabella-Mitchell commented Nov 20, 2025

Resolves JIRA: https://bbc.atlassian.net/browse/WS-869

Summary

Align WS bylines (AMP + canonical) with PS plans

Code changes

  • Updates JSX and CSS to make bylines match PS designs (figma)
    -- renders "horizontally"
    -- changes to hover/ focus styles
    -- does not render images for multiple contributors
    -- does not render twitter handles
  • Keeps semantic HTML unchanged (nested unordered lists)
  • Correctly attributes authorTopicUrls
  • Permits 'byline' or 'subByline' block types
  • Updates/ adds unit tests

Not done as part of this work

  • Change layout for single contributor with image (will raise a ticket)
  • Match PS visually hidden text and use of 'and' (will check with product since I believe we require translations)
  • Permit multiple bylines/ subBylines like PS do (WS do not do this, and changing this is out of scope of this work)

Developer Checklist

  • UX
    • UX Criteria met (visual UX & screenreader UX)
  • Accessibility
    • Accessibility Acceptance Criteria met
    • Accessibility swarm completed
    • Component Health updated
    • P1 accessibility bugs resolved
    • P2/P3 accessibility bugs planned (if not resolved)
  • Security
    • Security issues addressed
    • Threat Model updated
  • Documentation
    • Docs updated (runbook, READMEs)
  • Testing
    • Feature tested on relevant environments
  • Comms
    • Relevant parties notified of changes

Testing

  • Manual Testing required?
    • Local (Ready-For-Test, Local)
    • Test (Ready-For-Test, Test)
    • Preview (Ready-For-Test, Preview)
    • Live (Ready-For-Test, Live)
  • Manual Testing complete?
    • Local
    • Test
    • Preview
    • Live

Additional Testing Steps

See these testing links

Useful Links

gridTemplateColumns: 'repeat(2, auto)',
},
paddingBottom: `${spacings.TRIPLE}rem`,
lineHeight: '1.35rem', // not sure I should do this due to script sizes
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure if I should do this due to Script Sizes. The benefit is that gives a bit more breathing room on latin languages (e.g. News, Mundo). And if I look at a service with a larger script (e.g. Sinhala), then I can see the default service line height overwrites this. The negative is that the designs only account for latin designs.

@Isabella-Mitchell Isabella-Mitchell added the a11y-swarm An a11y swarm (clarify dev or full team in the desc) needs to be carried out before moving to test label Nov 26, 2025
@Isabella-Mitchell Isabella-Mitchell marked this pull request as ready for review November 26, 2025 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a11y-swarm An a11y swarm (clarify dev or full team in the desc) needs to be carried out before moving to test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants