1126209 - [RTL][Homescreen]The confirm information is not right-aligned when deleting apps from the homescreen.

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::Homescreen, defect, P2)

Description

[Sue]

Attachment: deleteapps.png

[1.Description]:
[RTL][Homescreen]When you long press an app at homescreen and delete it, the confirm information is not right-aligned.
See attachment:deleteapps.png

[2.Testing Steps]: 
Prerequisite: Set system language as Arabic.
1. Long press an app at homescreen.
2. Tap the "x" button to delete a app.(Such as Marketplace)

[3.Expected Result]: 
2. The confirm information should be right-aligned.

[4.Actual Result]: 
2. The confirm information is left-aligned.

[5.Reproduction build]: 
Flame 2.2 build:
Gaia-Rev        0518f4581a0925c0b703d730ef289ab15cbd1216
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/c6aa604a7967
Build-ID        20150126002536
Version         37.0a2
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20150126.042138
FW-Date         Mon Jan 26 04:21:49 EST 2015
Bootloader      L1TC000118D0

[6.Reproduction Frequency]: 
Always Recurrence,5/5

[7.TCID]: 
Free Test

[8.Note]:
The delete button and cancel button are not mirrored when deleting the SmartCollections.
See attachment:deleteSmartCollections.png

Comment 1

[Sue]

Attachment: deleteSmartCollections.png

Comment 2

[Guillaume Marty [:gmarty]]

Attachment: Github PR

Kevin, can you have a look?

Comment 3

[Kevin Grandon :kgrandon]

Comment on attachment 8557042 [details] [review]
Github PR

This looks good. This does only fix the smart collections confirm screen, and not the uninstall app one. I believe the uninstall app one lives in the system somewhere.

Comment 4

[Sue]

Attachment: confirm.png

When delete app from App Permissions, the confirm message is left-aligned and Delete, Cancel buttons are not mirrored.
Repro Steps:
1. Set system language as Arabic.
2. Launch Settings -> App Permissions.
3. Select Geoloc and tap Uninstall App button.
**The confirm message is left-aligned and Delete, Cancel buttons are not mirrored.
See attachment:confirm.png
Rate:5/5

Flame 2.2 build:
Build ID               20150203002504
Gaia Revision          cd62ff9fe199fb43920ba27bd5fdbc5c311016fc
Gaia Date              2015-02-03 00:56:43
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/11d93135c678
Gecko Version          37.0a2
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150203.041704
Firmware Date          Tue Feb  3 04:17:15 EST 2015
Bootloader             L1TC000118D0

Comment 5

[Delphine Lebédel [:delphine - Use needinfo]]

Sue: not sure why you verified this bug again. It didn't land anywhere yet!

Comment 6

[Sue]

(In reply to Delphine Lebédel [:delphine - use need info] from comment #5)
> Sue: not sure why you verified this bug again. It didn't land anywhere yet!

Hi Delphine,
This case that was mentioned on comment 4 was found on free test. And according to comment 3, to make sure all cases are covered, I add this case to this bug for your further reference.

Comment 7

[Michael Henretty [:mikehenrty][:mhenretty]]

Guillaume, does it make sense to fix the bug mentioned in comment 4 here, or should we file a new bug?

Comment 8

[Sue]

Update the result info of step 3 in comment 4. The buttons are mirrored, but confirm message is not right-aligned.

Comment 9

[Guillaume Marty [:gmarty]]

Hey Kevin, I double-checked and it seems that this PR fixes both dialog as they're based on <gaia-confirm> even though they're not using the same markup.

I'll land it for now and we can always reopen it if anything is missing.

Comment 10

[Kevin Grandon :kgrandon]

Sounds good. Just make sure the commit is linked in this bug? Thanks!

Comment 11

[Michael Henretty [:mikehenrty][:mhenretty]]

(In reply to Kevin Grandon :kgrandon from comment #10)
> Sounds good. Just make sure the commit is linked in this bug? Thanks!

And request uplift :)

Comment 12

[Guillaume Marty [:gmarty]]

Landed in master https://github.com/mozilla-b2g/gaia/commit/6855dbe2b6ed6e15a0380c1229d9631837a2bda5

Comment 13

[Guillaume Marty [:gmarty]]

Comment on attachment 8557042 [details] [review]
Github PR

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): RTL
[User impact] if declined: Bad UI for RTL languages
[Testing completed]: Manual testing
[Risk to taking this patch] (and alternatives if risky): CSS change only
[String changes made]: None

Comment 14

[Brogan Zumwalt [Inactive]]

Issue verified fixed in Flame 3.0

With language set to Arabic, deletion confirmation screen when deleting Smart Collections, Marketplace Apps, and Browser Bookmarks from homescreen, as well as uninstalling an app from App Permissions in settings shows text as right aligned.

Adding verifyme for 2.2 verification ahead of uplift

Device: Flame 3.0 Master
Build ID: 20150210010523
Gaia: 0cf517083f7eb5fc269e1236edba50534f65e3cd
Gecko: 2cb22c058add
Gonk: e7c90613521145db090dd24147afd5ceb5703190
Version: 38.0a1 (3.0)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:38.0) Gecko/38.0 Firefox/38.0

Comment 15

[(PTO May 7-12) Ryan VanderMeulen [:RyanVM]]

v2.2: https://github.com/mozilla-b2g/gaia/commit/70a042446985ab13cfc91e2e48a4eafccf3d2036

Comment 16

[Sue]

Attachment: verify.png

This issue has been verified successfully on Flame v2.2
See attachment:verify.png
STR1:
Prerequisite: Set system language as Arabic.
1. Long press an app at homescreen.
2. Tap the "x" button to delete a app.(Such as Marketplace)
**The confirm information is right-aligned.
Rate:0/5

STR2:
1. Set system language as Arabic.
2. Launch Settings -> App Permissions.
3. Select Marketplace and tap Uninstall App button.
**The confirm message is right-aligned.
Rate:0/5

Flame 2.2 build:
Gaia-Rev        791e53728cd8018f1d7cf7efe06bbeb1179f0370
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/5dec207fcbeb
Build-ID        20150212002504
Version         37.0a2
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20150212.042208
FW-Date         Thu Feb 12 04:22:18 EST 2015
Bootloader      L1TC000118D0

Comment 17

[Sue]

Test case has been added in moztrap:
https://moztrap.mozilla.org/manage/case/15439/