Skip to content

Conversation

@Mikescops
Copy link
Member

Migrate the last views and remove the old grid.

@Mikescops Mikescops requested a review from skjnldsv October 15, 2020 09:23
@Mikescops Mikescops self-assigned this Oct 15, 2020
@Mikescops Mikescops added 3. to review Waiting for reviews enhancement New feature or request labels Oct 15, 2020
@skjnldsv
Copy link
Member

/compile /

Copy link
Member

@skjnldsv skjnldsv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works nicely!! 🚀
Found one issue that doesn't seems related, missing padding on top of timeline

Capture d’écran_2020-10-15_12-04-19

@skjnldsv skjnldsv added this to the Nextcloud 21 milestone Oct 15, 2020
@Mikescops
Copy link
Member Author

@skjnldsv that's a bug when we forget to reset the grid i think.
Can you try to reproduce it?

@skjnldsv
Copy link
Member

Can you try to reproduce it?

It happens on initial load for me 🤔

@Mikescops
Copy link
Member Author

😨 what, not me...

@Mikescops
Copy link
Member Author

Can you force refresh the page? Maybe you got cache I don't know

@skjnldsv skjnldsv force-pushed the feature/grid-migration branch 2 times, most recently from c6e0735 to f6d61ac Compare October 15, 2020 10:39
@skjnldsv
Copy link
Member

skjnldsv commented Oct 15, 2020

Can you force refresh the page? Maybe you got cache I don't know

Well, paddingTop is set to 0

padding: 0px #{$marginW}px 256px #{$marginW}px;

@Mikescops
Copy link
Member Author

You should have the first title of the month that do a padding, that's what we discussed with Jan, you likely don't have a title, so it should be an issue with the js

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants