Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix : updated Twitter new logo to X #235

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

aialok
Copy link

@aialok aialok commented Oct 26, 2023

Update Twitter Logo on Atproto Website - Issue #219

@aialok
Copy link
Author

aialok commented Oct 26, 2023

here's how the website look now !
hope it will get merged @pfrazee

Updated-page
updated-page-2

Copy link
Contributor

@bnewbold bnewbold left a comment

Choose a reason for hiding this comment

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

Other than the Windows/Mac thing looks good to me.

Though i'd also be fine with keeping the old twitter bird logo instead of the X 😈

Comment on lines -260 to +268
"@next/swc-darwin-arm64@12.3.1":
"@next/swc-win32-x64-msvc@12.3.1":
version "12.3.1"
resolved "https://registry.npmjs.org/@next/swc-darwin-arm64/-/swc-darwin-arm64-12.3.1.tgz"
integrity sha512-hT/EBGNcu0ITiuWDYU9ur57Oa4LybD5DOQp4f22T6zLfpoBMfBibPtR8XktXmOyFHrL/6FC2p9ojdLZhWhvBHg==
resolved "https://registry.npmjs.org/@next/swc-win32-x64-msvc/-/swc-win32-x64-msvc-12.3.1.tgz"
integrity sha512-W1ijvzzg+kPEX6LAc+50EYYSEo0FVu7dmTE+t+DM4iOLqgGHoW9uYSz9wCVdkXOEEMP9xhXfGpcSxsfDucyPkA==
Copy link
Contributor

Choose a reason for hiding this comment

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

I assume this is included because the last commit was from a dev on Apple hardware and you are running Windows. As context almost all the team uses Macs, but I run Linux, and we run CI/deployments on Linux.

I'm not sure what the impact of these lines changing will be for other devs.

Copy link
Author

Choose a reason for hiding this comment

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

Thanks for the context, @bnewbold !

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.

None yet

2 participants