Skip to content

New popup styles#12

Open
chinaowl wants to merge 6 commits into350dotorg:gh-pagesfrom
thoughtworks:popup-styles
Open

New popup styles#12
chinaowl wants to merge 6 commits into350dotorg:gh-pagesfrom
thoughtworks:popup-styles

Conversation

@chinaowl
Copy link
Contributor

@chinaowl chinaowl commented Aug 3, 2016

Hi @ejucovy,

These are the CSS changes for the new popup styles that Matthew wanted. I showed Matthew a bunch of screenshots, which he approved in Slack. (Is he on GitHub?)

image

A big thank you to @nliaoTW and @NataliePollard for contributing to this PR! 👏

@CLAassistant
Copy link

CLAassistant commented Aug 3, 2016

CLA assistant check
All committers have signed the CLA.

@@ -0,0 +1,283 @@
@charset "utf-8";
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is a CSS file that Matthew gave us to use.

@chinaowl
Copy link
Contributor Author

chinaowl commented Aug 3, 2016

One important thing to note is that for the styles to work, I had to change the markup in the Google spreadsheet for the existing popups (local groups, power shifts, and fossil free groups). I tested it with the copy that I have: https://docs.google.com/spreadsheets/d/1FW0GW7YQjY_9_wVf1XGKkpqImaHGuGpIgOt13aCp1qY/edit?pref=2&pli=1#gid=2

But you will have to copy over the markup to the production Google sheet after this is merged.

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

Successfully merging this pull request may close these issues.

3 participants