ChainSafe / Multix

Use a Multisig to do anything, on Polkadot
https://multix.chainsafe.io/
Apache License 2.0
27 stars 21 forks source link

Make the "connect" button of variant "primary" #389

Closed asnaith closed 1 year ago

asnaith commented 1 year ago

The current color of the "Connect" button lacks sufficient contrast against the background colour and is not unique in comparison to other components in the interface. Establishing a connection to the dApp is a pre-requisite for most feature interactions other than watch mode.

The current low contrast can make it difficult for it to stand out as a prominent part of the interface, especially for those with visual impairments.

If we make it easier for all users to visually distinguish we may increase the attention to this button and increase the chance of users connecting their wallet as one of their first interactions with Multix.

273064579-8dce769e-146d-429b-b291-551f582fc5ff

Acceptance Criteria: The new colour should provide a clear and distinct visual separation from the background and other components, drawing attention and making it easier for all users to identify.

Tbaut commented 1 year ago

Maybe I missed something, but regarding the contrast, I think this is sufficient tbh, just checking on https://coolors.co/contrast-checker/18191a-fafcfd it's basically black on basically white :shrug: Also all our buttons with "secondary" variant are all like this.

asnaith commented 1 year ago

To be clearer of my intention, my idea was to make this button more prominent/noticeable than others. I'll edit the description.

Essentially, makes it easier to distinguish and more prominent, to encourage users to connect to the dApp as one of their first actions.

Tbaut commented 1 year ago

Ok, so it's more about having it of variant primary (blue). I think that's ok. My main concern is that I think we should only have 1 primary button per page. But since there's actually not much you can do when you're only watching multisigs (there's no "new transaction', no "create multisig" nor "approve/reject" I guess we can make it blue then :)