Foto

Can't remove albums

Bug #1366303 reported by Fabio Zaramella
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Foto
Fix Committed
High
Erasmo Marín
Foto foto-1.0 "freya"

Bug Description

There should me a minus icon in the sidebar and if i click an album it should be sensitive allowing me to remove the whole album.

Revision history for this message
Erasmo Marín (erasmo-marin) wrote :

I'm not really sure how to implement actions in the sidebar items, like a contextual menu, because these items are not widgets, but using a "minus" icon at the bottom is a good solution.

Changed in foto:
status: New → Confirmed
importance: Undecided → High
milestone: none → foto-1.0
assignee: nobody → Erasmo Marín (erasmo-marin)
Revision history for this message
Erasmo Marín (erasmo-marin) wrote :

I also added a dialog asking for removing or not the album. Not sure if buttons should be or not in the header bar, because the other dialogs has them at bottom.

Changed in foto:
status: Confirmed → Fix Committed
Revision history for this message
Erasmo Marín (erasmo-marin) wrote :

*Also, I forgot to mention that right now the remove button is insensitive when no albums are selected, but maybe would be better to have it hidden.

Revision history for this message
Fabio Zaramella (fabiozaramella) wrote :

Good :) however i think it's better to have the remove button insensitive because if there isn't a selected album the user can immediately understand the purpose of the button (tooltip_text :p). Having it hidden wouldn't be so immediate.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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