kodadot / nft-gallery

Generative Art Marketplace
https://kodadot.xyz
MIT License
595 stars 346 forks source link

Profile-fix-visuals #10153

Closed daiagi closed 2 weeks ago

daiagi commented 2 weeks ago

Thank you for your contribution to the Koda - Generative Art Marketplace.

👇 __ Let's make a quick check before the contribution.

PR Type

Needs Design check

Context

Did your issue had any of the "$" label on it?

netlify[bot] commented 2 weeks ago

Deploy Preview for koda-canary ready!

Name Link
Latest commit 3bfe476ffc653ed285533ae7359f972afa4bf02b
Latest deploy log https://app.netlify.com/sites/koda-canary/deploys/66326858692e6d0008553d63
Deploy Preview https://deploy-preview-10153--koda-canary.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.

daiagi commented 2 weeks ago

@exezbcz

use our disabled button for farcaster disabled button - turn it all grey

not sure what you mean. codewise it is disabled exactly like any other button is disabled

do you mean change the bg color from purple to grey when disabled

rest of stuff mentioned in the issue is fixed, i think

exezbcz commented 2 weeks ago

image

image

image

capital bold please

daiagi commented 2 weeks ago

center align please capital bold please

@exezbcz Done

prury commented 2 weeks ago

mobile buttons too close and also close without capital C:

image

prury commented 2 weeks ago

title needs some centering on mobile:

image

daiagi commented 2 weeks ago

thx @prury

done

codeclimate[bot] commented 2 weeks ago

Code Climate has analyzed commit 3bfe476f and detected 0 issues on this pull request.

View more on Code Climate.

sonarcloud[bot] commented 2 weeks ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

daiagi commented 2 weeks ago

@prury

i'm merging this

@exezbcz

plz open new issue if any visual imperfections remain

Thank you