search

chrisleekr / binance-trading-bot

Automated Binance trading bot - Trade multiple cryptocurrencies. Buy low/sell high with Grid Trading. Integrated with TradingView technical analysis
MIT License
5.09k stars 1.1k forks source link

feat: Collapsable ProfitLoss header #624

Closed rando128 closed 1 year ago

rando128 commented 1 year ago

Description

A small UX improvement that makes the Closed Trades and Opened Trades accordions collapsable (same behaviour than the Account Balance accordion). On mobile, these accordions are collapsed by default.

The tooltip text of the Closed Trades header that was missing has been added.

This PR only impacts only public/js/ProfitLossWrapper.js file.

Motivation and Context

On mobile when you are trading multiple asset bases (USDT, ETH, BTC, BUSD...), these headers tend to be quite long forcing the user to scroll couple of times before reaching the symbol section.

How Has This Been Tested?

This is working on my production setup.

Screenshots (if appropriate):

Screenshot 2023-04-16 at 19 24 50
chrisleekr commented 1 year ago

Yap, ok. Looks good, as usual.

Merged!