STR: * In sync prefs, change the device name to an empty string. It remains an empty string after clicking the change button. * Close Sync prefs and re-open - the name now has a default value. Either the prefs pane should reject an empty name, or the default value for the device name should be shown immediately after changing it to an empty string. The latter is much easier - we just re-get the device name after setting it.

Review commit: https://reviewboard.mozilla.org/r/30953/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/30953/

Comment on attachment 8708053 [details] MozReview Request: Bug 1229625 - Use the default device name as a placeholder in Sync prefs. r?markh https://reviewboard.mozilla.org/r/30953/#review28579 LGTM

[bugday-20160323] Status: RESOLVED,FIXED -> INCOMPLETE Comments: Test is working Component: Name Firefox Version 46.0b9 Build ID 20160322075646 Update Channel beta User Agent Mozilla/5.0 (Windows NT 6.1; WOW64; rv:46.0) Gecko/20100101 Firefox/46.0 OS Windows 7 SP1 x86_64 Expected Results: 1. Ask for device name if left empty 2. Reset to default device name if left empty. Actual Results: It does not ask for input even if empty string for device name 2nd case is working.

(In reply to Mayur Patil from comment #5) > [bugday-20160323] > > Status: RESOLVED,FIXED -> INCOMPLETE > ... > It does not ask for input even if empty string for device name My testing shows a placeholder with the default name when you clear the text, and that default name is used. The "INCOMPLETE" status makes me wonder if you think this bug needs a followup to work as expected?

My intention behind INCOMPLETE status is that: One test is complete while other one is not. The default name comes after the field gets cleared - done. Pref pane reject the empty string - not done. I would like to modify it to "Once user device name is saved, it should deny empty string without showing default name" Lets see if this could work it will also help to reduce devices with same name I think. What do you think?

(In reply to Mayur Patil from comment #7) > My intention behind INCOMPLETE status is that: > > One test is complete while other one is not. > > The default name comes after the field gets cleared - done. > > Pref pane reject the empty string - not done. > > I would like to modify it to "Once user device name is saved, it should deny > empty string without showing default name" We decided that "rejecting" is bad UX - it presumably means we must show a modal alert dialog with an error message, then force the user to enter some name. The alternative, and what we have implemented, is to treat the empty string as meaning the default device name should be used - and that the UI clearly shows this (ie, it is impossible to see the device name with an empty string - as soon as it becomes empty, the default device name is shown as a placeholder). Personally I think this is much better, but I guess we should ask UX. Ryan, This conversation is about what happens when the user clears the "device name" field - currently in that case we show "placeholder" text with the default device name, and that default name is used if the user presses "Save" while that placeholder is shown. Mayur is advocating that we should actively reject an empty string forcing the user to enter something in that field, and presumably showing some error message when we reject it. Do you think the current behaviour should be changed to behave more like that?

I think it's great the way it is (and as you describe it Mark). The placeholder copy is exactly what should be saved if the user clears the field.