tum-esi/wade

[UI] min-height / min-width not working correctly

eileenoo opened this issue · 7 comments

A minimum height and width of the electron window was added with this issue: #48
Due to this, the view is scrollable horizontally when the size expands the min. width.

Somehow the bg is not updated accordingly (see screenshot)
Screenshot 2020-06-16 at 13 27 54

The min. width / height can either be removed (as the UI was updated to resize, or this needs to be fixed.

Then it can be removed, @Fady222 what do you think and do you see the same behavior?

I can confirm bug.
Removing minimum height and width would allow the user to basically minimize the window all the way down to being not visible at all. If that is acceptable then it can be removed. @egekorkan @neniman

A quick fix would be to disable scrolling in CSS

Edit: added quick fix

I would say that there should be a minimum but if this CSS fix does not solve it, then we should remove it.
@Fady222 where is the fix?

@egekorkan Didn't do it yet. Would do it by today or by tomorrow at most

I can confirm bug.
Removing minimum height and width would allow the user to basically minimize the window all the way down to being not visible at all. If that is acceptable then it can be removed. @egekorkan @neniman

A quick fix would be to disable scrolling in CSS

Edit: added quick fix

If you disable scrolling, some content would not be visible.. so we have to find another solution / remove it 🌝 @Fady222

This issue is already resolved. Should we close it? @egekorkan