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

Rough cut #1

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Rough cut #1

wants to merge 12 commits into from

Conversation

arunasank
Copy link
Collaborator

@arunasank arunasank commented Feb 12, 2018

👋 @iamasid

To check out the progress:

  1. git clone https://github.com/Veditum/postcards.git: makes a copy of this repo on your machine
  2. cd postcards/
  3. git checkout first-cut: Checks out the first-cut branch (main branch is called master)
  4. open postcards.html

Next actions

  • @iamasid to confirm the location of a postcard that is wrongly placed presently
  • @iamasid to review the LICENSE
  • @iamasid to review the work so far, and offer any critique he has, for the next few iterations.

I want to redesign the map to make it more stark - something like a white background, so all the data on it has a lot of emphasis, and so that the postcards stand out more. Also want to take some advice regarding the front end look from some good front-end engineer friends.

Will continue pushing here.

@iamasid
Copy link

iamasid commented Feb 12, 2018

👋 @arunasank
Just updated the lat-long for the one remaining image, and re-ordered the files in the html file. Now we can focus on:

  1. Flight: so that it appears like a forward-backward movement when user scrolls.
  2. Labels for locations of the postcard stories.
  3. Short description for each postcard maybe.
  4. Experimenting with map style .

@iamasid
Copy link

iamasid commented Feb 20, 2018

Super! Now to adjust zoom + pitch for all and bearing for the last postcard.

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