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

Mobile Support #41

Open
JohnMcLear opened this issue Mar 17, 2013 · 7 comments
Open

Mobile Support #41

JohnMcLear opened this issue Mar 17, 2013 · 7 comments

Comments

@JohnMcLear
Copy link

Is there any active work on better support for mobile devices?

@JohnMcLear
Copy link
Author

CSS I ended up using

@media all and (max-width: 400px){
  #gritter-notice-wrapper{
    max-height:172px;
    overflow:hidden;
    width:100% !important;
    background-color: #ccc;
    bottom:20px;
    left:0px;
    right:0px;
    color:#000;
  } 
  .gritter-close {
    display:block !important;
    left: auto !important;
    right:5px;
  }
  #gritter-notice-wrapper.bottom-right{
    left:0px !important;
    bottom:30px !important;
    right:0px !important;
  }
  .gritter-item p{
    color:black;
    font-size:16px;
  }
  .gritter-title{
    text-shadow: none !important;
    color:black;
  }
  .gritter-item{
    padding:2px 11px 8px 4px;
  }
  .gritter-item-wrapper{
    margin:0;
  }
  .gritter-item-wrapper > div{
    background: none;
  }

It's obviously pretty hacky because of the !important's -- perhaps you can craft it in cleaner or perhaps @0ip can

I also found that the animations are a bit jerky on mobile, any chance of a way of overriding these?

@jboesch
Copy link
Owner

jboesch commented Mar 17, 2013

Mobile support is something I will work on when I have more time. Animations should use webkit animations instead of JavaScript animations.

@matheo
Copy link

matheo commented Nov 6, 2014

This is still an issue?

@JohnMcLear
Copy link
Author

Well I did a ghetto CSS hack that should really be adopted upstream, so it's up to the project maintainer really :)

@kpx-dev
Copy link

kpx-dev commented Dec 17, 2014

@JohnMcLear please issue a PR, I'll help CR and merge

@JohnMcLear
Copy link
Author

#60

@7shifts
Copy link

7shifts commented Dec 17, 2014

@kienpham2000 Thank you! :)

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

No branches or pull requests

4 participants