Skip to content
This repository has been archived by the owner on Mar 11, 2024. It is now read-only.

Asset list modal is too small #685

Closed
nop33 opened this issue May 4, 2023 · 3 comments
Closed

Asset list modal is too small #685

nop33 opened this issue May 4, 2023 · 3 comments
Assignees
@nop33
Labels
bug Something isn't working
Milestone
v2.1.0

Comments

@nop33
Copy link
Member

nop33 commented May 4, 2023

@JKtranslator reported this:

image
@nop33 nop33 added the bug Something isn't working label May 4, 2023
@nop33
Copy link
Member Author

nop33 commented May 4, 2023

Can't reproduce it 😢

image

@nop33
Copy link
Member Author

nop33 commented May 4, 2023 •
edited
Loading

@mvaivre not sure why this code is not working as expected here (I gotta say, I am not so happy with how the width of the popup is being handled at the moment, it feels fragile.

Maybe for now we could just increase the min width of the popup nad give the small popups (like network switch) a width of their own.

@nop33 nop33 mentioned this issue May 4, 2023
Merged
@nop33
Copy link
Member Author

nop33 commented May 4, 2023

Steps to reproduce is:

  • Add second asset
  • Remove first asset
  • Open asset selection modal

@nop33 nop33 modified the milestones: v2.x.x - NEXT, v2.1.0 May 15, 2023
@nop33 nop33 self-assigned this Jun 27, 2023
@nop33 nop33 closed this as completed in 8314fc3 Jul 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@nop33 nop33

Labels
bug Something isn't working
Projects
None yet
Milestone
v2.1.0
Development

No branches or pull requests
1 participant
@nop33