Proof-Of-Chaos / website

https://proofofchaos.app
MIT License
2 stars 2 forks source link

Make website mobile friendly - Integrate Nova Wallet #35

Open gabrieljaegerde opened 1 year ago

gabrieljaegerde commented 1 year ago

Incorporate this feedback

niklasp commented 1 year ago

so this is about wallets on mobile right? I am an android user and one idea eould be to show a link to the play store to nova wallet and an info text that shows. install nova?

any iphone users here who cqn test/ recommend an app? does nova work on ios?

also this would mean quite some dev work because we would need to alter the nova connect modal and write our own from that. vurrently we use an out of the box component from their repo

gabrieljaegerde commented 1 year ago

I have played around a little with nova and it's pretty neat.

First of all we should not display the talisman option to mobile users right since that is desktop only and could be confusing.

Is the nova connect modal currently on beta? Where do i see it? Why do we need to write our own connect modal?

niklasp commented 1 year ago

we are using the component from their repo. its basically only a nova demo component with no config optiona other than css styling. to be more individual we have to write our own component.

niklasp commented 1 year ago

https://github.com/nova-wallet/nova-wallet-dapp-js/wiki/Integration-guide-for-DApp

niklasp commented 1 year ago

Screenshot_20221117-091124_Nova.png

fix overflow, menu buttons on mobile

gabrieljaegerde commented 1 year ago

Can we please fix the overflow issue on Home Page and About Page before we push the new code.

niklasp commented 1 year ago

fixed

gabrieljaegerde commented 1 year ago

It would be great to let mobile users know to use nova wallet instead of giving them the talisman/polkadot.js options when connecting a wallet.