Update Branding

Bug #1272472 reported by Daniel Fore on 2014-01-24
24
This bug affects 13 people
Affects Status Importance Assigned to Milestone
Photos
Fix Released
Medium
Unassigned

Bug Description

Update branding to reflect the name change from Shotwell to Photos

Replace the street sign image in the about dialog with the app icon

Change binary and package name from shotwell to pantheon-photos

Changed in pantheon-photos:
status: New → In Progress
Tristan Petersen (tristang4) wrote :

I made mostly aesthetic changes inside the app so everything (except the app name) looks like "Photos". I didn't change the binary name, because it seems like anything associated with building the app still needs modification - ./configure, Makefile, etc.

I changed a few links to point to elementary but we'll need to work out how to handle the changes to the wiki and documentation before implementing changes to those links.

Cody Garver (codygarver) on 2014-01-28
Changed in pantheon-photos:
milestone: none → 0.16
importance: Undecided → Medium
description: updated
description: updated
Tristan Petersen (tristang4) wrote :

Folks, I deleted my old branch of this that was a mess.

I pushed a newly created branch, and it's linked above.

I have not proposed for merging yet because I am taking it step by step. The new branch contains changes to the Makefile only as of this writing. Feel free to add on from here or I will as I get around to it.

Tristan Petersen (tristang4) wrote :

Renamed & updated more files

Cody Garver (codygarver) wrote :

Tristan, thanks for continuing to work on this.

I suggest you join the launchpad group https://launchpad.net/~elementary-dev-community and then visit your branch https://code.launchpad.net/~tristang4/pantheon-photos/fix-1272472 and change the owner to elementary-dev-community. You will still get the same credit upon merge.

Then propose the branch for merge but under "Extra options", untick "Needs review". This will make it easier to track the progress of the branch, at least for me ;)

Cody Garver (codygarver) wrote :

The reason I want you to change the branch owner is so other developers can help with the work since there's a lot to change.

Rico Tzschichholz (ricotz) wrote :

I pushed a new rebranding branch linked to this bug.

Tristan Petersen (tristang4) wrote :

Cody, my time how the time flies. It felt like just yesterday I posted the branch online.

I changed the branch to elementary devs and proposed for merging as you mentioned.

Thanks for the tips!

Eduard Gotwig (gotwig) wrote :

Any idea why this merge isnt reviewed yet?

Justin Zobel (justin-zobel) wrote :

Just a note, shotwell isn't actually included in latest 0.3 builds. I installed it manually and even still won't appear in slingshot, can launch from terminal though.

Daniel Fore (danrabbit) wrote :

I'm going to mark this as fixed.

If we want to make Shotwell and Photos able to be installed at the same time that's another issue.

There's also another branch/report for using Granite About to fix the lack of icon in the About dialog

Changed in pantheon-photos:
status: In Progress → Fix Committed
Daniel Fore (danrabbit) on 2014-12-29
Changed in pantheon-photos:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers