Tails

#1 Updated by sajolida 2015-05-12 15:26:05

• blocks Feature #9385: CSS for "download and verify" page added

#2 Updated by sajolida 2015-05-13 19:18:25

• Status changed from New to In Progress
• Assignee changed from sajolida to tchou
• QA Check set to Ready for QA
• Blueprint set to https://tails.boum.org/blueprint/bootstrapping/extension/prototype/

Done a first version in commit:e80fc8b.

Tchou, please have a look and comment. I guess that starting to format all this for bootstrap will make it more clear what is missing.

#3 Updated by intrigeri 2015-07-01 10:37:50

• Target version changed from Tails_1.4.1 to Tails_1.5

#4 Updated by sajolida 2015-08-15 11:12:17

• Priority changed from Normal to Elevated
• Target version changed from Tails_1.5 to Tails_1.6

Postponing now that 1.5 is out. Please take more care of the tickets marked for a target version when this one comes close to the release as the release manager has to handle all these tickets and it adds noise to his work.

This has now been postponed twice, so raising priority. I hope Maone is not blocked to much by this work…

#6 Updated by tchou 2015-09-02 07:57:44

• Assignee changed from tchou to sajolida

I go on on Feature #9385, because even if I change the html, it’s mostly because of css implementation.

#7 Updated by sajolida 2015-09-02 09:55:11

• QA Check changed from Ready for QA to Dev Needed

Thanks for the review. I’m glad the HTML code was almost good on the first try!

It’s also good to see that you can work on Git branch! I think we should do this more while working on the assistant otherwise we’ll get into trouble and pain.

Still, it seems like we need to clarify where to continue working on the HTML and CSS code for the download page as we’re doing things differently now.

I was working on that code in /blueprint on master, so that Giorgio and others could see the code live in there already. I’m not sure that’s a strict requirement but I though that would be useful. While I see that you copied that code into your branch 9384 (branched off web/assistant).

I think we need to clarify where do we want to work on that in the future.

To be able to make this decision, let’s keep in mind that in the final code of the assistant, the download page will be slightly different for each scenario (at least in order to know where people are coming from and where they are going to). See the URL scheme on https://tails.boum.org/blueprint/bootstrapping/assistant#index5h1). So the page you created /install/download-and-verify.mdwn is anyway not going to be the final destination of this code. Also, when we’ll want to implement these download pages, one per scenario, we’ll probably have to use inlines and other CSS tricks and we still don’t know where these inlines will be stored.

So here is what I propose for now:

• We keep working on this code /blueprint has we have been doing until now. This implies rescuing your code from your branch and reapplying it on top of the current blueprint. It won’t be funny but I’ll do it this time.
• We start writing these download pages, one per scenario, in the web/assistant branch.
• While doing this we adapt the code in the blueprint as needed for it’s reuse in web/assistant. This will probably involve creating inlines and other funky ikiwiki tricks.

I’ll try to do that this week.

As a general note, be careful with the state of the branches you are working on while writing your code. For example, please work on topic branches as much as you can, and branch them always from the web/assistant of origin. For example, I still couldn’t merge the work you did 2 months ago on Feature #9313 because you didn’t reply to my review so if you continue building things up on top of this it will create important Git issues that either you (ideally) or I (in sacrifice mode) will have to deal with. Consult your local Git expert for more help.

#9 Updated by sajolida 2015-09-07 10:35:18

• Assignee changed from sajolida to tchou

#10 Updated by tchou 2015-09-22 07:54:35

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

#11 Updated by tchou 2015-10-31 04:14:11

• Assignee changed from tchou to sajolida

Just to say the the latest version of the html code is in Feature #9385 (working on the css classes made me modify html).

#12 Updated by sajolida 2015-11-02 15:21:04

• Assignee changed from sajolida to tchou
• Feature Branch set to web/9385-download-and-verify-css

Updating as of Feature #9385#note-15 as you’re working on both in the same branch.

#13 Updated by sajolida 2015-11-19 09:09:26

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