That page is currently inconcistent with all the surrounding steps - they all have a continue/save and continue/start button, but the language selection work off a link. with no continue button.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Bojhan’s picture

You are correct, could you make a screenshot of your solution?

catch’s picture

I clicked on the language link for the first time in probably a couple of years by accident, and got a bit confused. Probably need to try this out to see how it works, but seems like clicking one button + radio would've prevented that.

seutje’s picture

Status: Active » Needs review
FileSize
3.09 KB

Attached patch will change the list item for English into a radio-input when it's the only language found and will display a full list of languages if more than 1 language is found (like it did before)

When the form is exposed and the default profile is used, the 'Learn how to install Drupal in other languages'-link will be added under all radios but before the button

Maybe this should be 'When any profile but the expert profile is used' and currently English will not be selected by default, even when it's the only option, so perhaps we should discuss that since there's no real disadvantages that arise from default to the only option available

seutje’s picture

Status: Needs review » Needs work

patched wrong version... fail

seutje’s picture

Status: Needs work » Needs review
FileSize
3.47 KB

patched the right version now, tested by manually applying the patch and adding a couple languages, seems to work as expected

Status: Needs review » Needs work

The last submitted patch failed testing.

seutje’s picture

Status: Needs work » Needs review

1 silly exception on upload user picture :/

NaheemSays’s picture

The patch works well (thanks for taking this up!), and this works well, is less disconcerting, however a couple of nitpicks:

1. With only one language (english) there, it should IMO be selected by default.
2. the bit in the end "(built in)" should IMO be replaced with "(default)".

seutje’s picture

@8

1. I agree, seems silly not to
2. I dunno, I would remove it all together

seutje’s picture

FileSize
123.36 KB
19.58 KB

forgot to add screens

Bojhan’s picture

FileSize
3.46 KB

I think we should remove, the li. And thus so, did.

Bojhan’s picture

FileSize
3.46 KB

Changed it to Save and continue, to be consistent with the rest of buttons in the installation process. Although select language, was already good - using something consistent here, will avoid any hesitation later on.

seutje’s picture

what should it default to?
if no other languages to english
if one other language to nothing or to the only other language?
if more than 1 language to nothing or to english?

NaheemSays’s picture

I think "always to english" is a good default... but I may be biased here. :)

For other languages, the person will have to take additional steps anyway, though I can see the logic in having the other language as default if only that and english are listed.

Bojhan’s picture

English, is the default. We could add logic, but not sure if that is correct.

NaheemSays’s picture

The patch in comment 12 seems to be missing the change in text of the button (on irc, I queried it and it was suggested to use the same "save and continue" as used elsewhere for consistency).

Also. if the "Learn how to install Drupal in other languages" is to be moved next to the select button, there needs to be a gap in between the two.

NaheemSays’s picture

FileSize
3.53 KB

patch also replacing "select language" with "save and continue".

In addition to that, with the removal of the list item in the previous patch I did not like how there was no separation between the text and the select button, so I added a paragraph tag to where there originally were the list tags.

NaheemSays’s picture

FileSize
3.84 KB

and now with english selected by default (if it is present).

Bojhan’s picture

Intresting, if someone can give this a code review, then its a RTBC

Status: Needs review » Needs work

The last submitted patch failed testing.

sign’s picture

Status: Needs work » Needs review
FileSize
4.05 KB

re-rolled and tested, looks ok to me

Bojhan’s picture

Status: Needs review » Reviewed & tested by the community

Looks good.

Dries’s picture

- "Save and continue" sounds a little funny -- would it be better to say just "Next"?

- Having only one radio button looks a bit odd, but I guess that is OK?

Bojhan’s picture

- Actually, save and continue is good. We use it consistently in Drupal core and in this installer?

- I think its ok, just makes people press continue faster - which should be an win

sign’s picture

Radio button looks ok to me, imho it makes it more clear that there could be more options, rather than a link saying "Install in English"

webchick’s picture

Status: Reviewed & tested by the community » Needs review

Yeah, I agree that the one radio button looks like a total UI wtf, and it's one that probably 98% of our users will be exposed to on literally the first screen that they see in Drupal. Radios are only supposed to be used if there is more than one choice available. From http://www.useit.com/alertbox/20040927.html:

Radio buttons are used when there is a list of two or more options that are mutually exclusive and the user must select exactly one choice.

I spoke with Bojhan about two possible alternatives:

1. Remove the radio button when there's only one choice available (change it into #type => markup). He doesn't like this approach because it doesn't inform the user about the fact that there will be further choice there if they follow the instructions in the link below.

2. Just remove this step in the installer altogether if there's only one language. Bojhan is skeptical that we can pull this off by our 12/1 deadline. I'm inclined to agree, and I'm not sure what the i18n team would think about this change.

I do agree that offering radios as opposed to links is a much better UI construct, since it's not remotely obvious that links are presenting a choice, but...

I'm going to kick this back to "needs review" for another 48 hours and see if we can come up with something better. If not, bump it back to RTBC and we'll go with this approach.

