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 vinteum logo colors #243

Merged
merged 1 commit into from
Jan 7, 2025
Merged

Conversation

plebhash
Copy link
Collaborator

@plebhash plebhash commented Jan 6, 2025

vinteum logo was added via #242

unfortunately the dark background makes it very hard to read it

image

this PR turns the letters white

image

Copy link

netlify bot commented Jan 6, 2025

Deploy Preview for startling-capybara-2231ea ready!

Name Link
🔨 Latest commit b39ed56
🔍 Latest deploy log https://app.netlify.com/sites/startling-capybara-2231ea/deploys/677c4d6758438d0008347804
😎 Deploy Preview https://deploy-preview-243--startling-capybara-2231ea.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@pavlenex
Copy link
Collaborator

pavlenex commented Jan 6, 2025

Did they approve of logo changes? They didn't have light version when I looked ,so I assume you just edited it, if so, in such cases best is to get approval, I agree white looks better but it may not be according to brand guidelines.

@lucasdcf
Copy link

lucasdcf commented Jan 7, 2025

lgtm

@pavlenex pavlenex merged commit cc24dea into stratum-mining:main Jan 7, 2025
5 checks passed
@plebhash plebhash deleted the fix-vinteum-logo branch January 7, 2025 15:17
@plebhash plebhash mentioned this pull request Jan 18, 2025
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.

3 participants