search

MemePlace / Frontend

Frontend Repo
3 stars 0 forks source link

fixes sidebar padding issue and lints #88

Closed bandgeekdante closed 6 years ago

bandgeekdante commented 6 years ago

If the dialog is too large, Material doesn't resize it and it blows up past the screen without a scroll bar. I did have (in a previous version) everything wrapped in a mat-dialog-content, but this enforced a maximum height that I thought was a bit too small. If we can change that behaviour then that could be another option

Step7750 commented 6 years ago

Another thing I was thinking about was whether the dialog width should be fixed across all images. We could then center the image if need be but we want consistency and for comments to not be crammed if the image is not very wide.