Skip to content

Fix: Set width and height attributes for Bluesky SVG#3122

Merged
HarHarLinks merged 1 commit into
matrix-org:mainfrom
awtj8o81ryywg793:fix/footer-bluesky-icon-display-issue
Dec 25, 2025
Merged

Fix: Set width and height attributes for Bluesky SVG#3122
HarHarLinks merged 1 commit into
matrix-org:mainfrom
awtj8o81ryywg793:fix/footer-bluesky-icon-display-issue

Conversation

@awtj8o81ryywg793
Copy link
Copy Markdown
Contributor

Description

Add width and height attributes to the Bluesky icon SVG. The lack of these attributes is causing a visual bug. See #3121.

Related issues

Closes #3121.

Role

Individual not affiliated with any project relevant to this PR.

Timeline

There’s no immediate deadline, but considering that the bug prevents users from clicking the Bluesky link, the sooner, the better.

Signoff

Please sign off your individual commits or whole pull request.

Signed-off-by: Jack S. <181536874+awtj8o81ryywg793@users.noreply.github.com>
@awtj8o81ryywg793 awtj8o81ryywg793 marked this pull request as ready for review December 25, 2025 02:03
@awtj8o81ryywg793 awtj8o81ryywg793 requested a review from a team as a code owner December 25, 2025 02:03
Copy link
Copy Markdown
Contributor

@HarHarLinks HarHarLinks left a comment

Choose a reason for hiding this comment

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

You're right, thank you. Not sure how this could have slipped through.

I wonder why these aren't styled using CSS instead anyway?

@HarHarLinks HarHarLinks merged commit 86f88a1 into matrix-org:main Dec 25, 2025
5 checks passed
@awtj8o81ryywg793 awtj8o81ryywg793 deleted the fix/footer-bluesky-icon-display-issue branch December 25, 2025 10:52
@awtj8o81ryywg793
Copy link
Copy Markdown
Contributor Author

Quoted from a previous comment:

I wonder why these aren't styled using CSS instead anyway?

That wouldn't be a bad idea, though it doesn't hurt to have a "default" value defined as an attribute.

@HarHarLinks
Copy link
Copy Markdown
Contributor

I tried in devtools and it seemed you can't override with css if an attribute is set 😕

@awtj8o81ryywg793
Copy link
Copy Markdown
Contributor Author

Quoted from a previous comment:

I tried in devtools and it seemed you can't override with css if an attribute is set 😕

I've just tried overwriting the styles via "Developer Tools," and I'm able to do so just fine.

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.

Bug: Bluesky icon is missing from the footer

2 participants