Skip to content
This repository has been archived by the owner on Nov 17, 2021. It is now read-only.

o-teaser-header and label spacing #129

Closed
thurstontye opened this issue May 15, 2019 · 2 comments
Closed

o-teaser-header and label spacing #129

thurstontye opened this issue May 15, 2019 · 2 comments
Labels
bug Something isn't working
Projects

Comments

@thurstontye
Copy link
Contributor

Bug

What

Loss of space between o-teaser-header and o-labels elements in some contexts but not all. Think it just needs some padding on o-teaser-header a so will raise a PR to that effect.

Details

example on streams
Also at the bottom of article pages in the [topic tracker related content] (https://www.ft.com/content/aaaf613c-768d-11e9-be7d-6d846537acab)

Additional information

See: https://trello.com/c/CkoIA7pZ/889-padding-issue-with-topic-tracker

@thurstontye thurstontye added the bug Something isn't working label May 15, 2019
@thurstontye
Copy link
Contributor Author

On reflection this may be an issue with o-labels

@chee chee added this to incoming in Origami ✨ Feb 1, 2020
@notlee
Copy link
Contributor

notlee commented Apr 28, 2020

Sorry for the lack of response here. Looking at the examples I think this must have been fixed. x-dash adds the premium label within x-teaser, using a space between the end of the heading and the teaser. Let us know if this is stll an issue though

@notlee notlee closed this as completed Apr 28, 2020
Origami ✨ automation moved this from incoming to complete Apr 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
Origami ✨
  
Done
Development

No branches or pull requests

2 participants