Project

General

Profile

Bug #14627

Bug #12328: Tails Verification: Migrate DAVE to Web Extensions and support Chrome

Design the new download page

Added by sajolida 3 months ago. Updated 22 days ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
Installation
Target version:
Start date:
07/28/2016
Due date:
% Done:

0%

QA Check:
Pass
Feature Branch:
master
Type of work:
User interface design
Blueprint:
Starter:
Affected tool:
Verification Extension

Description

1. Design new wireframes
2. Test the wireframes on paper
3. Write the resulting HTML code

Let's say that this tickets covers everything needed to have a download page that works with the new extension and has enough CSS to be usable.

The integration with the other pages and the replacement of DAVE will happen on #14630.

The polishing of the CSS will happen on #14921.


Related issues

Blocks Tails - Feature #14628: Migrate Firefox extension to Web Extensions Confirmed 09/13/2017
Blocks Tails - Feature #14629: Tails Verification for Google Chrome Confirmed 11/08/2017

History

#1 Updated by sajolida 3 months ago

  • Target version changed from Tails_3.3 to Tails_3.2

#2 Updated by sajolida 3 months ago

  • Blocks Feature #14628: Migrate Firefox extension to Web Extensions added

#3 Updated by sajolida 3 months ago

#4 Updated by intrigeri 3 months ago

  • Status changed from Confirmed to In Progress

#5 Updated by sajolida 3 months ago

  • Assignee changed from sajolida to uzairfarooq
  • QA Check set to Ready for QA

I consider this as Ready for QA since I sent my mail to Uzair:

https://mailman.boum.org/pipermail/tails-dev/2017-September/011694.html

Now I'm waiting for Uzair to clarify whether this works for him or what I would need to adjust.

I'll do the rest of the HTML and CSS work as part of #14630, hopefully in another branch.

Uzair: once you're done answering the points I raised over email or you feel like I should be the one to work again on this, reassign it to me and mark it as QA Check: Dev Needed.

#6 Updated by anonym 2 months ago

  • Target version changed from Tails_3.2 to Tails_3.3

#7 Updated by sajolida about 1 month ago

  • Description updated (diff)
  • Assignee changed from uzairfarooq to sajolida
  • QA Check changed from Ready for QA to Dev Needed

#8 Updated by sajolida about 1 month ago

  • Subject changed from Design DAVE 2 to Design the new download page

#9 Updated by sajolida about 1 month ago

  • Feature Branch set to master

I solved the issues identified by Cody in #14630#note12 and did some more stuff:

77aa685f4b Install Chrome extension when clicking 'chrome' links
0839177672 Lower minimum version until release
f6b45948ed Use same htmlscrubber_skip setting in both config
d500dc9e17 Put most significant works first in the bullet list
c2ef736a4a Add explanation about what is BitTorrent
a767793b5c Download ISO image using the JSON mirror pool
5db3efa3cb Link to ISO image and Torrent file
4fb057ec40 Always download latest version of Tails Verification XPI
bc8f14c371 Let's call the extension 'Tails Verification'
7528371613 Use more meaningful labels

Note: the inline installation of the Chrome extension is broken. I don't know why but don't have time to investigate this further now. Maybe waiting a bit could solve this actually... :)

#10 Updated by sajolida about 1 month ago

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

Cody: Can you review these commits?

#11 Updated by cbrownstein about 1 month ago

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

The "Install Tails Verification extension (no restart)"; "Skip verification"; and "Skip download" links don't work. (I'm not sure if that will be fixed on #14630.)

Nitpicky, but I think "Install Tails Verification extension (no restart)" should read "Install Tails Verification extension (no restart needed)." "No restart" by itself isn't particularly instructive.

Also nitpicky: "You need to install a BitTorrent software on your computer...." I don't believe it's proper to refer to specific software as "a software." E.g., you'd instruct someone to "install text-editing software" to open a text file. You wouldn't instruct them to "install a text-editing software."

#12 Updated by sajolida 28 days ago

  • Target version changed from Tails_3.3 to Tails_3.5

#13 Updated by sajolida 25 days ago

  • Status changed from In Progress to Resolved
  • Assignee deleted (sajolida)
  • QA Check deleted (Dev Needed)

Changing "a BitTorrent software" to "BitTorrent software".

The "(no restart)" labeling originally came straight from Mozilla but I think it's not relevant for Web Extensions anymore, so I removed it.

Yes, all the links should be fixed in #14630.

So I think I can close this ticket no. Wow!

#14 Updated by sajolida 25 days ago

  • Assignee set to cbrownstein
  • QA Check set to Ready for QA

Actually, I did some more tiny changes that I want you to review:

8fac860a59 Add missing period
8c97d18222 Ask people without JavaScript to allow it temporarily
949eb24d42 Remove '(no restart)' label
8b02c3468e Fix grammar
fabeb1cf7f Always point to latest version

#15 Updated by sajolida 25 days ago

  • Status changed from Resolved to In Progress

#16 Updated by cbrownstein 25 days ago

  • Assignee changed from cbrownstein to sajolida
  • QA Check changed from Ready for QA to Pass

Everything looks good to me!

#17 Updated by intrigeri 24 days ago

  • Affected tool changed from Download and Verification Extension to Verification Extension

#18 Updated by sajolida 22 days ago

  • Status changed from In Progress to Resolved
  • Assignee deleted (sajolida)

Cool, so I'm closing this ticket again :)

Also available in: Atom PDF