Page MenuHomePhabricator

{{R from alternative language}} parameters should be optional
Closed, ResolvedPublicBUG REPORT

Description

Reported by @DreamRimmer on Discord

  • {{r from alternative language}} "When using the Page Curation Toolbar to add redirect category {{r from alternative language}}, we are currently required to input languages in the "From language (two-letter code):" and "To language (two-letter code):" fields. but I think these parameters should be optional, not mandatory" (patch)
  • {{r from alternative language}} "also some grammar problems"

Novem:

  • the below text box alignment doesn't look great either. may be worth putting them both on different lines using <br> or something

image.png (264×716 px, 28 KB)

Event Timeline

Novem_Linguae renamed this task from {{R from alternative language}} parameters should be optional to {{R from alternative language}} and {{Cleanup}} parameters should be optional.Sep 7 2023, 4:33 AM
Novem_Linguae updated the task description. (Show Details)
Novem_Linguae updated the task description. (Show Details)
Novem_Linguae renamed this task from {{R from alternative language}} and {{Cleanup}} parameters should be optional to {{R from alternative language}} parameters should be optional.Sep 7 2023, 4:36 AM
Novem_Linguae updated the task description. (Show Details)

also some grammar problems

Please suggest new phrasing. The current phrase is "redirect from or to a title in another language", which i don't have any particular problem with.

"From language" and "To language" parameters should be optional : I've made the required edit request for the test wiki here.

This comment was removed by MPGuy2824.

Text box alignment issues: These might be solved by the same changes done in T345153: Fix Date range alignment issues in filter menu in some browsers.

I don't see any issue with the grammar. Perhaps I got a bit confused. My apologies for the mistake.

This comment was removed by MPGuy2824.

Change 956101 had a related patch set uploaded (by MPGuy2824; author: MPGuy2824):

[mediawiki/extensions/PageTriage@master] Set two parameters for a redirect template to be optional

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

Change 956101 abandoned by MPGuy2824:

[mediawiki/extensions/PageTriage@master] Set two parameters for a redirect template to be optional

Reason:

forgot to branch

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

Change 956102 had a related patch set uploaded (by MPGuy2824; author: MPGuy2824):

[mediawiki/extensions/PageTriage@master] Set two parameters for a redirect template to be optional

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

Change 956102 had a related patch set uploaded (by MPGuy2824; author: MPGuy2824):

[mediawiki/extensions/PageTriage@master] Set two parameters for a redirect template to be optional

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

Works on patchwiki.

Change 956102 merged by jenkins-bot:

[mediawiki/extensions/PageTriage@master] Set two parameters for a redirect template to be optional

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