ui icon indicating copy to clipboard operation
ui copied to clipboard

Update Outdated Twitter Logo to Current Branding

Open Jagadeeshftw opened this issue 1 year ago • 2 comments
trafficstars

Description

This PR addresses the issue of updating the Twitter icon to the latest version by updating the Twitter SVG icon.

Related Issue

Fixes #554

Changes Made

Verified and updated the Twitter SVG icon in the assets. Replaced the outdated SVG with the latest version of the Twitter icon.

Screenshots

Dark mode:

image

Light mode:

image

Jagadeeshftw avatar Aug 09 '24 00:08 Jagadeeshftw

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
decentraland-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 4, 2024 5:19pm

vercel[bot] avatar Aug 09 '24 00:08 vercel[bot]

@LautaroPetaccio Can you please review this?

Jagadeeshftw avatar Aug 20 '24 23:08 Jagadeeshftw

Hey @Jagadeeshftw,

I noticed that the first commit is not signed, which is preventing the PR from being available for merge. Could you please sign it? This is a requirement from the base branch. 🙏

Thanks again for your help!

kevinszuchet avatar Sep 04 '24 16:09 kevinszuchet

Maybe this can help https://stackoverflow.com/a/59351278

kevinszuchet avatar Sep 04 '24 16:09 kevinszuchet

Is it fine now @kevinszuchet ?

Jagadeeshftw avatar Sep 04 '24 16:09 Jagadeeshftw

Could you try leaving only the original commit that is now correctly signed?

kevinszuchet avatar Sep 04 '24 17:09 kevinszuchet

Let me create new PR

Jagadeeshftw avatar Sep 04 '24 17:09 Jagadeeshftw