Adium

Ticket #13097 (closed enhancement: fixed)

Opened 5 months ago

Last modified 5 months ago

Chat transcripts deleted

Reported by: assetburned Owned by:
Milestone: Adium 1.4 Component: Adium UI
Version: 1.4b9 Severity: normal
Keywords: Cc:
Patch Status: Accepted

Description

Simply pressing the backspace key shouldn't delete a log. there should be a message for confirmation!

Attachments

wildwobby-13097-101109.diff (0.9 KB) - added by wildwobby 5 months ago.
Patch will ask for confirmation when deleting with the backspace button using the already instate confirmation methods.
addition-13097-102809.diff (1.6 KB) - added by wbowling 5 months ago.
Patch to stop confirmation or try to deletion of 0 selected logs
addition-13097-102809-v2.diff (1.6 KB) - added by wbowling 5 months ago.
Uses == 1 istead of != 0

Change History

Changed 5 months ago by wildwobby

Patch will ask for confirmation when deleting with the backspace button using the already instate confirmation methods.

Changed 5 months ago by Robby

  • patch_status set to Needs Dev Review
  • milestone set to Adium 1.4

Go, Robby! :)

Changed 5 months ago by Robby

23:29:40 kbotc: #13097 looks pretty good to me..

Changed 5 months ago by Zachary West <zacw@…>

  • status changed from new to closed
  • resolution set to fixed

(In  3d815231bb66) Patch from wildwobby which confirms before removing logs when hitting the delete key. Fixes #13097.

Changed 5 months ago by Robby

  • patch_status changed from Needs Dev Review to Accepted

Changed 5 months ago by Zachary West <zacw@…>

(In  ff31212ab05e) Patch from wildwobby which confirms before removing logs when hitting the delete key. Fixes #13097.

Changed 5 months ago by wbowling

Patch to stop confirmation or try to deletion of 0 selected logs

Changed 5 months ago by wbowling

Currently if you press the delete key with no logs selected you get a message asking "Are you sure you want to send 0 logs to the Trash?", not the right behavior I think :)

Also, pressing the delete toolbar button with no logs selected will highlight the last log in the list.

Simple patch to fix

Changed 5 months ago by wbowling

Uses == 1 istead of != 0

Changed 5 months ago by boredzo

  • patch_status changed from Accepted to OK; Needs Check-In

Looks good to me. Checking it in.

Changed 5 months ago by boredzo

Pushed as bc1492a4a7c7 in adium-1.4 and b9fd746d2a9a in adium.

Changed 5 months ago by boredzo

  • patch_status changed from OK; Needs Check-In to Accepted
Note: See TracTickets for help on using tickets.