-
Notifications
You must be signed in to change notification settings - Fork 48
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
Popups need updating to latest #16
Comments
I think they are already? |
They are mostly. But I had been working on some new functionality and efficiency that I need to roll in. |
Oh, I also need to set up the main project for the popup system. |
Lovely 😄 ...let's roll this in to Concrete ✌🏼 |
Cam
pushed a commit
that referenced
this issue
Aug 2, 2018
* Created a snippet to use responsive images * Renamed filename, added responsive bg-image * Add comments and clean up * Remove comments * Add more comments * Code review, removed attributes and pieces of ode * Removed the bgset styling into a separate module * Used 1 container instead of 2 containers * Revert one image to regular implentation and fix comments * Fix comment and change the format padding-top * Clean up conditions * Add dashes to remove whitespaces, removed unnecessary classes and replace image.master * Add dashes to avoid whitespaces * Renamed width & height to max_height, max_width + fix typo
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I'll update these myself soon 👍
The text was updated successfully, but these errors were encountered: