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

Contributors mark and osm.org/contributors #180

Closed
wants to merge 5 commits into from

Conversation

samanpwbb
Copy link
Member

This is a larger change, an RFC on [talk] is pending.

This pull request adds a new page under openstreetmap.org/contributors like this:

Screen Shot 2013-01-10 at 2 56 21 PM

It is linked from a contributor's mark on the front page map:

Screen Shot 2013-01-10 at 2 59 42 PM

The idea behind this contributor's mark and this new page is to provide a visually compelling way to link back to OpenStreetMap from maps and communicate clearly what OpenStreetMap is about: community built, open geographic data, anybody can contribute.

Ideally this contributor mark finds broad adoption and is used by OSM data users like this.

The contributor's page can be configured to:

  • Reflect custom tile sets
  • Reflect the specific OSM data user ("Foursquare uses maps powered by data from OSM...")
  • Set an unobtrusive link to OSM service provider ("Maps from MapBox")

To help other users customize openstreetmap.org/contributors & use the link icon on their maps, there is a link in the bottom left corner of openstreetmap.org/contribors that brings up a little 'attribution builder' wizard that looks like this:

Step 1:
Screen Shot 2013-01-10 at 3 03 39 PM

Step 2:
Screen Shot 2013-01-10 at 3 01 58 PM

That code snippet is for a contributors mark meant to be dropped in the bottom right corner of maps, which links back to the /contributors page.

Aside from a pending review phase, this pull request still needs:

  • Translatable strings on osm.org/contributors
  • Model releases and permissions from photographers
  • Reconciliation with osm.org/copyright

@lxbarth will follow up momentarily with a corresponding RFC on [talk] in a bit elaborating on this idea.

Original design is by @yhahn. You can explore example usages of the contributors mark here (click on the mark on each of the examples and review how osm.org/contributors changes):

http://yhahn.github.com/byosm/examples.html

@lxbarth
Copy link
Contributor

lxbarth commented Jan 11, 2013

@tmcw
Copy link
Contributor

tmcw commented Jan 11, 2013

Small bug in sprite positioning:

@samanpwbb
Copy link
Member Author

@tmcw thanks for catching this: there was a mismatch on the background-size for the retina sprite. Fixed!

@lxbarth
Copy link
Contributor

lxbarth commented Jan 15, 2013

  • We'll need to include a specific reference to the ODbL license of the data and link to /copyright

@lxbarth
Copy link
Contributor

lxbarth commented Jan 16, 2013

@lxbarth
Copy link
Contributor

lxbarth commented Jan 16, 2013

  • We'll want to take a stronger cue from the overall osm.org design, @samanpwbb latest commit (681f7f9) is a first stab at that, but let's think about this a little more comprehensively together with all other adjustments that need to happen.

@lxbarth
Copy link
Contributor

lxbarth commented Jan 16, 2013

  • Adjust message to encourage potential mappers.

@lxbarth
Copy link
Contributor

lxbarth commented Jan 16, 2013

@lxbarth
Copy link
Contributor

lxbarth commented Jan 16, 2013

  • Needs browser testing

@samanpwbb
Copy link
Member Author

Updating this pull request to note that, based on Alex's new RFC, this pull request will require some heavy re-tooling: For details, see http://lists.openstreetmap.org/pipermail/talk/2013-April/066802.html

@jfirebaugh
Copy link
Member

Proposal evolution continued in #246.

@samanpwbb samanpwbb closed this May 2, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improvements and new feature requests work-in-progress Pull request is not ready to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants