GNOME Bugzilla – Bug 738077
Setting non-default language is unnecessarily difficult
Last modified: 2014-10-31 08:24:33 UTC
There is unnecessarily too much steps to set non-default language thru users panel: 1/ click on editable combo, it shows combo with default languages 2/ click on "Other...", it shows language chooser with the same default languages (just sorted differently) 3/ click on "..." to show all languages One of the steps could be omitted, so it could be fixed in two ways: - do not show the combo and show the language chooser immediately (same as it is done in region panel) - or show directly all languages in the chooser after clicking to "Other..." in the combo Designers, what is better?
Looks like we are duplicating the "common case" twice, indeed. I'd drop the menu myself and go directly to the modal dialog. I think it's going to be cleaner in terms of code as well?
Yes, this option is definitely cleaner from my perspective. :-)
Turning the drop-down menu into a button is probably the easiest indeed.
Created attachment 288576 [details] [review] user-accounts: simplify language changing thru users panel Do not show combo box with common languages and show the language chooser immediately as it is done in the region panel.
Created attachment 288577 [details] [review] cc-language-chooser: do not show all languages after _clear_filter There is another bug fix, because this bug is more visible now thanks to the previous patch. Function cc_language_chooser_clear_filter clears just filter label, however the label is shown and all languages are listed. Clear the dialog to the initial state on cc_language_chooser_clear_filter so that we can reuse the dialog later.
Review of attachment 288576 [details] [review]: Looks good.
Review of attachment 288576 [details] [review]: Make that after freeze, we haven't branched yet.
Review of attachment 288577 [details] [review]: What's that patch for? The commit message doesn't explain the user-visible results of this.
The language chooser dialog is used repeatedly. When we click to show more languages and/or select language, close the dialog and show it again e.g. for different user, the chooser isn't in an initial state. So filter entry isn't hidden (and all languages are shown) and/or it is focused the last focused language. But I think the dialog should be in the initial state when it is open... is this clearer?
(In reply to comment #9) > The language chooser dialog is used repeatedly. When we click to show more > languages and/or select language, close the dialog and show it again e.g. for > different user, the chooser isn't in an initial state. So filter entry isn't > hidden (and all languages are shown) and/or it is focused the last focused > language. But I think the dialog should be in the initial state when it is > open... is this clearer? It's clearer, can you please incorporate this into the commit message, and make sure that the subject line contains the description of the user-visible changes, not a shorter version of what's in the body.
Created attachment 288591 [details] [review] cc-language-chooser: Reset dialog to original state when used the 2nd time With updated description...
Review of attachment 288591 [details] [review]: Looks good.
Comment on attachment 288591 [details] [review] cc-language-chooser: Reset dialog to original state when used the 2nd time commit 066d66383e9c3d476eb85c6f6e8dfd42e6eb2bac Thanks for the reviews.
Comment on attachment 288576 [details] [review] user-accounts: simplify language changing thru users panel commit 96c0c0a838dd14fa27962968649395949b0664b5