From 2656ef88989e6f33bb35ffdf5add30d0ef035881 Mon Sep 17 00:00:00 2001 From: phroi <90913182+phroi@users.noreply.github.com> Date: Fri, 29 Nov 2024 03:36:43 +0700 Subject: [PATCH] Resolve feedback --- feedback.md | 46 +-- index.html | 8 +- package.json | 6 +- pnpm-lock.yaml | 751 +++++++++++++++++++++++----------------------- src/Action.tsx | 48 ++- src/Dashboard.tsx | 2 +- src/Form.tsx | 5 +- src/queries.ts | 2 +- src/utils.ts | 17 +- 9 files changed, 462 insertions(+), 423 deletions(-) diff --git a/feedback.md b/feedback.md index 19d5bb9..a5cdc1e 100644 --- a/feedback.md +++ b/feedback.md @@ -2,7 +2,7 @@ > Some other small things would be beneficial in addition to more explanation text, such as a link to the explorer once a transaction is submitted. [Jordan] -Yup, to do +Just displaying the tx status is ok, user can access the explorer with the title link > I think it'd be good if there some info about what ickb is, and have some sort of a wireframe for the landing page. you know like we should have blurb saying ickb is funded by the foundation with foundation logo etc and so on [Ahrom] @@ -10,7 +10,7 @@ Halfway done, not sure about the foundation logo on the homepage. > You will always get less ickb than what you exchanged. That’s because the lifetime earnings of the dao are baked into the ickb you receive correct? Looks like around 13%. [Sean] -Halfway done, now the exchange ratio is shown in the app and in the homepage there is a short explanation with links about the protocol. Not sure I want to add a tooltip in the app itself. +The exchange ratio is shown in the app and in the homepage there is a short explanation with links about the protocol. > Is there a wait to exchange ickb back to CKB? [Sean] @@ -31,109 +31,109 @@ Done > ![][image1] > Text is cut off on sides and bottom. Make sure line height is sufficient for vertically holding the text, or enable overflow on the container. Don’t cut off on the left and right. Find a different way. [Jordan] -TODO +Done > ![][image2] > Use “.toLocaleString()” to format the amount numbers. [Jordan] -TODO +Done > ![][image2] > The address should fit entirely or cut off with expected sizes. Use CSS clamp, calc, or breakpoints to adjust the size to fit based on resolution. If you need to cut off on smaller resolutions, use something more common like a fixed 8…8 characters or 16…16 characters. [Jordan] -TODO +Done > ![][image3] > The focus box on the amount looks strange. I’d recommend simply removing it. [Jordan] -TODO +Done > ![][image4] > This information has a lot of good detail, but it does not tell you anything about efficiency or difficulties that may occur. From the standpoint of a user, a standard deposit vs a limit order doesn’t matter. > > One way to go about this is to have a simple view and an advanced view. In the simple view, hide all of the details that are not needed except amounts, rate, and any helpful information. Is there an optimal amount they should use to get a better deal or have it go through faster? [Jordan] -TODO +Done > ![][image5] > This adds another step and potential confusion for the user. They should not have to refresh manually. Automatically refresh it if it needs to for the amounts. If it needs to refresh again immediately before signing, do it automatically. [Jordan] -TODO +Won't fix, when user refresh outputs may change, user have to acknowledge the updated tx > ![][image6] > This is a strange way to say it. I think you mean “waiting for transaction to confirm”. [Jordan] -DONE +Done > ![][image7] > I see this in the console, but it’s not on the page. This isn’t very user friendly. Once it is confirmed it needs to display a confirmation message at the very least. I have something similar to this on [https://ckb.tools/sudt](https://ckb.tools/sudt). [Jordan] -TODO +Done > ![][image8] > I don’t really understand this. Melting a completed order? Is this an open limit order? [Jordan] -TODO +Done > ![][image9] > Could use an info bubble to simply explain the exchange rate and how it is always decreasing slowly as the value of iCKB continues to rise. This should be extremely high level. The reason is you don’t want the average user, who didn’t read anything else, to think that the rate might be “bad” and search for a better time to do it. The rate is calculated so it’s never a bad time. [Jordan] -TODO +Done > ![][image10] > When hovering the mouse over the input field use the text cursor. [Jordan] -TODO +Done > ![][image11] > When the rate changes as a result of an automatic refresh, it is beneficial to give an effect to the text to show it changed. Sometimes this means giving it a bright shadow that fades out in a second. Something like that. [Jordan] -TODO +For now is ok, doesn't change enough to make a difference > ![][image12] > Use your horizontal space better on mobile devices. This usually means 10px padding around the sides. [Jordan] -TODO +Done > ![][image13] > Make sure you don’t have text overflows on things like buttons on mobile devices. [Jordan] -TODO +Done > ![][image14] > It will not let me type in a number greater than 0\. It is not clear on why. I see I have 1000 CKB. [Jordan] -TODO +Done > ![][image15] > Same as above going on here. I can’t exchange and there is a strange message about maturity. The user will not be clear on what is going on, or more importantly, what they need to do. [Jordan] -TODO +Done > ![][image16] > Received this message when sending transactions too quickly. If this is expected, that’s ok, but the user needs to be given an indication of some kind that it doesn’t work so they can resubmit. [Jordan] -TODO +Done > ![][image17] > When there is no amount typed in I see this. I believe this is indicating I have an open limit order that isn’t filled and this will cancel it? There is no place to see open orders I have that are pending, which is confusing. [Jordan] -TODO +Done > ![][image18] > A fee of 99.79 CKB is significant. Is this purely a fee or does this include the refundable cell cost? If it’s refundable it needs to be indicated. [Jordan] -TODO +Doesn't include cell cost. You were converting almost 100k CKB, that 99.79 CKB is a 0.1% Fee to the bot for covering the capital rent needed to run it. No fee for conversions using directly the core of the protocol. > ![][image19] > It is very difficult to select the number to delete it or copy it. Selecting the field to paste also seems problematic. [Jordan] -TODO +Done > ![][image20] > It looks like the button text might be selectable, which puts little corners outside the round corners. Disable text selection. [Jordan] -TODO +Done [image1]: diff --git a/index.html b/index.html index 4a14b04..ab1f31e 100644 --- a/index.html +++ b/index.html @@ -39,26 +39,26 @@

id="JoyID_mainnet" class="text-s min-h-12 w-full cursor-pointer rounded border-2 border-amber-400 leading-relaxed font-bold tracking-wider text-amber-400 uppercase disabled:cursor-default disabled:opacity-50" > - Yes, connect JoyID to Mainnet 🚧 (experimental) + Yes, connect JoyID to Mainnet - No, I'll feedback the dev 📝 + No, I'll feedback the dev - No, I'll use NervDAO 👀 + No, I'll use NervDAO