Update Branding

Bug #1272472 reported by Danielle Foré
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
Revision history for this message
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)
Changed in pantheon-photos:
milestone: none → 0.16
importance: Undecided → Medium
description: updated
description: updated
Revision history for this message
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.

Revision history for this message
Tristan Petersen (tristang4) wrote :

Renamed & updated more files

Revision history for this message
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 ;)

Revision history for this message
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.

Revision history for this message
Rico Tzschichholz (ricotz) wrote :

I pushed a new rebranding branch linked to this bug.

Revision history for this message
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!

Revision history for this message
Eduard Gotwig (gotwig) wrote :

Any idea why this merge isnt reviewed yet?

Revision history for this message
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.

Revision history for this message
Danielle Foré (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
Changed in pantheon-photos:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.