898987 - [bb][refactor] Freeze current BB by moving them into v1 folder

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia, defect)

Comment 1

[Arnau March [:arnau] ( not working in Firefox OS anymore :( )]

Attachment: patch in github

Comment 2

[Ismael Gonzalez [:basiclines]]

Comment on attachment 784931 [details]
patch in github

This patch looks robust, anyway i will feel more confortable if some Gaia peer take a look at the webapp-zip.js also.
Thx guys!

Comment 3

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Comment on attachment 784931 [details]
patch in github

I would like to have Vivien to review the patch too.

Also I think Yuren could read through the webapps-zip.js part.

Please removing the type as originally planned.

Comment 4

[Yuren [:yurenju] (AFK ~ Oct. 12) (volunteer mode)]

Comment on attachment 784931 [details]
patch in github

looks fine for me, but please make sure you have tested it on windows.

Comment 5

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Comment on attachment 784931 [details]
patch in github

I don't have the bandwidth to r? this correctly but from what I've understood of the patch f+.

Comment 6

[Fabien Cazenave [:kaze]]

Comment on attachment 784931 [details]
patch in github

Please, let’s not merge that today: the “Building Blocks v2” aren’t ready yet and they’ll change a lot in the next few weeks (hopefully).

Before we can document and finalize the class names, we need to have the “full picture” — which supposes at least these three steps imho:
 • have a v2 stylesheet for *every* Building Block (we only have “headers.css” so far and its class names are likely to change);
 • (my preferred part) bikeshed about class names so that they make sense for designers, UX engineers and developers;
 • have an expert review on the whole, especially a maintainability evaluation.

One of the reasons why the current shared/style folder has become “what it is” is that it’s very difficult to improve these shared stylesheets once they’re used by apps. Let’s not repeat this mistake, please.

Once our v2 Building Blocks become usable, applying this patch and starting the upgrade will still be doable. There’s no point in rushing this patch.

Comment 7

[Arnau March [:arnau] ( not working in Firefox OS anymore :( )]

We won't work on that by now.