widelands

Merge lp://staging/~widelands-dev/widelands/bug-1818013-new-logo into lp://staging/widelands

  1. bug-1818013-new-logo
  2. Merge into trunk
Proposed by GunChleoc
Status: Merged
Merged at revision: 9113
Proposed branch: lp://staging/~widelands-dev/widelands/bug-1818013-new-logo
Merge into: lp://staging/widelands
Diff against target: 98 lines (+18/-6)
4 files modified
src/ui_basic/icon.cc (+6/-0)
src/ui_basic/icon.h (+4/-0)
src/ui_fsmenu/main.cc (+6/-6)
src/ui_fsmenu/main.h (+2/-0)
To merge this branch: bzr merge lp://staging/~widelands-dev/widelands/bug-1818013-new-logo
Reviewer Review Type Date Requested Status
Widelands Developers Pending
Review via email: mp+366502@code.staging.launchpad.net

Commit message

New Widelands logo by Nytren. Add it as an icon rather than an overlay for more fine-grained positioning.

To post a comment you must log in.
Revision history for this message
Benedikt Straub (nordfriese) wrote :

Do the splash screen and the editor loading screen also get the new font and style?

Revision history for this message
bunnybot (widelandsofficial) wrote :

Continuous integration builds have changed state:

Travis build 4799. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/524373614.
Appveyor build 4580. State: failed. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_bug_1818013_new_logo-4580.

Revision history for this message
GunChleoc (gunchleoc) wrote :

Impossible for them at the moment without replacing the images as well, because the "Widelands" text is baked into the images, and we have no layered versions.

Revision history for this message
Toni Förster (stonerl) wrote :

Shouldn't we consider replacing the pictures entirely? They are only available in 800x600.

Revision history for this message
GunChleoc (gunchleoc) wrote :

I'd certainly be in favor of that.

Revision history for this message
bunnybot (widelandsofficial) wrote :

Continuous integration builds have changed state:

Travis build 5008. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/533694061.
Appveyor build 4789. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_bug_1818013_new_logo-4789.

Revision history for this message
GunChleoc (gunchleoc) wrote :

Any objections to merging this?

Revision history for this message
kaputtnik (franku) wrote :

From my side: no.

Not tested though.

Revision history for this message
GunChleoc (gunchleoc) wrote :

@bunnybot merge

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to status/vote changes: