Project

General

Profile

Feature #8859

Feature #9005: Improve Tails Installer UX: phase 1

Get rid of the splash screen of Tails Installer

Added by sajolida over 2 years ago. Updated 1 day ago.

Status:
In Progress
Priority:
Elevated
Assignee:
Category:
Installation
Target version:
Start date:
02/04/2015
Due date:
% Done:

70%

QA Check:
Dev Needed
Feature Branch:
Type of work:
Code

Description

This is a broader objective of simplification of the workflow in Tails Installer:

  • In the context of Tails, on the side "Use existing Live system ISO" it should have a button to clone exisiting Tails.
  • Outside of the context of Tails it doesn't need that cloning button.

This implies to change some parts of the code in depth: the "clone" feature is currently implemented as a command-line option, and quite a few things are done differently depending on whether it's set or not, e.g. in the LiveUSBDialog class constructor => they will now need to be done differently dynamically, after the UI has been set up and the
target device has been chosen.

Team: u, kurono, sajolida, tchou


Related issues

Blocks Tails - Feature #13424: Core work 2017Q3: User experience Confirmed 07/05/2017
Blocks Tails - Bug #14650: Package new upstream version of tails-installer Confirmed 09/13/2017

Associated revisions

Revision a4121b5b (diff)
Added by anonym 5 days ago

Fix up on tails-installer-launcher still existing.

Refs: #8859

Revision 8b3cd52d (diff)
Added by anonym 5 days ago

Test suite: adapt for new Tails Installer.

Refs: #8859, #8860

Revision ead75282 (diff)
Added by anonym 5 days ago

Test suite: drop obsolete scenarios.

Now that the splash screen is gone, these scenarios happen in the
exact same context as their "Upgrade by cloning" version.

Refs: #8859

Revision df1e6b1a (diff)
Added by anonym 5 days ago

Test suite: drop obsolete scenarios.

Now that the splash screen is gone, this scenarios happen in the exact
same context as its "Install by cloning" version in
usb_install.feature.

Refs: #8859

History

#1 Updated by intrigeri over 2 years ago

The "splash screen" is liveusb-creator-launcher in our liveusb-creator Git repository. The bulk of the code lives in liveusb/launcher.py in the same repo.

#2 Updated by intrigeri over 2 years ago

  • Assignee changed from u to sajolida
  • QA Check set to Info Needed

I don't get why this is a subtask of #8549, and hence transitively blocking #8538. Given:

  • this change (adding the button) wasn't made yet in Tails Installer as we ship it in Tails
  • there's no clear roadmap wrt. when this change will be done on the Tails side of things (no idea who'll do it and when)

=> I don't understand why we should adjust to this change in the next 5 months for inclusion in Debian.

In other words: I don't understand why adding this button on the Tails side, only to then remove it on the Debian side, is a blocker for #8549. Especially since the package will only be in testing and jessie-backports, so there's plenty of time to update it again later to hide that button.. once it actually exists.

