Create a metadata context pane [$100]

Bug #1332978 reported by Danielle Foré
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Photos
Fix Released
Low
meese

Bug Description

We currently have like 30 context menu items strewn about that have to do with one thing: metadata.

We should either create an "Edit Image Info" dialog to parallel the "Edit Song Info" dialog from Music, or create an "Info" context pane that you can hide/reveal.

This should include all of the fields and data from:

* Basic Info
* Extended Info
* Add/Modify Tags
* Flag
* Set Rating
* Edit Title
* Edit Comment
* Adjust Date and Time

Related branches

Revision history for this message
Harvey Cabaguio (harveycabaguio) wrote :

When I've seen people manage photos using like iPhoto or whatever, it was pretty important for them to have these details all the time while going through multiple pictures. A popover wouldn't be a good idea, and we hate multiple windows so I think a context pane would make the most sense.

It would be pretty familiar to a lot of users I think, if they come from g+ or iPhoto.

Revision history for this message
Harvey Cabaguio (harveycabaguio) wrote :

I designed how it could possibly look. Check out the attachment.

I guess the button to reveal this pane would show up in the headerbar.

Revision history for this message
Danielle Foré (danrabbit) wrote :

Since we don't yet have a headerbar, for now we could place a button in the bottom right corner of the bottom toolbar (same position like it is in Music)

Changed in pantheon-photos:
status: New → Confirmed
Revision history for this message
Danielle Foré (danrabbit) wrote :
summary: - Create a metadata context pane
+ Create a metadata context pane [$100]
Revision history for this message
meese (meese) wrote :

Heya, I did a good bit on this today, screenshot is attached. Obviously it is rough on the formatting and prettiness still but besides that any input?

-I used expander widgets for the different sections, yes or just have the section titles as normal labels and no expandiness?

-waiting on https://code.launchpad.net/~victored/granite/rating to be done for the actual star widget not the +.- buttons..

-Should all of the rating /edit metadata stuff be removed from the context menu? The context menus are pretty cluttered but it's nice to have them there too when you don't want a sidebar taking up space.

-Should the image properties frame on the bottom Library sidebar be completely removed?

-I used the noise icons for the reveal sidebar button, but should an Icon that makes it more obvious the sidebar is about metadata be used?

Changed in pantheon-photos:
status: Confirmed → In Progress
assignee: nobody → meese (madelynn-r-may)
Revision history for this message
Danielle Foré (danrabbit) wrote :

Hey Madelynn, let's move discussion to a MR on the relevant branch :) I've started one here https://code.launchpad.net/~madelynn-r-may/pantheon-photos/fix-1332978/+merge/227644

Changed in pantheon-photos:
milestone: none → freya-beta2
meese (meese)
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.

Other bug subscribers

Remote bug watches

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