Skip to content

fix: social media icon excess whitespace#768

Merged
wesleyboar merged 3 commits intomainfrom
fix/new-social-media-icon-whitespace
Dec 14, 2023
Merged

fix: social media icon excess whitespace#768
wesleyboar merged 3 commits intomainfrom
fix/new-social-media-icon-whitespace

Conversation

@wesleyboar
Copy link
Copy Markdown
Member

@wesleyboar wesleyboar commented Dec 14, 2023

Overview

Remove excess whitespace under social media icons.

Related

Changes

  • added CSS to remove whitespace

Testing

  1. Verify size of social media icon links is the size of the social media icons.

UI

<a> <svg>
a svg

wesleyboar and others added 3 commits December 14, 2023 10:18
* feat: TUP-553 hide byline prefix

* docs: TUP-553 simpler byline comments

* feat: TUP-553 new byline layout

* feat: TUP-553 new byline layout, page vs list
@wesleyboar wesleyboar marked this pull request as ready for review December 14, 2023 18:17
@wesleyboar wesleyboar changed the base branch from main to feat/tup-584-news-metadata December 14, 2023 18:17
@wesleyboar wesleyboar changed the base branch from feat/tup-584-news-metadata to main December 14, 2023 18:17
@wesleyboar wesleyboar changed the title Fix/new social media icon whitespace fix: social media icon excess whitespace Dec 14, 2023
@wesleyboar wesleyboar merged commit 97fea1b into main Dec 14, 2023
@wesleyboar wesleyboar deleted the fix/new-social-media-icon-whitespace branch December 14, 2023 18:18
wesleyboar added a commit that referenced this pull request Dec 14, 2023
wesleyboar added a commit that referenced this pull request Dec 14, 2023
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