David_Rothstein’s picture

2. Just remove this step in the installer altogether if there's only one language. Bojhan is skeptical that we can pull this off by our 12/1 deadline. I'm inclined to agree, and I'm not sure what the i18n team would think about this change.

From a technical standpoint this is very easy to do - because it's actually what we do already for all other profiles except the default profile :) However, that behavior is basically a bug: #562722: Language choice during installation - Drupal(minimal) profile skips language choice

***

In terms of other ideas, I almost wonder if the "Learn how to install Drupal in other languages" link could be turned into a radio button also? In other words, something like this:

Choose language

  • English (built-in)
  • Other languages (download required)

[Continue]

webchick’s picture

Interesting! What would happen when the second option was chosen? We'd be taken to the existing instructions page?

That would resolve my concerns with using radio buttons here. yoroy/Bojhan what say you?

David_Rothstein’s picture

What would happen when the second option was chosen? We'd be taken to the existing instructions page?

Yeah, that's what I was thinking.

Bojhan’s picture

Well I am really not sure if this is better. Yes it solves the problem, of one radio but it creates a another problem. It sets the expectation, that one can have radio's which don't actually offer any choice - but rather continues to a help text page? I am really unsure, if we want to introduce this into Drupal.

yoroy’s picture

don't think the latest suggestion makes more sense than a single radio. The 'learn how to' is really a link to somewhere else and should behave like that.

1. Remove the radio button when there's only one choice available (change it into #type => markup). He doesn't like this approach because it doesn't inform the user about the fact that there will be further choice there if they follow the instructions in the link below.

Hmm, I actually like this option best. Is it really up to this particular screen to describe the consequences of clicking the 'learn how to…' link? Couldn't that be mitigated on the 'learn how to' page itself? I don't see how pointing out the consequences of things you haven't chosen to do yet is good ux. It sounds like the overly helpful help texts we've actively been cutting out of the interface.

If we must, a rewrite of the link text could solve this a bit: 'Get more languages to show up here…"

yoroy’s picture

… which is just me argueing for the solution that's considered not feasible anymore at this stage, so no new insights after all…

Sooo, with no better idea than the single radio I think we should just do it. It's a bit stupid but there's nothing in it that will break if you mindlessly/impatiently click the 'save and continue' button. It's sub-optimal but not too much mental friction introduced here I think.

seutje’s picture

I agree with Bojhan that a radio button that just brings the user to a help page is a really bad solution, it's basically using a form for something it's not intended to be used for

why don't we just stick to the current behaviour when there is only 1 language available, and use radio buttons only when there are more than 1?

so in case of 1 language: show 2 links, no form
in case of more: show radio buttons with the link to the help stuff and a button

if it's imperative that we keep a consistent workflow during the install, as in use a form with a consistent button to proceed, then I'd say a regular markup item with "Installing Drupal in English." and a link to the help stuff and a button to proceed seems like the best option

even if this means using a form without presenting any options... This is done quite a lot, just think of those forms where they make you review everything you've entered before proceeding, it's essentially the same thing, you're making the user stop and think if he wants to proceed with the current settings

webchick’s picture

I dunno. I still think that presenting a single radio button choice here puts way too much visual emphasis on Drupal's multilingual installation capabilities which only apply to the tiniest fraction of the people installing Drupal. Most will just install it in English and then add their native languages later on through the Locale UI afterwards.

Bojhan, is there any chance in swaying your opinion towards #26 option 1?

Bojhan’s picture

Status: Needs review » Reviewed & tested by the community

I am still somewhat estranged by this, why is it better to have 2 radio's where only one offers an actual choice - then having 1 radio? If it puts to much visual emphasis, I think its even more now - as we treat both options as equals even though they aren't.

I am sorry webchick, but I mean - we can do a lot of things not to make it one radio, but none of them seem right (are we not just focusing to much on finding the perfect solution here? Take a look at the flow, it doesn't seem right in the flow thats what we should fix first)

Setting patch #21 back to RTBC.

webchick’s picture

Status: Reviewed & tested by the community » Fixed

All right. I still think this looks like a bug, and works against the well-established UI pattern for radios, but I agree that the current situation with two links and no button doesn't make any sense, so (I guess) it's better than doing nothing...

Committed to HEAD.

David_Rothstein’s picture

why is it better to have 2 radio's where only one offers an actual choice

So, the unspoken followup issue here would have been to actually make both of them offer an actual choice.... (i.e., use the update manager to let people download and install new languages directly from within the UI on the "help screen", which would then no longer just be a help screen anymore).

That would be really neat, but any effort to get the update manager working in the installer in Drupal 7 should probably be focused on #418302: Copy default.settings.php to settings.php during install IFF webserver owns files (FTP on shared hosting) at this point (since it's a much more important issue), so for now, I guess a single radio button is the best we can do here...

Bojhan’s picture

@David_Rothstein Right on, yup!

Status: Fixed » Closed (fixed)
Issue tags: -Usability, -D7UX

Automatically closed -- issue fixed for 2 weeks with no activity.