Skip to content
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

Merged
merged 3 commits into from
Mar 25, 2024
Merged

social redesign #844

merged 3 commits into from
Mar 25, 2024

Conversation

futurepaul
Copy link
Collaborator

No description provided.

Copy link

cloudflare-workers-and-pages bot commented Jan 22, 2024

Deploying mutiny-web with  Cloudflare Pages  Cloudflare Pages

Latest commit: 12a8cf8
Status: ✅  Deploy successful!
Preview URL: https://b1e8c895.mutiny-web.pages.dev
Branch Preview URL: https://new-home.mutiny-web.pages.dev

View logs

@futurepaul futurepaul force-pushed the new-home branch 3 times, most recently from 7f8bda3 to de3bdef Compare January 29, 2024 17:21
@futurepaul futurepaul force-pushed the new-home branch 3 times, most recently from ee3e247 to 0051b1c Compare February 6, 2024 11:43
@futurepaul
Copy link
Collaborator Author

rebased to master, relies on MutinyWallet/mutiny-node#992

@futurepaul futurepaul force-pushed the new-home branch 3 times, most recently from 7fb9bcc to b5a1f06 Compare February 15, 2024 15:49
src/components/Activity.tsx Outdated Show resolved Hide resolved
src/components/ActualSearch.tsx Outdated Show resolved Hide resolved
@TonyGiorgio
Copy link
Collaborator

Looks like it needs a rebase and maybe an pnpm lock update?

Copy link
Collaborator

@TonyGiorgio TonyGiorgio left a 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

src/components/PendingNwc.tsx Outdated Show resolved Hide resolved
@futurepaul
Copy link
Collaborator Author

futurepaul commented Feb 19, 2024

addressed most of tony's comments above.

except for:

  • 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 (this would be fun)
  • Make message sending instant, instead of blocking. (this is hard to do without causing a flicker that feels worse, at least in my attempts so far)

noticed a few things along the way:

  • somehow broke loading my chat with mutinynet. loading other chats works. I think it's a mutiny-node problem no 100% sure though (@benthecarman)
Screenshot 2024-02-19 at 6 08 57 PM
  • can end up on a send screen and clicking "back" doesn't work??
  • for getting the zap feed I should ask the backend "what's my contact list?" (instead of asking primal api)
  • minifab menu on desktop isn't above the minifab button

@TonyGiorgio
Copy link
Collaborator

  • Should pressing enter make the DM send button go?

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.

@benthecarman
Copy link
Collaborator

benthecarman commented Feb 20, 2024

  • On firefox I am just getting a blank screen, tried clearing storage and its still happens. (its not my dark mode plugin).
  • On a fresh wallet the first time I hit receive it took awhile for the USD amount to show up
  • Something we talked about before, not sure if we want to get in here, but removing the password after backup could be a little nice
  • instinctively I click on people's faces in my activity when trying to send to them, maybe the face should go to chat, or the activity item view should have a link to get to the chat, maybe both
  • definitely agree on pressing enter to send in dms
  • should we have some of CTA inside the empty chat state, it feels weird with nothing there
  • see a couple weird zaps in the friends feed like this, if i click on their face it doesn't take to the chat like the others do image

@benthecarman
Copy link
Collaborator

Not sure what could be going wrong with paul's dms, maybe he's sent to many something is crashing?

@TonyGiorgio
Copy link
Collaborator

TonyGiorgio commented Feb 20, 2024

instinctively I click on people's faces in my activity when trying to send to them, maybe the face should go to chat

I did the same

couple weird zaps in the friends feed like this

Might be private zaps or directly from a minibits wallet or something

@benthecarman
Copy link
Collaborator

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

@futurepaul
Copy link
Collaborator Author

  • On firefox...

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

@futurepaul
Copy link
Collaborator Author

futurepaul commented Feb 20, 2024

