-
Notifications
You must be signed in to change notification settings - Fork 805
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
AMP support #258
Comments
Why i would not promote AMP: |
I don't get that hatred for AMP. Of course one can make a performant website without it, but in the real life almost all companies bloat their websites. AMP is a radical step to forbid web bloat and stick to lightweight solutions. It really improves loading web pages, you can try it on desktop using my AMP Browser. |
@niutech I will not add an AMP flavor. |
Finally I have made an AMP version of your framework: https://github.com/niutech/amp-spectre/ |
@niutech , Did you generate this version using an original fork? I mean the source files Sass, gulpfile.js, etc? |
@AdsonCicilioti I generated AMP Spectre using this file: https://github.com/picturepan2/spectre/blob/master/dist/spectre.min.css |
@niutech, the show/hide layout utility class din't work on elements with display:flex, like as navbar-sections. |
Spectre could be compatible with AMP by slightly modifying the code: removing
!important
, replacing<img>
with<amp-img>
and removing inlinestyle=""
from HTML. I made the AMP components page demo: https://repl.it/KPR2/3 or fullscreen: http://s.cssdeck.com/labs/full/x5pjudqw/noframeWould you like to add an official AMP flavor of the Spectre CSS framework?
The text was updated successfully, but these errors were encountered: