search

BuildOnViction / bug-reports

TomoChain's Bug Report & Support
https://tomochain.com
17 stars 5 forks source link

notification column redundant #170

Closed 0xwhalerush closed 4 years ago

0xwhalerush commented 4 years ago

To Reproduce Steps to reproduce the behavior:

  1. Place order 1-2 filled
  2. Click notifitacion

when taking action on the Tomo dex the notification column only increases, whereas currently only taking 1-2x actions only the notification column is so wide

IMG_20200308_011927-picsay

0xwhalerush commented 4 years ago

the notification column should increase when we take action.but in this case the column is too redundant when just doing some actions

thanhson1085 commented 4 years ago

After discussed with team, we do NOT consider it an issue. We keep the max height of the box.

0xwhalerush commented 4 years ago

but this shouldn't be that big. I suggest every time a new box is added

thanhson1085 commented 4 years ago

Thanh you for you suggestion. But team decided to keep the current height of the box, not depends on the number of notifications.

On Thu, Mar 12, 2020 at 11:51 AM Herooo notifications@github.com wrote:

but this shouldn't be that big. I suggest every time a new box is added

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/tomochain/bug-reports/issues/170#issuecomment-598005480, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEFN3DUCIFONOV57HFQ2BTRHBS63ANCNFSM4LDR7IIQ .

0xwhalerush commented 4 years ago

please see other markets / dex