spacemeshos / smapp-lite

The light wallet application for Spacemesh network
3 stars 2 forks source link

Design #62

Closed andreivcodes closed 2 months ago

andreivcodes commented 3 months ago

styling and layout according to the design at https://www.figma.com/design/bLEBwz5KSP6CNRZ2ooyJto/Light-wallet-files?node-id=0-1&t=ZOgYNwfzZScZwOrO-0

brusherru commented 3 months ago

It breaks up the mobile version.

Needs to be fixed and tuned for mobile. Left comprehensive comments in Slack, here are some screenshots for the protocol.

image image image image image

Does not check accounts with transactions and rewards, vault account with unlocked balance, and how it works on desktop.

pigmej commented 3 months ago

I dislike it, honestly. It is way too wide, and functionality is affected by it.

Smidgh is also I guess not defined shortcut too.

dioexul commented 3 months ago

I would prefer not to introduce that many changes before the launch. The main idea was to change colours for existing components, not to change components itself (this requires too much validation).

andreivcodes commented 3 months ago
Screenshot 2024-08-06 at 2 09 17 AM Screenshot 2024-08-06 at 2 09 23 AM Screenshot 2024-08-06 at 2 09 32 AM Screenshot 2024-08-06 at 2 10 18 AM Screenshot 2024-08-06 at 2 10 34 AM

Updated with fixes for mobile

brusherru commented 3 months ago

Well, it's better, but still:

  1. Horizontal scroll on the unlock page. The fact that some pages still has this problem, and some not means that something implemented wrong. It should be part of basic template / components.

  2. Transaction list still has excessive padding

    image
  3. Regression: Vault account lost the feature that shows unlocked amount (https://github.com/spacemeshos/smapp-lite/pull/46)

  4. I don't like that modals now lost the "darkening" of the underlaying content (or this darkening is almost invisible)

    image

    Also I think such a big paddings are excessive. Let's use the space wisely.

  5. And I totally does not like this part:

    image

    There are two problems:

    • a lot of empty space on mobile because all of this does not fit in a single line
    • address, account name and switch, and it's balance are falling apart and seems not related

    For clarity, here is what we currently have:

    image

    All account related info is grouped in the single block

  6. Rewards looks well enough, but I'd tweak padding a little bit to make it looks more accurate

    image
  7. Big amounts looks bad:

    image
  8. Not sure that it makes sense to use a full width of the page on a desktop. It's harder to work with.

    image

Let's work on this better and without rush. I think there is nothing critical to release the wallet as is, less "fancy" but more solid and logical. And later release a really good GUI.

noamnelke commented 3 months ago

One thing I didn't see mentioned is the color choice. I think the "really dark green" and "slightly less dark, but still really dark green" don't have enough contrast between them. I had to turn up the brightness on my display because I wasn't even sure it was two different colors...

brusherru commented 2 months ago

I still dislike it, but since we really need to officially release it and we need to have it "branded" — I'm merging it. Hopefully all defects (design & code) will be fixed very soon.