Page MenuHomePhabricator

Grammar error in Special:Preferences (and Special:GlobalPreferences)
Closed, ResolvedPublic

Description

Under the "advanced options" section, at the third option ("Use non-JavaScript interface"), the small text under reads: "Loads RecentChanges without filters search or highlighting functionality.". This should be corrected to "Loads RecentChanges without filters, search or highlighting functionality.".

Event Timeline

Jdforrester-WMF subscribed.

I believe that this label might be intentionally talking about the "filters search" functionality. Tagging with the relevant project; this isn't an issue with the preferences system.

kostajh added subscribers: MMiller_WMF, Trizek-WMF, kostajh.

@MMiller_WMF @Trizek-WMF we added this message a while back. Does the proposed change look OK to you?

I believe that this label might be intentionally talking about the "filters search" functionality. Tagging with the relevant project; this isn't an issue with the preferences system.

Hmm, if it is so, then perhaps correcting it to "filter search" would be appropriate?

Yeah, you can search for a given filter. I let native English speakers decide on the right way to explain it. :)

While we are here, if we are correcting this sentence, we should cover the fact that is is also covering Special:Watchlist and Special:RecentChangesLinked. And probably other pages in the future. Maybe we should have something more neutral and capable of dealing with the diversity of the usages?

OK, so how about:

Loads RecentChanges, Watchlist, and similar pages without the filtered search or the highlighting functionality.

?

I think @Jdforrester-WMF's version looks good to me. Thank you!

Change 697716 had a related patch set uploaded (by Kosta Harlan; author: Kosta Harlan):

[mediawiki/core@master] RCFilters: Make preference help string more broadly applicable

https://gerrit.wikimedia.org/r/697716

Change 697716 merged by jenkins-bot:

[mediawiki/core@master] RCFilters: Make preference help string more broadly applicable

https://gerrit.wikimedia.org/r/697716

Um, actually, if it's fine, I'd suggest a clarification to this:

Loads RecentChanges, Watchlist, and similar pages without the filtered search or highlighting functionality.

I think this makes it clearer and more concise.

In T283734#7130869, @EpicPupper wrote:

Um, actually, if it's fine, I'd suggest a clarification to this:

Loads RecentChanges, Watchlist, and similar pages without the filtered search or highlighting functionality.

I think this makes it clearer and more concise.

I intentionally re-injected the definite article to clarify that the highlighting functionality is distinguishable from the search functionality, especially for non-English translations, but perhaps?

That sounds fair. Closing this as resolved, thanks for the patch!