Problem/Motivation

On file field widget configuration forms, the user is presented with this text to help in choosing between throbber and progress bar:

The throbber display does not show the status of uploads but takes up space. The progress bar is helpful for monitoring progress on large uploads.

The word "less" is missing.

Proposed resolution

Change the text to:

The throbber display does not show the status of uploads but takes up less space. The progress bar is helpful for monitoring progress on large uploads.

Remaining tasks

There is some discussion below as to whether the setting should even exist, or whether the text makes sense at all. These issues should possibly be addressed in the future. The patch itself merely corrects what is clearly a typo.

User interface changes

String change: On file field widget configuration forms, the description will change from:

The throbber display does not show the status of uploads but takes up space. The progress bar is helpful for monitoring progress on large uploads.

to:

The throbber display does not show the status of uploads but takes up space. The progress bar is helpful for monitoring progress on large uploads.

API changes

(API changes/additions that would affect module, install profile, and theme developers, including examples of before/after code if appropriate)

Original report by @mlncn

When editing an imagefield, for instance, the user is presented with this text to help in choosing between throbber and progress bar:

"The throbber display does not show the status of uploads but takes up space. The progress bar is helpful for monitoring progress on large uploads."

I think the word "less" is missing in the first sentence? Unless the author really doesn't like throbber!

Shall i roll a patch to change the text to:

"The throbber display does not show the status of uploads but takes up less space. The progress bar is helpful for monitoring progress on large uploads."

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

webchick’s picture

Component: forms system » user interface text

LOL. Yeah, sounds good.

mlncn’s picture

Component: user interface text » file system
Priority: Minor » Normal
Status: Active » Needs review
Issue tags: +#d7ux
FileSize
987 bytes

In a small way, this is a usability issue. Patch attached.

mlncn’s picture

Component: file system » user interface text

Crosspost- just correcting the component back, here. I didn't even know user interface text was an option, heh.

Dries’s picture

Seems like a silly setting to begin with.

tstoeckler’s picture

Re #4:
I think what users will experience is that they have an uploadprogress bar on their local site (that's the default setting, right?), then push the site to their shared hosting and just see the throbber, because on that server the extension is not installed.
For that reason I would either:
a. never show this setting which means less UI for most users and a bit confusion for the above mentioned users.
b. always show this setting (also when the extension is not installed) which means more confusing (?) UI for all users but no confusion for the above users.

Currently we only show the setting if the extension is installed. I agree with Dries that that doesn't make much sense.

henrrrik’s picture

This is a no-brainer, no? The current text stating that the throbber "does not show the status of uploads but takes up space" doesn't make any sense.

yoroy’s picture

Version: 7.x-dev » 8.x-dev

I would prefer to drop this setting from core, pick an option already and make it easy to override/customize it.

henrrrik’s picture

Version: 8.x-dev » 7.x-dev
Status: Needs review » Reviewed & tested by the community

Switching this back to 7.x. This issue just concerns a typo. I think a functional change for 8.x should have its own issue.

Can someone please commit this? It's a one word change.

yoroy’s picture

Version: 7.x-dev » 8.x-dev
Priority: Normal » Minor

Even typos get fixed in current head first, then possible backported. I'm fine with keeping this issue limited to that.

droplet’s picture

Status: Reviewed & tested by the community » Needs work

"The throbber display does not show the status of uploads"

It doesn't make sense, it show an icon to you.

henrrrik’s picture

Status: Needs work » Reviewed & tested by the community

Can we create a new separate issue if we are to dive into the semantics of this? That way we can get this typo fixed and anyone who wants to work on improving the rest of the message can do that until the end of time if they like.

This issue celebrates 1 year next week. One year to fix a typo.

xjm’s picture

Tagging issues not yet using summary template.

xjm’s picture

Issue summary: View changes

Updated issue summary.

xjm’s picture

Issue summary: View changes

Better formatting.

xjm’s picture

Summary added, such as it is.

sun’s picture

Indeed, this typo is ridiculous. Removing the setting in D8+ can be done afterwards, in this or in a new issue.

Dries’s picture

Version: 8.x-dev » 7.x-dev

Committed to 8.x. Moving to 7.x for webchick.

webchick’s picture

Status: Reviewed & tested by the community » Fixed
Issue tags: +String freeze

LOL! That's a pretty funny typo. :)

We normally don't break strings in stable releases for minor changes like this. But in this case the string is actually wrong, so I think fixing it makes sense. it's also a string buried in an admin page so hopefully won't affect too many people.

Committed and pushed to 7.x. Thanks.

Status: Fixed » Closed (fixed)
Issue tags: -String freeze, -#d7ux, -dgd7

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

Anonymous’s picture

Issue summary: View changes

Meh, whitespace.