Skip to content
This repository was archived by the owner on Sep 18, 2020. It is now read-only.

Feature/sidenav fix - #186

Open
KapRom wants to merge 5 commits into
developfrom
feature/sidenav-fix
Open

KapRom wants to merge 5 commits into
developfrom
feature/sidenav-fix

Conversation

@KapRom

@KapRom KapRom commented May 7, 2018

Copy link
Copy Markdown
Contributor
  • Fixed the horizontal scroll of sidenav
  • Fixed the squeezing of main content on mobile version
  • On desktop the sidenav is as before - open by default
  • On mobile the sidenav is hidden by default, opened is over the main content:
    image

Comment thread src/app/app.component.ts Outdated
isMobile() {
const innerWidth = window.innerWidth || document.documentElement.clientWidth || document.body.clientWidth;

if (innerWidth < this.minimalPixels) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

simplify: return innerWidth < this.minimalPixels;

@gmrukwa

gmrukwa commented Jun 16, 2018

Copy link
Copy Markdown
Collaborator

I've just seen this review, after almost 2 months hanging... :D

@gmrukwa

gmrukwa commented Jun 16, 2018

Copy link
Copy Markdown
Collaborator

Fix sidenav

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants