Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Header fixes and public note #10838

Merged
merged 8 commits into from
Aug 24, 2018
Merged

Header fixes and public note #10838

merged 8 commits into from
Aug 24, 2018

Conversation

skjnldsv
Copy link
Member

Fix headers apps calculation
Added breakpoints on mobile for the public share
Fixed order of shares list (public on top)
Fix header menu mobile break

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
@MorrisJobke
Copy link
Member

@skjnldsv Please check out the failing acceptance and jsunit tests

Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
…tcloud/server into header-fixes-and-public-note

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
@skjnldsv skjnldsv force-pushed the header-fixes-and-public-note branch from 3ffac6b to 03b112c Compare August 24, 2018 15:41
@skjnldsv skjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Aug 24, 2018
@MorrisJobke
Copy link
Member

@skjnldsv JS Unit still fails 😕

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
@rullzer
Copy link
Member

rullzer commented Aug 24, 2018

@rullzer rullzer added the 2. developing Work in progress label Aug 24, 2018
@rullzer rullzer removed the 4. to release Ready to be released and/or waiting for tests to finish label Aug 24, 2018
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
@skjnldsv skjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 2. developing Work in progress labels Aug 24, 2018
@MorrisJobke MorrisJobke merged commit 8761856 into master Aug 24, 2018
@MorrisJobke MorrisJobke deleted the header-fixes-and-public-note branch August 24, 2018 20:21
@MorrisJobke MorrisJobke mentioned this pull request Aug 24, 2018
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. enhancement feature: sharing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants