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

OpenLayers information sheet: Add A4 format and German version #90

Merged
merged 7 commits into from
Oct 4, 2017

Conversation

marcjansen
Copy link
Contributor

This PR suggests to add the following two variants of the OpenLayers information sheet

I am not 100% sure if we want the language specific variants be in the main repo, but maybe @astroidex can weigh in here as well. I am happy to update this PR if needed.

@astroidex
Copy link
Member

ol_korrektur

  • some corrections see screenshot

@marcjansen
Copy link
Contributor Author

Thanks, will update

@marcjansen
Copy link
Contributor Author

Ping @astroidex:

I addressed your comments, the current PDF is here https://github.com/marcjansen/osgeo/blob/6f500975b3ecec1ec756329a1bb86fb460b57500/marketing/print/openlayers_a4_de.pdf.

Some notes:

  • I have left 'build-tools' in the text, as I couldn't come up with a German word that sounded well
  • The previous export missed one sentence wrt performance in the third paragraph, mea culpa
  • I have added another paragraph describing the interactions that are possible with OpenLayers (probably somthing that might make sense to vbackport to the en-variant).
  • I rephrased two or three sentences

I hope you like my changes, and thanks for the input you provided. If you have more feedback to the current version, just drop it here. If you are fast enough, I will update once again (In case that is needed)

@jodygarnett
Copy link
Sponsor Contributor

This looks great, can we match the new folder structure - and possibly make a print/de folder :)

@marcjansen
Copy link
Contributor Author

Thanks for the input @jodygarnett:

I have …

  • … rebased this on the latest master
  • … moved arround the resources as suggested:
    • new print/de/-folder
    • added the English A4-variant to the eu/-folder
  • … left the _de-suffix on the PDF
  • … also added the _de.sla to a new dedicated de/-folder in the collateral/-folder

It'd be nice if you could please take another look.

@jodygarnett
Copy link
Sponsor Contributor

jodygarnett commented Oct 4, 2017

Thanks! Happy to merge, thanks for so quickly updating.

I am going to use this as an example for discussing folder structure with the marketing list:

  • print/eu - for a4 and similar
  • print/us - for letter
  • print/de - for german

I do not know if this is a good idea; it is more work but it may result in an easier time directing people to a folder of suitable materials they can just grab and print.

@jodygarnett jodygarnett merged commit bd63e5f into OSGeo:master Oct 4, 2017
@marcjansen marcjansen deleted the openlayers-a4-german branch October 4, 2017 19:12
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.

None yet

3 participants