Fix scrolling for new design #1205
Conversation
Signed-off-by: Louis Chemineau <louis@chmn.me>
|
TagContent and floders views are also broken. Not sure about On this day, favorites, people and shared with you |
On this day and favorites are using the Timeline component so they are fixed by this PR. Can you push a fix for people and tags ? Should be the same logic as for Albums and AlbumContent |
| @@ -530,14 +531,13 @@ export default { | |||
| } | |||
|
|
|||
| &__photos { | |||
| height: 100%; | |||
| height: calc(100% - 60px); | |||
There was a problem hiding this comment.
This makes FileListViewer the scrollable element, but shouldn't the app content be the scrollable element?
There was a problem hiding this comment.
Ah, we probably need vue v7 for that, right?
There was a problem hiding this comment.
At this point, we don't really care which element is scrollable, no ?
|
Superseded by #1206 |
Signed-off-by: Louis Chemineau louis@chmn.me