(Side note: we discovered #8980 recently, that will already add lots of unexpected work before #8549 can happen, so let's please limit the additions we make to the bare minimum.)

#3 Updated by intrigeri over 2 years ago

  • Assignee changed from sajolida to u
  • QA Check deleted (Info Needed)

Seen the ML thread, I'll reply there if I'm still surprised or in disagreement.

#4 Updated by intrigeri over 2 years ago

  • Blocked by Feature #8860: Tails Installer should autodetect if Tails is installed on the destination device added

#5 Updated by sajolida over 2 years ago

  • Description updated (diff)
  • Assignee deleted (u)
  • Parent task deleted (#8549)

Right, backing up now.

#6 Updated by intrigeri over 2 years ago

  • Blocked by deleted (Feature #8860: Tails Installer should autodetect if Tails is installed on the destination device)

#7 Updated by intrigeri over 2 years ago

  • Parent task set to #9005

#8 Updated by intrigeri over 2 years ago

  • Blocks Feature #8860: Tails Installer should autodetect if Tails is installed on the destination device added

#9 Updated by intrigeri over 2 years ago

  • Blocks deleted (Feature #8860: Tails Installer should autodetect if Tails is installed on the destination device)

#10 Updated by intrigeri over 2 years ago

  • Blocked by Feature #8860: Tails Installer should autodetect if Tails is installed on the destination device added

#11 Updated by intrigeri over 2 years ago

  • Blocked by deleted (Feature #8860: Tails Installer should autodetect if Tails is installed on the destination device)

#12 Updated by intrigeri over 2 years ago

  • Blocked by Feature #9006: Simplify the Tails Installer splash screen added

#13 Updated by intrigeri over 2 years ago

  • Description updated (diff)

#14 Updated by sajolida about 2 years ago

  • Description updated (diff)
  • Target version set to 2016

#15 Updated by kurono almost 2 years ago

  • Assignee set to kurono

#16 Updated by sajolida over 1 year ago

  • Blueprint set to https://tails.boum.org/blueprint/bootstrapping/installer#simplify

#17 Updated by kurono over 1 year ago

  • Target version changed from 2016 to Tails_2.5
  • QA Check set to Ready for QA
  • Feature Branch set to kurono/feature/9005-Improve-tails-installer

#18 Updated by intrigeri about 1 year ago

  • Target version changed from Tails_2.5 to Tails_2.6

#19 Updated by anonym 12 months ago

  • Target version changed from Tails_2.6 to Tails_2.7

#20 Updated by BitingBird 11 months ago

  • Status changed from Confirmed to In Progress
  • % Done changed from 0 to 50

kurono: if you want review from someone, you should unassign yourself :)

#21 Updated by kurono 11 months ago

  • Assignee deleted (kurono)

#22 Updated by sajolida 11 months ago

  • Assignee set to sajolida

#23 Updated by sajolida 10 months ago

  • Target version deleted (Tails_2.7)

#24 Updated by sajolida 9 months ago

  • Assignee changed from sajolida to kurono
  • QA Check changed from Ready for QA to Dev Needed

Review sent.

#25 Updated by kurono 8 months ago

  • Assignee changed from kurono to sajolida
  • QA Check changed from Dev Needed to Ready for QA

#26 Updated by sajolida 6 months ago

  • Assignee changed from sajolida to kurono
  • QA Check changed from Ready for QA to Dev Needed

#27 Updated by sajolida 6 months ago

  • Blocked by deleted (Feature #9006: Simplify the Tails Installer splash screen)

#28 Updated by kurono 5 months ago

  • Assignee changed from kurono to sajolida
  • QA Check changed from Dev Needed to Ready for QA

#29 Updated by sajolida 4 months ago

  • Assignee changed from sajolida to kurono
  • QA Check changed from Ready for QA to Dev Needed

#30 Updated by kurono 3 months ago

  • Assignee changed from kurono to sajolida
  • QA Check changed from Dev Needed to Ready for QA

#31 Updated by sajolida 3 months ago

  • Assignee changed from sajolida to intrigeri
  • Feature Branch changed from kurono/feature/9005-Improve-tails-installer to feature/9005-Improve-tails-installer

All my concerns are solved now.

Reassigning to intrigeri for the final code review.

#32 Updated by intrigeri 3 months ago

  • Assignee changed from intrigeri to anonym
  • Target version set to Tails_3.2

Reassigning to the RM for the next major release. If I am needed for something specific, just say the word (and tell me what that thing is).

#33 Updated by intrigeri 3 months ago

  • Feature Branch changed from feature/9005-Improve-tails-installer to installer:feature/9005-Improve-tails-installer

Also, I think the test suite will need an update. anonym, that's part of "Test suite maintenance". I suggest you file a dedicated sibbling ticket for that (and I'll happily review it).

#34 Updated by intrigeri 3 months ago

sajolida: if the corresponding doc update isn't tracked by a ticket yet, please create one and ensure it blocks this one.

#35 Updated by sajolida 3 months ago

#36 Updated by sajolida 3 months ago

#37 Updated by sajolida 3 months ago

#38 Updated by intrigeri 8 days ago

I suspect my work on #12705 will be non-trivial to merge with the large amount of changes this one brings. The freeze is in two days and this branch hasn't been reviewed yet, plus we're missing the corresponding test suite and doc update. So if we have to choose, I say let's take #12705 (that I plan to complete either today or tomorrow by noon CEST), postpone this one and try harder to review this in a timely manner.

#39 Updated by u 7 days ago

  • Blocks Bug #14650: Package new upstream version of tails-installer added

#40 Updated by anonym 5 days ago

  • Assignee changed from anonym to kurono
  • % Done changed from 50 to 70
  • QA Check changed from Ready for QA to Dev Needed
  • Feature Branch changed from installer:feature/9005-Improve-tails-installer to tails:feature/12705-bump-system-partition-size installer:feature/9005-Improve-tails-installer

#41 Updated by anonym 5 days ago

  • Feature Branch deleted (tails:feature/12705-bump-system-partition-size installer:feature/9005-Improve-tails-installer)

#42 Updated by anonym 5 days ago

For the review, see: #8860#note-38 and later comments.

#43 Updated by intrigeri 1 day ago

  • Assignee changed from kurono to anonym
  • Priority changed from Normal to Elevated

(Same as #8860.)

Also available in: Atom PDF