-
Notifications
You must be signed in to change notification settings - Fork 58
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
social redesign #844
social redesign #844
Conversation
Deploying mutiny-web with Cloudflare Pages
|
7f8bda3
to
de3bdef
Compare
ee3e247
to
0051b1c
Compare
rebased to master, relies on MutinyWallet/mutiny-node#992 |
7fb9bcc
to
b5a1f06
Compare
Looks like it needs a rebase and maybe an pnpm lock update? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This feels awesome, really nice work. I made some notes testing it:
- Should the backup prompt be above the contacts search? Looks a little broken up currently.
- For the empty hints, should clicking on the hints themselves do the action?
- Quick actions icon don't work on either the home page or in messages
- Make message sending instant, instead of blocking. Just have a pending icon instead of saying 'Just now' until it sends.
- For "everyone" should we pull from the global feed? Or change that to "friends" instead of everyone since that's just the local friend list? Maybe a call to action (eventually) to follow some people instead of a blank list.
- Tags are not vertically centered in their pill bubble
- Remove the lightning address "elegant-giraffe@mutiny.plus" part for now, replace with an "add lightning address" link that does nothing for now?
- For the nostr npub at the bottom, maybe say "Nostr Identity" instead? Just to give some indication about what it is.
- In DMs, "send" should be grayed out if they do not have a lightning address
- In DMs, paying a normal invoice should not allow the option of anon vs private switcher, it should just be private since it's just an invoice
- In a follow up, would it be possible to show a notification for friends that have recently messaged you? I think that'll be important without having to click into the DM first
addressed most of tony's comments above. except for:
noticed a few things along the way:
|
That's the only thing I got, looks good! Can create issues for other follows up and throw them in the project board. My chat with mutinynet works so you may have to get with @benthecarman on why it's not working for you locally. |
Not sure what could be going wrong with paul's dms, maybe he's sent to many something is crashing? |
I did the same
Might be private zaps or directly from a minibits wallet or something |
that was my assumption, there were some showing up as private in my feed though, we probably just aren't handling some properly |
sweet thank you for this list, will work through these today. I've been patching up the tests because a lot of nav has changed that breaks the tests |
okay got most of these @benthecarman (with your help)
I know you're still looking into
|
I gotta update all the tests for the new ui. so far I have
|
found a bunch of little ui problems when testing on mobile:
Can't figure this one out:
|
App feels pretty good, nice work! Mostly visual/cleanup things but one core functionality that I think should be in this release is adding friends. |
Yeah we haven't cut an rc in awhile |
Having some trouble with the fedimint test on the send timing out, curious if a rc3 would fix that |
can you run it on master and see? What specifically is the error? |
ran it on master, similar flakiness so doesn't seem to be a mutiny-node problem |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Huge, very well done on this. Feel free to make any small patches that you like and then merge.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚢 it
No description provided.