Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

Responsive design can be improved for small devices #962

Closed
ghost opened this issue Dec 17, 2014 · 13 comments
Closed

Responsive design can be improved for small devices #962

ghost opened this issue Dec 17, 2014 · 13 comments
Assignees
Milestone

Comments

@ghost
Copy link

ghost commented Dec 17, 2014

Toolbar title, breadcrumb and directive name are poorly displayed:
capture d ecran 2014-12-17 a 13 33 49

A proposed solution would be to switch to menu when width is smaller than 771 pixels instead of 600.

@ThomasBurleson
Copy link
Contributor

When the content width [box area with Basic Usage] is great than 900px, then the SideNav component is locked into the open position. Less than that and it auto-hides.

@mzbyszynski
Copy link
Contributor

@ThomasBurleson Are you sure? I think currently on the documentation site, the sidenav is locked open to gt-sm, which is greater than 600px not 900px. Maybe it should be gt-md?

  <md-sidenav layout="column"
              style="overflow: hidden; display: flex;"
              class="md-sidenav-left md-whiteframe-z2"
              md-component-id="left"
              md-is-locked-open="$media('gt-sm')">

@ThomasBurleson
Copy link
Contributor

Doe the commit c62b970536d on master not address this correctly?

<md-sidenav class="md-sidenav-left md-whiteframe-z2" md-component-id="left" md-is-locked-open="$media('gt-md')">

Or perhaps I am misunderstanding something ?

Note this will be available later this week in a tagged 0.7.0-rc1 release.

@mzbyszynski
Copy link
Contributor

Hi @ThomasBurleson, The screenshot that @MCBL posted wasn't of the md-sidenav demo (which I agree is working correctly), it was the progress-circular demo with the overall sidenav that is used to navigate the documentation site, so I took that to mean his issue was with that sidenav, but I'm not totally sure 😄 . I do think it might make sense to hide the sidenav on the documentation site for medium sized devices though...

@ThomasBurleson
Copy link
Contributor

@MCBL - So this issue is about the responsive layout of the title bar text ? Missed that... sorry.

@ghost
Copy link
Author

ghost commented Dec 19, 2014

Exactly yes, it's the header bar text that is poorly displayed. It clearly needs more space. That's why I propose to switch to menu when width is smaller than 771 pixels instead of 600.

@gkalpak
Copy link
Member

gkalpak commented Dec 19, 2014

@ThomasBurleson: The commit you mentioned is for the sidenav demo.
@MCBL is basically proposing the same change for the Docs App's sidenav.

@marcysutton
Copy link
Contributor

Closed with bfc947f

@ghost
Copy link
Author

ghost commented Jan 12, 2015

It is still poorly displayed. Juste resize your browser on page with a lot of text. It's ugly!
capture d ecran 2015-01-12 a 11 46 24

@marcysutton
Copy link
Contributor

Are you looking at material.angularjs.org or a local build from master? We are still working to get the site updated, which won't reflect these changes.

@ghost
Copy link
Author

ghost commented Jan 13, 2015

At material.angularjs.org. Ok all right, thanks.

@ashishpatel02
Copy link

@ThomasBurleson as you said "When the content width [box area with Basic Usage] is great than 900px, then the SideNav component is locked into the open position. Less than that and it auto-hides."

How can i fix to open for all the resolutions?

@ThomasBurleson
Copy link
Contributor

@ashishpatel02 - try md-is-locked-open="true"

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

Successfully merging a pull request may close this issue.

5 participants