946693 - Fix sticky headers in [SMS] app

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia, defect)

Comment 1

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

Attachment: patch in github

Comment 2

[Jose Manuel Cantera]

Comment on attachment 8343050 [details]
patch in github

redirecting the review to Borja

Comment 3

[Borja Salguero [:borjasalguero]]

Comment on attachment 8343050 [details]
patch in github

We are going to wait Travis to be green an we are ready to merge! If 'Travis' is not green rebase and push again for re-launching Travis tasks. Thanks!

Comment 4

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

Thanks Borja. Travis green!
Merged: a7d5a5e582b11b14ea7f43b5b6dc2db92a2d9131

Comment 6

[Jason Smith [:jsmith]]

We really need to nominate this for 1.3 approval. The VD fallout here is ugly.

Comment 7

[Julien Wajsberg [:julienw]]

Comment on attachment 8343050 [details]
patch in github

NOTE: Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to better understand the B2G approval process and landings.

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): I don't know
[User impact] if declined: ugly visual
[Testing completed]: yes
[Risk to taking this patch] (and alternatives if risky): quite safe
[String changes made]: none

Comment 8

[Ryan VanderMeulen [:RyanVM]]

John, can you please assist with this uplift :)

Comment 9

[John Ford [:jhford] CET/CEST Berlin Time]

Uplifted a7d5a5e582b11b14ea7f43b5b6dc2db92a2d9131 to:
v1.3: 8ae3596d2d1b215626aa3e7838ce078146e926c7