Opened 6 years ago

Closed 5 years ago

#1999 closed improve feature (fixed)

"Delete" button in BW inbox deletes message without any confirmation dialogue

Reported by: sitatara Owned by: crumbking
Priority: major Milestone: 2.3
Component: BW Mail Keywords: messages, delete
Cc:

Description

The "delete" button in the BeWelcome inbox is very conveniently placed to delete messages by accident. What makes matters worse is that it actually deletes the message directly without any confirmation dialogue.

Please introduce a confirmation dialogue to avoid the situation that members delete their messages accidentally!

Change History (15)

comment:1 Changed 5 years ago by shevek

  • Milestone changed from unassigned to 2.1

comment:2 Changed 5 years ago by shevek

  • Milestone changed from 2.1 to 2.2

comment:3 Changed 5 years ago by crumbking

  • Status changed from new to local_testing

https://gitorious.org/bewelcome/rox/commit/a566ac213442f845aad5da778144f1cac7325f74

This is a first idea. In the long run a trash box would be the better idea. Which will clean mails every 30 days ...

comment:4 Changed 5 years ago by shevek

  • Status changed from local_testing to to_alpha

comment:5 Changed 5 years ago by shevek

  • Status changed from to_alpha to testing

comment:6 Changed 5 years ago by shevek

  • Resolution set to fixed
  • Status changed from testing to closed

Works fine in the Inbox but if you click delete when displaying the message it is gone directly as well.

I spin off a ticket for a trash box and close this one for 2.2.

comment:7 Changed 5 years ago by sitatara

  • Resolution fixed deleted
  • Status changed from closed to reopened

comment:8 Changed 5 years ago by sitatara

Reopening this since it's still possible to delete messages without confirmation. Just happened to me when I opened a message and clicked on delete then.

comment:9 Changed 5 years ago by crumbking

Oh some more delete buttons. Should be fixed with:

https://gitorious.org/bewelcome/rox/commit/7225e184d56b2db77cd79b6626ee6b56945623c6

We really need some more testers ;-)

comment:10 Changed 5 years ago by crumbking

  • Owner set to crumbking
  • Status changed from reopened to assigned

comment:11 Changed 5 years ago by crumbking

  • Status changed from assigned to local_testing

comment:12 Changed 5 years ago by crumbking

  • Milestone changed from 2.2 to 2.3

comment:13 Changed 5 years ago by shevek

  • Status changed from local_testing to to_alpha

comment:14 Changed 5 years ago by shevek

  • Status changed from to_alpha to testing

comment:15 Changed 5 years ago by sitatara

  • Resolution set to fixed
  • Status changed from testing to closed

Works now in both cases (overview and single message) - thanks!

Note: See TracTickets for help on using tickets.