Project

General

Profile

Feature #15578

Feature #14568: Additional Software Packages

Bug #15567: Fix bugs and UX issues in the Additional Software beta

Explain better what removing a package means

Added by sajolida about 1 year ago. Updated 11 months ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Start date:
05/05/2018
Due date:
% Done:

100%

Feature Branch:
feature/14594-asp-gui
Type of work:
Code
Blueprint:
Starter:
Affected tool:
Additional Software Packages

Description

During the user testing of the Additional Software beta:

  • 2 participants hesitated before clicking the [X] button.
  • After clicking "Remove" in the confirmation dialog, 1 participent still doubted whether removing meant uninstalling or deleting. Actually, it means something else :)

We could:

  • Add a tooltip on the [X] button.
  • Explain a bit more in the confirmation dialog.

Associated revisions

Revision f31a870a (diff)
Added by alant 12 months ago

ASP: add tooltip to remove package button in configuration windows

Rationale: we're not explaining anywhere what removing the package from
the list really means. "Removing" means "stop installing automatically"
(but not "deleting from persistence" or "uninstalling immediately", for
example). Let's keep the phrasing that we already have in the
confirmation dialog and be a bit more explicit in the tooltip, who only
people who will hesitate will see.

Will-fix: #15578

Revision 9942abd3 (diff)
Added by alant 12 months ago

ASP: add explanation to package removing confirmation dialogs

Will-fix: #15578

History

#1 Updated by sajolida about 1 year ago

  • Assignee set to sajolida
  • Parent task changed from #14592 to #15567
  • Affected tool set to Additional Software Packages

We'll try to fix that but it's not a blocker to deliver SponsorW_2017.

Next step: I should propose a wording for a tooltip.

#2 Updated by sajolida about 1 year ago

  • Target version set to Tails_3.9

#3 Updated by sajolida about 1 year ago

  • Assignee changed from sajolida to alant
  • Type of work changed from User interface design to Code

Proposed wording for the tooltip:

Stop installing $package automatically

If it's too complicated to include the name of the package:

Stop installing automatically

Rationale: we're not explaining anywhere what removing the package from the list really means. "Removing" means "stop installing automatically" (but not "deleting from persistence" or "uninstalling immediately", for example). This was only a slight problem for P3 who hesitated a second to click on the cross button. So let's keep the phrasing that we already have in the confirmation dialog (this would be too late for P3 anyway) and be a bit more explict in the tooltip, who only people who will hesitate will see.

#4 Updated by alant 12 months ago

  • Status changed from Confirmed to In Progress
  • % Done changed from 0 to 30
  • QA Check set to Ready for QA
  • Feature Branch set to feature/14594-asp-gui

Implemented in feature/14594-asp-gui, to be tested in real ISO.

#5 Updated by alant 12 months ago

  • Assignee changed from alant to sajolida
  • QA Check changed from Ready for QA to Info Needed

Shouldn't we also add in the confurmation dialog "Remove $package from your additional software? This will stop installing the package automatically." or something similar?

#6 Updated by sajolida 12 months ago

  • Assignee changed from sajolida to alant
  • QA Check changed from Info Needed to Dev Needed

Yeap, that would be great.

#7 Updated by alant 12 months ago

  • % Done changed from 30 to 60
  • QA Check changed from Dev Needed to Ready for QA

#8 Updated by u 12 months ago

  • Assignee changed from alant to sajolida

I'm assigning this to sajolida, as you probably meant to do when marking this as ready for QA? If not, please reassing back to you. Note that the deadline to get this merged is in one week!

#9 Updated by sajolida 11 months ago

  • Assignee changed from sajolida to alant

I'm happy with what I see in the Git log.

u: I think that sometimes Alan assigns stuff as "Ready for QA" to himself until he tests his changes.

Alan: if you tested this already please mark it as "Fix committed".

#10 Updated by intrigeri 11 months ago

Alan assigns stuff as "Ready for QA" to himself until he tests his changes.

Indeed. At least anonym and I do the same pretty often. And incidentally (or on purpose) this enables ISO reproducibility testing on Jenkins :)

#11 Updated by alant 11 months ago

  • Status changed from In Progress to Resolved
  • Assignee deleted (alant)
  • % Done changed from 60 to 100
  • QA Check changed from Ready for QA to Pass

Also available in: Atom PDF