yearn / seafood

Seafood
https://seafood.yearn.watch
13 stars 21 forks source link

Bug: Vault dash filter UI overflows on laptop screens #166

Open murderteeth opened 1 year ago

murderteeth commented 1 year ago

image

tasks

Update the toolbar to use fixed widths for its different components and use ellipsis to truncate text overflow. Here's a template: image

udhaykumarbala commented 1 year ago

Hi @murderteeth, can i work on it

murderteeth commented 1 year ago

Hello @udhaykumarbala, instead of fixing this specific overflow problem, we may redesign the toolbar a bit. It will be at least several days, but I'll post a new spec here and AT you. If you have ideas on that, feel free. I'm also going to post a couple more issues tomorrow or day after that might interest you.

udhaykumarbala commented 1 year ago

Hello @udhaykumarbala, instead of fixing this specific overflow problem, we may redesign the toolbar a bit. It will be at least several days, but I'll post a new spec here and AT you. If you have ideas on that, feel free. I'm also going to post a couple more issues tomorrow or day after that might interest you.

For this issue, i thought of two ideas

  1. truncate with dots like showing "vaults, v2,..."
  2. just show the filter icon, since the state is already shown on click

just thoughts🙈, feel free to ignore.

Happy to work on more issues, Thanks for your kind guidance

murderteeth commented 1 year ago

Your welcome and thank you for contributing @udhaykumarbala! Can you explain your idea #2 a little more?

Truncating with dots (ellipsis) has proved tricky actually, because the filters are laid out with relative width as I recall. Or anyway, I couldn't get it to work in a sane way when I tried! That's one of the reasons for a slight re-design.. the current design makes it hard to do things like adding ellipsis to truncate labels =)

udhaykumarbala commented 1 year ago

Your welcome and thank you for contributing @udhaykumarbala! Can you explain your idea #2 a little more?

Truncating with dots (ellipsis) has proved tricky actually, because the filters are laid out with relative width as I recall. Or anyway, I couldn't get it to work in a sane way when I tried! That's one of the reasons for a slight re-design.. the current design makes it hard to do things like adding ellipsis to truncate labels =)

Agreed

murderteeth commented 1 year ago

Hiya @udhaykumarbala, I just updated this issue with a suggestion on how to implement, let me know what you think. No major re-design for now.

I also just posted https://github.com/yearn/seafood/issues/169, have a look ;)

udhaykumarbala commented 1 year ago

Hi @murderteeth, I added the ellipsis to the filter chip label. It seems to fix the problem without setting the width

CleanShot 2023-07-29 at 12 52 47@2x

Do I still need to add width?

and in the mobile, also the vault and network overflow out of the screen sometimes CleanShot 2023-07-29 at 12 57 46@2x

Can we add an ellipsis to all screen sizes?

udhaykumarbala commented 1 year ago

170

murderteeth commented 1 year ago

Good question about mobile @udhaykumarbala! See my pr review comment.