Skip to content

Conversation

@wesleyboar
Copy link
Member

@wesleyboar wesleyboar commented Nov 13, 2025

Overview

All .svg inherit color & size from context.

Related

Changes

  • changed SVG internal code
  • added styles

Testing

  1. Show logos via Icons.
  2. Show logos via Snippet using template share_on_social.html.
  3. Verify color is inherited.
  4. Hover over linked SVG.
  5. Verify color changes.
    Color of brand of software, or TACC's accent color. Check out Instagram 😉

UI

default.icon.size.and.color.and.ux.mov

@wesleyboar wesleyboar changed the title feat: logo icons inherit color feat: logo icons inherit color & size Nov 13, 2025
@wesleyboar wesleyboar force-pushed the feat/icons-inherit-color-from-context branch from d271ae1 to f9e77c7 Compare November 13, 2025 23:17
@wesleyboar wesleyboar force-pushed the feat/icons-inherit-color-from-context branch from 148627a to a09a4c9 Compare November 14, 2025 00:43
@wesleyboar wesleyboar merged commit e6685c0 into main Nov 14, 2025
@wesleyboar wesleyboar deleted the feat/icons-inherit-color-from-context branch November 14, 2025 00:52
wesleyboar added a commit that referenced this pull request Nov 14, 2025
* feat: logo icons inherit color

* feat: organization logos inherit color

* feat: core logo inherit color

* feat: logos inherit size

* fix: ending `<p>` tag

* feat: Icon svg inherit size

* fix: no empty underlined space on hover

* fix: tacc formal logo fill wrong

* fix: bluesky logo duplicate code

* feat: color icons

* fix: bust cache
wesleyboar added a commit that referenced this pull request Nov 14, 2025
feat: logo icons inherit color & size (#1027)

* feat: logo icons inherit color

* feat: organization logos inherit color

* feat: core logo inherit color

* feat: logos inherit size

* fix: ending `<p>` tag

* feat: Icon svg inherit size

* fix: no empty underlined space on hover

* fix: tacc formal logo fill wrong

* fix: bluesky logo duplicate code

* feat: color icons

* fix: bust cache
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.

2 participants