okay got most of these @benthecarman (with your help)

  • On firefox I am just getting a blank screen, tried clearing storage and its still happens. (its not my dark mode plugin). (this is an adblocker problem with the new icon lib... don't think it's a problem when we ship but we should double check)
  • On a fresh wallet the first time I hit receive it took awhile for the USD amount to show up (idk what's up with this)
  • Something we talked about before, not sure if we want to get in here, but removing the password after backup could be a little nice
  • instinctively I click on people's faces in my activity when trying to send to them, maybe the face should go to chat, or the activity item view should have a link to get to the chat, maybe both
  • definitely agree on pressing enter to send in dms
  • should we have some of CTA inside the empty chat state, it feels weird with nothing there
  • see a couple weird zaps in the friends feed like this, if i click on their face it doesn't take to the chat like the others do

I know you're still looking into

  • anon zaps not showing up

@futurepaul
Copy link
Collaborator Author

futurepaul commented Feb 20, 2024

I gotta update all the tests for the new ui. so far I have

  • routes
  • encrypt
  • fedimint
  • load
  • restore
  • roundtrip

@futurepaul
Copy link
Collaborator Author

Screenshot 2024-02-22 at 4 18 17 PM

@futurepaul
Copy link
Collaborator Author

futurepaul commented Feb 22, 2024

found a bunch of little ui problems when testing on mobile:

  • overflow scroll (my old nemesis)
  • balance isn't centered
  • the "shiny" edges of buttons are thicker on mobile? (maybe need 0.5px on sm screens?)
  • can I make the multiple tab screen less ugly? (always see it when moving seed to new device)
  • it's time to toggle hide balance we've waited long enough
  • really bugs me that the amount editor screens don't squeeze down when the keyboard is open
  • feels a little empty with just one tx and all the prompts gone?
  • need to add nsec import if I'm not gonna have setup screen
  • can I get rid of the weird camera view resize flicker? (old problem)

Can't figure this one out:

  • don't suggest autofill in search (old problem)

@TonyGiorgio
Copy link
Collaborator

  • There's a few places where it says "your activity is private by default" when it comes to nostr profile information. This is not true and should be removed.
Screenshot 2024-03-13 at 3 55 47 PM
  • The text at the top in general is redundant and unnecessary.

  • It should probably let people set a lightning address in general until it's ready. Also if they are not subscribers then they don't get a mutiny one but can set the lightning address from another place?

  • The federation QR code is showing up as undefined. This is not a problem on signet currently. @benthecarman is this a hotfix problem? The federation invite code didn't get applied to v0.6.0-rc?

  • should both of these be here for the same payment?

Screenshot 2024-03-13 at 4 02 00 PM
  • there's not a flow for adding friends. This takes the user to a search screen, where they can "find" a contact but can't do anything with them except pay or request funds. Clicking on one of these users will 'add to contact' but doesn't actually add them in any meaningful way. I think there needs to be an "add friend" button somewhere when searching or finding a contact. Remember, we're a nostr client now, and one where this might be their first client. Adding friends/following is a necessary part of that social functionality.
Screenshot 2024-03-13 at 4 04 14 PM
  • on that note, I say we remove the npub import completely and force npriv import or nothing at all. If we want to remove browser extension support, that's fine too. Paul has been struggling with this and if it helps in any way I say we remove anything that isn't going to be covered for a majority of new users with the focus on the ios/android app.

  • there's something about these home screen call to action "buttons" that they don't really feel like buttons. And when they are stacked up, it is not very visually appealing. This will be what everyone lands on, can we make it look and feel better? The "secure funds" one should be way more pressing for urgency at the very least. It just feels like it blends in.

Screenshot 2024-03-13 at 4 11 01 PM

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.

@benthecarman
Copy link
Collaborator

Yeah we haven't cut an rc in awhile

@futurepaul
Copy link
Collaborator Author

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

@TonyGiorgio
Copy link
Collaborator

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?

@futurepaul
Copy link
Collaborator Author

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

@benthecarman
Copy link
Collaborator

Something I noticied when testing. This is an on-chain payment but it says I have 0 balance and the activity item has a lightning bolt.

Maybe the balance at the top should be an aggregate or at least display the highest.

image

@benthecarman
Copy link
Collaborator

When I click on an activity item the UI in the background gets squished into the corner

image

Copy link
Collaborator

@TonyGiorgio TonyGiorgio left a 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.

Copy link
Collaborator

@benthecarman benthecarman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚢 it

@futurepaul futurepaul marked this pull request as ready for review March 25, 2024 19:36
@futurepaul futurepaul changed the title wip social redesign social redesign Mar 25, 2024
@futurepaul futurepaul merged commit 74a9f07 into master Mar 25, 2024
5 checks passed
@futurepaul futurepaul deleted the new-home branch March 25, 2024 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants