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

added program page #44

Merged
merged 6 commits into from
Oct 9, 2018
Merged

added program page #44

merged 6 commits into from
Oct 9, 2018

Conversation

tridip1931
Copy link
Member

@tridip1931 tridip1931 commented Sep 20, 2018

ref #43

  • updated nav links
  • added new program page

cc: @geohacker

@geohacker
Copy link
Member

@tridip1931 - thank you! This is looking good. Could you add the entire program? Some things to fix:

  • We want permalinks to each day. Can perhaps just use div links?
  • We want some intro text at the top to help participants orient the schedule like https://2018.stateofthemap.org/program/. I'll share the copy but can you just put dummy text for now?

@tridip1931
Copy link
Member Author

tridip1931 commented Sep 24, 2018

Awesome, is the program finalised, I am seeing the dummy texts in the sheet.

We want permalinks to each day. Can perhaps just use div links?

Yeah, div links would be better.

@geohacker
Copy link
Member

Awesome, is the program finalised, I am seeing the dummy texts in the sheet.

@tridip1931 you'll see the first draft in a sheet called "agenda" in the spreadsheet. We'll use this for now, and update as needed.

@tridip1931
Copy link
Member Author

👍

@geohacker
Copy link
Member

Ref #45
@tridip1931 I took a stab:

image

I think the margins are a bit off. This seems like a better spot than replacing the twitter button on the top. What do you think?

@tridip1931
Copy link
Member Author

screen shot 2018-10-09 at 10 07 08

added some margins to organizer div and cards below, is this better?

@geohacker geohacker merged commit 4c0e304 into master Oct 9, 2018
@geohacker geohacker deleted the add-program-page branch October 9, 2018 13:44
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.

2 participants