897303 - [Buri][Fugu][Settings]The highlighted language will instead of selected language previous.

Status: RESOLVED DUPLICATE of bug 819619

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

Description

[sync-1]

AU_LINUX_GECKO_ICS_STRAWBERRY.01.01.00.019.158
 Firefox os  v1.1
 Mozilla build ID:20130709070206
 
 Created an attachment (id=467305)
 the log
 
 DEFECT DESCRIPTION:
 The highlighted language will instead of selected language previous.
 
 REPRODUCING PROCEDURES:
 Go to Settings ->"Language"->I've selected "English(US)"->Click "English(US)" to access "select" list->Click any other language sunch as "Polski"->Don't click "OK",but press home key->You'll find "Polski" is selected as current Language->KO
 
 EXPECTED BEHAVIOUR:
 If you don't click"OK",the highlighted selected language should not be changed. ->OK
 
 ASSOCIATE SPECIFICATION:
 TEST PLAN REFERENCE:
 TOOLS AND PLATFORMS USED:
 USER IMPACT:
 Mid
 REPRODUCING RATE:
 5/5
 For FT PR, Please list reference mobile's behavior:

Comment 1

[sync-1]

Clone from brother

Comment 2

[sync-1]

Attachment: the log

Comment 3

[James Zhang (Spreadtrum)]

Attachment: 897303.patch

Fugu and buri also have this bad UE issue.
Please review the patch and fix it, thanks.

Comment 4

[Jason Smith [:jsmith]]

Tim probably help review your patch

Comment 5

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

Thanks for the patch.

This is a desired behavior of the <select> and value selector. The selected value changes BEFORE user tapping "OK". I've suggested "Close" would be a better label for the button, but Josh (UX) disagree.

Please sound your opinion in bug 819619.

Comment 6

[Steven Yang [:styang]]

james, do you want your patch to fugu's repo?

Comment 7

[James Zhang (Spreadtrum)]

(In reply to Steven Yang [:styang] from comment #6)
> james, do you want your patch to fugu's repo?

v1.2 or v1.3 branch is better. It's gaia issue.

Comment 8

[James Zhang (Spreadtrum)]

(In reply to Steven Yang [:styang] from comment #6)
> james, do you want your patch to fugu's repo?

Please ask owner to review jesse's patch.