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

OSGeo Branding Update Logos #238

Merged
merged 8 commits into from
Jun 30, 2017
Merged

OSGeo Branding Update Logos #238

merged 8 commits into from
Jun 30, 2017

Conversation

jodygarnett
Copy link
Sponsor Contributor

I provided these more than a week ago in SVG form, converted to PNG now and added as a pull request.

This pull request also:

  • updates the graduation status on a few projects (PostGIS, marble, gvSig, GeoServer)
  • changes from incubation to community logo use

@jodygarnett
Copy link
Sponsor Contributor Author

Please note this pull request allows edits from maintainers - @camerons there should be no need to make your own pull request if you have edits you wish to make no this one.

Users with write access to OSGeo/OSGeoLive-doc can add new commits to your logos branch

@kalxas
Copy link
Member

kalxas commented Jun 27, 2017

Thank you @jodygarnett
This looks really nice!

I will open a ticket on trac and link this PR.

@kalxas
Copy link
Member

kalxas commented Jun 27, 2017

@kalxas kalxas self-assigned this Jun 27, 2017
Noticed a few projects (geoserver, gvsig, marble) that had not been marked as graduated yet.
Not sure if this will be good long term as I do not have the style guide to work from
@camerons
Copy link
Contributor

I might be nostalgic, but I prefer the old OSGeo Logo, and the blue of our adaption to OSGeo-Live. But I think it valuable to be consistent with OSGeo branding, so I'm in favour of accepting proposed changes for this release. (If we have time to be creative and change the logo later, we can).
+1 from me to accept the new logos.

@kalxas
Copy link
Member

kalxas commented Jun 28, 2017

+1 from me too.
The horizontal banner is much better now, thanks @jodygarnett

@jodygarnett
Copy link
Sponsor Contributor Author

We can try the blue, as a sub brand it may be fun to have three colours to work with, need someone good it colour theory to choose a complementary shade.

@jj0hns0n
Copy link

+1, but concur we should continue to explore the boundaries of what we can do with this logo/system and still be 'on brand'

@kalxas
Copy link
Member

kalxas commented Jun 28, 2017

I am not sure we need to keep the Blue.
I also know that @darkblue-b is doing some tests with the logo

@jodygarnett
Copy link
Sponsor Contributor Author

That is perfect @darkblue-b is already authority on shades of blue (bada ping...)

@jodygarnett
Copy link
Sponsor Contributor Author

Note the "images/logos/OSGeo_compass_with_text_square.png" logo is against the style guide, I only made it because I wanted to have an unchanged size for our template. If @darkblue-b is experimenting perhaps we could find where that is used and change to the normal logo.

So what do we need to have happen to merge this pull request?

@kalxas
Copy link
Member

kalxas commented Jun 29, 2017

Since there hasn't been any more feedback here, I will bring this to the mailing list for final voting.

@cvvergara
Copy link
Contributor

Hi Jodi,
I don't have write access, so I can not change anything so I will comment:
I took the liberty of making a test merge in my fork, and ran these commands

grep OSGeo_community -A 5 $(git ls-files) | more
grep OSGeo_project -A 5 $(git ls-files) | more

I will focus this comment on the second command about OSGeo_project:
The majority look like this:

ca/overview/deegree_overview.rst:.. image:: /images/logos/OSGeo_project.png
ca/overview/deegree_overview.rst-  :scale: 100
ca/overview/deegree_overview.rst-  :alt: Projecte OSGeo
ca/overview/deegree_overview.rst-  :align: right
ca/overview/deegree_overview.rst-  :target: http://www.osgeo.org
ca/overview/deegree_overview.rst-

Some have

  • :alt: OSGeo Project in Incubation
  • :target: http://www.osgeo.org/incubator/process/principles.html
  • some have both like this one:
ca/overview/geomoose_overview.rst:.. image:: /images/logos/OSGeo_project.png
ca/overview/geomoose_overview.rst-  :scale: 100 %
ca/overview/geomoose_overview.rst-  :alt: OSGeo Project in Incubation
ca/overview/geomoose_overview.rst-  :align: right
ca/overview/geomoose_overview.rst-  :target: http://www.osgeo.org/incubator/process/principles.html
ca/overview/geomoose_overview.rst-

Some I don't understand the alt:

el/overview/ossim_overview.rst:.. image:: /images/logos/OSGeo_project.png
el/overview/ossim_overview.rst-  :scale: 100 %
el/overview/ossim_overview.rst-  :alt: Λογισμικό ενσωματωμένο στο OSGeo
el/overview/ossim_overview.rst-  :align: right
el/overview/ossim_overview.rst-  :target: http://www.osgeo.org
el/overview/ossim_overview.rst-
---
el/overview/postgis_overview.rst:.. image:: /images/logos/OSGeo_project.png
el/overview/postgis_overview.rst-  :scale: 100 %
el/overview/postgis_overview.rst-  :alt: Λογισμικό του OSGeo
el/overview/postgis_overview.rst-  :align: right
el/overview/postgis_overview.rst-  :target: http://www.osgeo.org/incubator/process/principles.html
el/overview/postgis_overview.rst-

but to be in the safe side, it can be written in English, and later a translator can translate

If you tell me what to do, I can fix and make a PR to your fork/logos branch (that will include the latest changes on master) and you can review the changes, and merge when satisfied (that will update this PR).

@kalxas
Copy link
Member

kalxas commented Jun 30, 2017

@cvvergara I will merge now and we can fix issues in a new PR

@kalxas kalxas merged commit df6db67 into OSGeo:master Jun 30, 2017
@cvvergara
Copy link
Contributor

@kalxas
ok

@jodygarnett
Copy link
Sponsor Contributor Author

Excellent my first contribution in years breaks the build, I feel like a worthwhile human being :)

(more seriously thanks everyone, it will be great to have the new branding, and the update project relationships, in the mix).

@cvvergara
Copy link
Contributor

@jodygarnett
does not break the build

@kalxas
Copy link
Member

kalxas commented Jul 2, 2017

Actually the test server is still at alpha, not sure if it was this PR :)
http://adhoc.osgeo.osuosl.org/livedvd/docs/en/index.html

@cvvergara
Copy link
Contributor

@kalxas
Copy link
Member

kalxas commented Jul 3, 2017

Back in the subversion days, install scripts and docs were all in one repo.
The VERSION file exists in both git repositories now, but the actual release process is based on this one:
https://github.com/OSGeo/OSGeoLive/commits/master/VERSION.txt

@cvvergara
Copy link
Contributor

ah, then that file need be "automatically moved" to the docs repo

@kalxas
Copy link
Member

kalxas commented Jul 4, 2017

@cvvergara actually no.
Since we build the docs as a debian package, the docs become "final" many days before we tag OSGeo-Live as final. So the split is intentional and should stay like that.

@cvvergara cvvergara added this to the Release 11.0 milestone Jul 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants