search

getAlby / nostr-wallet-connect

Nostr Wallet Connect (NIP-47) application to allow apps to connect to your node
https://nwc.getalby.com
Apache License 2.0
103 stars 31 forks source link

feat: Logged account will be shown to navbar at nwc.getalby.com #180

Closed anishyadavv closed 8 months ago

anishyadavv commented 9 months ago

Solves: #158 here are the Screen shots of changes Big screens image

image image For Small devices image image image

stackingsaunter commented 9 months ago

Thanks @anishyadavv !

I think it will be better without that profile icon at all, also on desktop

anishyadavv commented 9 months ago

Hii @stackingsaunter, okay I'll remove profile icon

anishyadavv commented 9 months ago

Hey @rolznz and @stackingsaunter , I have removed the avatar icon. Now you can review my code

anishyadavv commented 9 months ago

All changes done Now you can review @rolznz

rolznz commented 9 months ago

I think this is the best we can do with the current frontend setup. Linking to https://github.com/getAlby/nostr-wallet-connect/issues/183 because with a React app we could do a cleaner approach (e.g. use a UI library with pre-made select/dropdown components)

bumi commented 8 months ago

@rolznz merge it if you've reviewed it.

rolznz commented 8 months ago

Thanks for your work @anishyadavv ! :rocket:

anishyadavv commented 8 months ago

You're welcome! @rolznz Happy to contribute to such an awesome project

bumi commented 8 months ago

awesome @anishyadavv \o/