Bug #7450

Tails OpenPGP Applet is too easy to exit

Added by sajolida 2014-06-22 08:16:15 . Updated 2015-08-06 03:18:07 .

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Target version:
Start date:
2014-06-22
Due date:
% Done:

100%

Feature Branch:
feature/7450-openpgp-applet-exit
Type of work:
Code
Blueprint:

Starter:
1
Affected tool:
OpenPGP Applet
Deliverable for:

Description

Doing a double right-click on the applet closes it. This is a bit too easy to exit and quite harder to restore.


Subtasks


Related issues

Related to Tails - Feature #7433: Turn Tails OpenPGP Applet into a standalone application Duplicate 2014-06-22
Related to Tails - Bug #8319: OpenPGP Applet: Add a confirmation dialog on exit Resolved 2014-11-26
Has duplicate Tails - Feature #8293: Add gpgApplet to the Applications menu Duplicate 2014-11-23

History

#1 Updated by intrigeri 2014-07-19 16:14:32

  • related to Feature #7433: Turn Tails OpenPGP Applet into a standalone application added

#2 Updated by intrigeri 2014-07-19 16:17:24

  • Status changed from New to Confirmed
  • Type of work changed from User interface design to Code
  • Starter changed from No to Yes

Agreed. Turning this into a “Code” ticket, as just removing the “Exit” menu entry should be enough.

The code lives in config/chroot_local-includes/usr/local/bin/gpgApplet in the main Tails Git repository. It’s written in Perl, but the fix is likely to be easy, even without prior knowledge of Perl.

#3 Updated by nodens 2014-08-02 00:38:32

How about just moving the Exit down (invert Exit & About) ? One might want to close the applet.

#4 Updated by nodens 2014-08-02 00:41:07

  • Assignee set to intrigeri
  • QA Check set to Info Needed

Note : I’ll do it in the openpgp-applet repository, whether we choose to remove the exit menu or just move it.

#5 Updated by intrigeri 2014-08-02 10:13:45

> One might want to close the applet.

In the context of Tails, I thought we should just remove the Exit menu entry (it’s always possible to kill the process anyway). But, now that we have serious plans to upload this to Debian, I agree with you.

> How about just moving the Exit down (invert Exit & About) ?

That would be a good short-term improvement, I think, but probably not a real solution. I suggest adding a confirmation dialog.

#6 Updated by intrigeri 2014-08-02 10:14:38

  • Assignee deleted (intrigeri)
  • QA Check deleted (Info Needed)

#7 Updated by sajolida 2014-08-02 10:27:55

> How about just moving the Exit down (invert Exit & About) ? One might want to close the applet.

Yeap, I think that would work!

#8 Updated by intrigeri 2014-11-24 13:20:40

  • has duplicate Feature #8293: Add gpgApplet to the Applications menu added

#9 Updated by nodens 2014-11-24 14:31:46

  • Assignee set to nodens

#10 Updated by nodens 2014-11-24 15:18:13

  • Status changed from Confirmed to In Progress
  • Assignee deleted (nodens)
  • % Done changed from 0 to 50
  • QA Check set to Ready for QA

The short-term workaround is available in the bugfix/7450-too_easy_to_exit branch of nodens/openpgp-applet git repository, and should apply cleanly on current Tails’ version.

https://git-tails.immerda.ch/nodens/openpgp-applet/commit/?h=bugfix/7450-too_easy_to_exit&id=a3aa1b67fce5ce25b726beeb4c2b7b7e87736c59

The longer-term solution (confirmation dialog) is quite easy to implement, but I’m unsure how to proceed, as current Tails’ version and the one in the nodens/openpgp-applet repository are starting to diverge (especially everywhere translated text is available).

How about using this workaround until the debian package (Feature #6507) is done and used in Tails ?
If that’s OK I’ll create a new bug for that (e.g Tails OpenPGP Applet : Add a confirmation dialog on exit).

Of course I could provide a patch for both versions but I’d rather spend time on other issues.

Cheers,

#11 Updated by intrigeri 2014-11-24 15:43:15

  • Target version set to Tails_1.3

nodens wrote:
> The short-term workaround is available in the bugfix/7450-too_easy_to_exit branch of nodens/openpgp-applet git repository, and should apply cleanly on current Tails’ version.

Yay :)

> The longer-term solution (confirmation dialog) is quite easy to implement, but I’m unsure how to proceed, as current Tails’ version and the one in the nodens/openpgp-applet repository are starting to diverge (especially everywhere translated text is available).
>
> How about using this workaround until the debian package (Feature #6507) is done and used in Tails ?

Agreed.

> If that’s OK I’ll create a new bug for that (e.g Tails OpenPGP Applet : Add a confirmation dialog on exit).

Yes, please.

#12 Updated by nodens 2014-11-26 22:10:06

Done : Bug #8319

#13 Updated by intrigeri 2014-11-27 09:23:29

  • related to Bug #8319: OpenPGP Applet: Add a confirmation dialog on exit added

#14 Updated by nodens 2014-11-29 12:26:46

Note : the branch has been merged in the openpgp-applet repository to avoid future conflicts, but I won’t delete it until the fix is commited in main repository.

#15 Updated by nodens 2014-12-23 16:58:53

Applied in changeset commit:e511fc0cd00586a25dea8cfa58b506ed460504e3.

#16 Updated by alant 2014-12-23 18:10:20

  • Feature Branch set to feature/7450-openpgp-applet-exit

Local test shows the patch works. I imported the commit to your repo and merged the feature branch in experimental so that we can actually test it in place.

#17 Updated by BitingBird 2015-01-04 17:39:31

  • Affected tool set to OpenPGP Applet

#18 Updated by alant 2015-01-26 19:37:43

  • Assignee set to alant

#19 Updated by Tails 2015-02-04 14:44:23

  • Status changed from In Progress to Fix committed
  • % Done changed from 50 to 100

Applied in changeset commit:59e32414e1816d98b2f5e73b3b072e19cc900d06.

#20 Updated by alant 2015-02-04 14:46:00

  • Assignee deleted (alant)
  • QA Check changed from Ready for QA to Pass

Merged, thanks!

#21 Updated by BitingBird 2015-02-24 22:50:32

  • Status changed from Fix committed to Resolved

#22 Updated by tchou 2015-08-06 03:17:55

  • Subject changed from Tails OpenPGP Applet is too easy to exit to Tails OpenPGP Applet is too easy to exit.

#23 Updated by tchou 2015-08-06 03:18:07

  • Subject changed from Tails OpenPGP Applet is too easy to exit. to Tails OpenPGP Applet is too easy to exit