Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[UX] Installer: Revamp the language selection page #3975

Open
klonos opened this issue Aug 10, 2019 · 9 comments
Open

[UX] Installer: Revamp the language selection page #3975

klonos opened this issue Aug 10, 2019 · 9 comments

Comments

@klonos
Copy link
Member

klonos commented Aug 10, 2019

Steps To Reproduce

  1. Start installing Backdrop:

Screen Shot 2019-08-10 at 5 55 46 am

  1. Click on "Learn how to install Backdrop in other languages" -> page refreshes, and you get this:

Screen Shot 2019-08-10 at 5 55 56 am

Actual behavior vs. expected/ideal behavior

  • The "Learn how to install Backdrop in other languages" link opens another installer page, but does not change the "Choose language" step on the sidebar (although there was a page refresh). That is not that bad I guess, but I expected the link to open another tab to a b.org handbook page; otherwise, why not have it be a button instead?

  • If all we are doing is to merely show a page with only textual info and links, with the intention being not to bloat the main page, then why not make the link a fieldset, which would present that info w/o any page refresh?

  • The page title with the textual info is also "Choose language", whereas there clearly is no such option in this page. If we are to keep this as a separate page, then why not have its title be "Install Backdrop in other languages" instead?

  • The "Follow these steps to translate Backdrop into your language" bit is not accurate. Following these instructions in not translating Backdrop; it is enabling another language (which Backdrop has already been translated to) for the installer, and the site once the installation has finished.

  • I would expect a 3rd step (bullet point), which would say something like "Refresh the installer page. The language you installed in the previous step should now be available in the language selection menu."

  • The "For more information..." bit is injected into a place that stops the natural flow of things in that page. I would prefer it at end of all that text, or preferably at the top, as an info message.

  • The links in the "How should the installation continue?" section feel like should have been action buttons instead. I understand that they may have been added added as links because of their lengthy text, but still feels wrong.

Proposed solution
Include the "Follow these steps ..." text in the "Choose language" step, as a collapsed-by-default fieldset:

Screen Shot 2019-08-10 at 8 14 38 am

...expanded:
Screen Shot 2019-08-10 at 8 14 52 am

PR by @klonos: backdrop/backdrop#2930

@klonos
Copy link
Member Author

klonos commented Aug 10, 2019

...ideally, at some point in the future, we should implement a feature where the installer is able to download languages from the translation server. This fieldset should still be shown, to account for custom languages (Klingon anybody? 😄), or languages that have not been translated yet.

@olafgrabienski
Copy link

olafgrabienski commented Aug 15, 2019

@klonos Thanks for the PR! I like the collapsed fieldset at the beginning, but was confused to see the "Install in another language" fieldset again, after adding a language file. To see the (quite prominent) heading again, it felt like I missed something when I had added the language file. Here are screenshots of the process:


1. Started without having a language file in the respective directory:

screen-backdrop-pr-2814-a-start


2. Opened the fieldset:

screen-backdrop-pr-2814-b-open


3. Added language file and clicked "Reload this page":

screen-backdrop-pr-2814-c-with-lang-file

In my opinion, when I just have provided a language file, the "Install in another language" section isn't needed (or at least too prominent).

@olafgrabienski
Copy link

Here's an idea: When a language file has been provided, we could change the wording of the field set. See in comparison:


Step 3, as in the PR:

screen-backdrop-pr-2814-c-with-lang-file


Step 3, proposal (collapsed):

screen-backdrop-pr-2814-d-proposal-collapsed


Step 3, proposal (expanded):

screen-backdrop-pr-2814-e-proposal-expanded

@olafgrabienski
Copy link

Another idea: make the field set heading less promiment. Mockup:

screen-backdrop-pr-2814-f-proposal-less-prominent

@klonos
Copy link
Member Author

klonos commented Oct 10, 2019

Thanks for the feedback @olafgrabienski and sorry for not responding earlier.

I see the point you are making re the wording of the fieldset, but I find it simpler to make it always be "Make more languages available". Besides, "Install in another language" implies that you can select other languages; whereas what is actually happening is you are given instructions on how to enable more.

Another UX issue is with the dropdown when there's only a single language available:

Screen Shot 2019-10-10 at 11 27 23 pm

The user is asked to Choose a language -> they click on the dropdown -> there is only a single option -> waste of clicks/time 👎 (which is basically the same as #1442). I was thinking that perhaps we could render the select as radios if there are say 3 or less languages:

Screen Shot 2019-10-10 at 11 26 58 pm

Screen Shot 2019-10-10 at 11 28 13 pm

...then switch to dropdown if more (which I believe will be a rare case btw):

Screen Shot 2019-10-10 at 11 29 34 pm

As for making the fieldset label lowercase, I will wait for more feedback, since that is how the Seven admin theme renders fieldsets.

@olafgrabienski
Copy link

I see the point you are making re the wording of the fieldset, but I find it simpler to make it always be "Make more languages available".

Agreed!

I was thinking that perhaps we could render the select as radios if there are say 3 or less languages

Nice idea. Would we use this pattern only here (for the time being), or are you considering to make it a more general rule which applies here and in other places?

As for making the fieldset label lowercase, I will wait for more feedback, since that is how the Seven admin theme renders fieldsets.

Yes, I guess it makes sense to open a separate issue for the upper vs. lowercase question of fieldsets in Backdrop.

@klonos
Copy link
Member Author

klonos commented Oct 12, 2019

I was thinking that perhaps we could render the select as radios if there are say 3 or less languages

Nice idea. Would we use this pattern only here (for the time being), or are you considering to make it a more general rule which applies here and in other places?

Only here. The most common use case would be for 1 or 2 additional languages to be added, so making them a set of radio options would be one less click (and there is no vertical space issue that we need to be concerned with here).

This select has the potential to grow to a big number of options though (if all possible languages are added), so falling back to a select after a threshold seems logical to me.

I have filed a new PR to implement this: backdrop/backdrop#2930

I also changed the fieldset label to always be "Make more languages available".

Yes, I guess it makes sense to open a separate issue for the upper vs. lowercase question of fieldsets in Backdrop.

Done: #4124 #4125 😉

@klonos
Copy link
Member Author

klonos commented Oct 12, 2019

Here's some screenshots from my local (with #4125 applied):

Most people will only ever see this:
Screen Shot 2019-10-12 at 11 38 52 am

...with this if they expand the fieldset:
Screen Shot 2019-10-12 at 11 39 01 am

...if they add 1 or 2 language files:
Screen Shot 2019-10-12 at 11 43 27 am

...and if they add more language files:
Screen Shot 2019-10-12 at 11 44 13 am

@olafgrabienski
Copy link

@klonos You were asking me in backdrop/backdrop#2814 (comment) if I could move my comment about the URL to the translation server here so we can get more feedback re the URL to use. Here it is:

I'm not sure which URL to the translation server we should use. Currently (and in the PR) we link to https://localize.backdropcms.org/translate/languages.

Possible alternatives:

@klonos klonos self-assigned this Nov 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants