Skip to content

Fixed firefox guest header height - #7470

Merged
rullzer merged 2 commits into
masterfrom
guest-flex-fix
Dec 12, 2017
Merged

Fixed firefox guest header height#7470
rullzer merged 2 commits into
masterfrom
guest-flex-fix

Conversation

@skjnldsv

Copy link
Copy Markdown
Member

After #7249

capture d ecran_2017-12-12_16-11-46

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
@skjnldsvskjnldsv added 3. to review Waiting for reviews bug design Design, UI, UX, etc. medium regression labels Dec 12, 2017
@skjnldsvskjnldsv added this to the Nextcloud 13 milestone Dec 12, 2017
@skjnldsvskjnldsv self-assigned this Dec 12, 2017
@jancborchardt

Copy link
Copy Markdown
Member

It’s fine by me. :) But FYI on mobile there’s a scrollbar very early (which was the issue originally to be solved by this whole working on the log in screen I think ;)
screenshot from 2017-12-12 16-38-19

@skjnldsv

skjnldsv commented Dec 12, 2017

Copy link
Copy Markdown
MemberAuthor

@jancborchardt what kind of mobile is this? This looks very small! :/

EDIT there is an empty div which is 70px high named '.push', any idea why it's there? @jancborchardt

@jancborchardt

Copy link
Copy Markdown
Member

@skjnldsv it’s Firefox responsive design mode, but looks the same on a Fairphone 2 for example (5" screen). And keep in mind the keyboard will also be shown there.

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
@codecov

codecovBot commented Dec 12, 2017

Copy link
Copy Markdown

Codecov Report

Merging #7470 into master will not change coverage.
The diff coverage is n/a.

@@ Coverage Diff @@## master #7470 +/- ##
=========================================
Coverage 51.11% 51.11% Complexity 24900 24900 =========================================
Files 1601 1601 Lines 94772 94772 Branches 1367 1367 =========================================
Hits 48438 48438 Misses 46334 46334

@skjnldsv

Copy link
Copy Markdown
MemberAuthor

Okay, fixed, It was an old div designed to push the footer 😱

@jancborchardtjancborchardt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me now! 🚀 :)

@rullzerrullzer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Lets do this!

@rullzer
rullzer merged commit f55dd96 into masterDec 12, 2017
@rullzer
rullzer deleted the guest-flex-fix branch December 12, 2017 20:49
@MorrisJobkeMorrisJobke mentioned this pull request Dec 13, 2017
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbugdesignDesign, UI, UX, etc.mediumregression

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@skjnldsv@jancborchardt@rullzer