Skip to content

Conversation

@skjnldsv
Copy link
Member

@skjnldsv skjnldsv commented Apr 30, 2020

Before
Kazam_screenshot_00010
Kazam_screenshot_00011
Kazam_screenshot_00013


After
Kazam_screenshot_00008
Kazam_screenshot_00009
Kazam_screenshot_00007

Broke by other stuff and #19913
Closes #19541
Fix #19479


We should have some minimal css for this, to not rely on the header.scss file.
This is supposed to be a minimal public template and we load 140kb of css ^^'

Signed-off-by: John Molakvoæ (skjnldsv) <[email protected]>
@skjnldsv
Copy link
Member Author

Not sure, but I'll look at the backport
/backport to stable18

@skjnldsv
Copy link
Member Author

/backport to stable17

@georgehrke
Copy link
Member

Tested with public calendars 👍

78472450-59A7-4CB0-8BEB-D96425CEC1C4

@MorrisJobke MorrisJobke added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Apr 30, 2020
@rullzer rullzer mentioned this pull request Apr 30, 2020
11 tasks
@skjnldsv
Copy link
Member Author

Failures unrelated

Copy link
Member

@jancborchardt jancborchardt left a comment

Choose a reason for hiding this comment

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

Great fix, thanks! :)

@jancborchardt jancborchardt merged commit ae665e2 into master Apr 30, 2020
@jancborchardt jancborchardt deleted the fix/public-design branch April 30, 2020 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish bug design Design, UI, UX, etc. feature: sharing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Three dots menu not viewable on mobile browser in portrait view when longish file name used

6 participants