jackburrus
commented
9 months ago - [x] The notification message for transaction confirmation is only visible for a brief 2 seconds.
- [x] The error message for insufficient balance is currently displayed at the bottom. Shouldn't it be positioned closer to the input field that's causing the error? (Decided not to implement this change.)
- [x] When attempting to remove more liquidity than available in an open LP position, an error message is missing in the "Remove Liquidity" modal. Suggestion: Place an "exceeds balance" message below the "Remove Liquidity" button.
- [x] In various modals (Open Long, Open Short, Remove Liquidity, Close Short), when "Savings DAI" is selected from the dropdown, it's displayed as "SDAI" instead of "sDAI" (with a lowercase 's'). Note: This has been reviewed, and no change will be made as "SDAI" is directly pulled from the contract
- [x] The transaction preview section of the "Open Short" modal is missing fee information, specifically the "Pool fee" (i.e., Curve fee) displayed for Long positions. This issue will require multiple pull requests to address due to necessary changes in hyperwasm.
- [x] Update the hover text on the current value in the "Open Long/Short" positions to: "Profit/loss since open, after closing fees."
- [x] Enhance the display of hover text over market metrics to show when hovering over the metric itself, not just the text description below it. Adjust the volume to display two decimal places, eliminate the secondary tooltip, and integrate its information into the main description tooltip.
- [x] Reconsider the date format for open/closed positions within the app. The current format (11/12/2024) can be confusing due to regional differences in date interpretation. Suggested format: 12 Nov, 2024
- [x] Evaluate whether only certain values should be bolded in the open position cells to improve readability and emphasis. Decided to only bold fixed rate and current value on longs, and current value on shorts.
akhamisa