Page MenuHomePhabricator

[M] Improve the custom license field for "not own work" in the upload wizard
Closed, ResolvedPublic

Assigned To
Authored By
Sneha
Jul 25 2024, 7:04 PM
Referenced Files
F57773003: Screen Shot 2024-12-02 at 11.44.42 AM.png
Dec 2 2024, 8:21 PM
F57754366: Screen Shot 2024-11-27 at 3.38.14 PM.png
Nov 27 2024, 11:56 PM
F57754362: Screen Shot 2024-11-27 at 3.37.47 PM.png
Nov 27 2024, 11:56 PM
F57754357: Screen Shot 2024-11-27 at 3.35.11 PM.png
Nov 27 2024, 11:56 PM
F57754353: Screen Shot 2024-11-27 at 3.24.52 PM.png
Nov 27 2024, 11:56 PM
F57754339: Screen Shot 2024-11-27 at 3.12.04 PM.png
Nov 27 2024, 11:56 PM
F57532750: custom PD tag.png
Sep 23 2024, 9:36 PM
F57532734: custom license.png
Sep 23 2024, 9:36 PM

Description

It may be better to work on this ticket after implementing T375494

Currently there is a custom license option under the first question in not-own-work flow. The option does not directly relate to the question we are asking users. In order to make the custom input option more relevant to what is being asked from the user, the proposal is to nest it under relevant options, there by keeping the main options simpler.

This will introduce under not-own-work flow:

A new option for custom license under the CC license option in q1
Link to UI

custom license.png (272×1 px, 54 KB)

  • Allow users to enter a different license not listed in the options by adding an option at end of the list of licenses as shown in the UI
  • Selecting "Enter a different license" option will reveal an input box as shown in the UI
  • The preview behaviour remains the same as it is currently implemented
  • Update the help text to the what is shown in the UI linking to the correct page as indicated in Figma.
  • Current character limit and error message for this field to be maintained
A new option for custom PD tag under the public domain option in q1
Link to UI

custom PD tag.png (290×1 px, 41 KB)

  • Add an input field for users to enter PD tag under the existing public domain questions as shown in the UI
  • The preview behaviour remains the same as it is currently implemented
  • Update the help text to the what is shown in the UI linking to the correct page as indicated in Figma
  • Current character limit and error message for this field to be maintained

Details

Related Changes in Gerrit:

Event Timeline

Sneha updated the task description. (Show Details)
MarkTraceur renamed this task from Improve the custom license field for "not own work" in the upload wizard to [M] Improve the custom license field for "not own work" in the upload wizard.Oct 15 2024, 2:33 PM

Change #1092830 had a related patch set uploaded (by Matthias Mullie; author: Matthias Mullie):

[mediawiki/extensions/UploadWizard@master] Copyright selection updates

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

Change #1092830 merged by jenkins-bot:

[mediawiki/extensions/UploadWizard@master] Copyright selection updates

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

@matthiasmullie for A new option for custom license under the CC license option in q1 - the label on commons beta says "This feature is temporarily disabled" - is it ok to have it on beta?

Screen Shot 2024-11-27 at 3.12.04 PM.png (1×2 px, 321 KB)

Also, please review the following (and adding @Sneha):

The new custom license field
(1) in current production there is validation for user input in "Enter a different license in wikitext format" field
The warning messages, e.g. "The wikitext here is too short to be a license" is placed close to the field itself:

Screen Shot 2024-11-27 at 3.24.52 PM.png (1×1 px, 200 KB)

For a new field (currently in commons beta) the warning will be displayed below several fields:

Screen Shot 2024-11-27 at 3.35.11 PM.png (1×2 px, 346 KB)
Screen Shot 2024-11-27 at 3.37.47 PM.png (1×1 px, 333 KB)
Screen Shot 2024-11-27 at 3.38.14 PM.png (1×2 px, 362 KB)

(2) {{CC-by-2.0}} and {{PD-old-70}} are underlined (and they are actual links in figma design), but in beta these are not the links.

That message was overwritten on-wiki to say "This feature is temporarily disabled" (see https://commons.wikimedia.beta.wmflabs.org/wiki/MediaWiki:Mwe-upwiz-license-custom). I deleted that and now it's fine.
FYI: in production, this is not the case. It used to be overwritten there too, but not anymore since 2012: https://commons.wikimedia.org/wiki/MediaWiki:Mwe-upwiz-license-custom

(1) As for validation: the license block as a whole is validated. That always has been the case, but I guess that wasn't as obvious in their old structure.
If that's something we want to change, let's file a new ticket!

(2) I didn't realize that these were actually a link in the design. I find it a bit confusing, though. especially since we have another link in that same line, that *is* blue. But happy to change that if needed.

That message was overwritten on-wiki to say "This feature is temporarily disabled" (see https://commons.wikimedia.beta.wmflabs.org/wiki/MediaWiki:Mwe-upwiz-license-custom). I deleted that and now it's fine.
FYI: in production, this is not the case. It used to be overwritten there too, but not anymore since 2012: https://commons.wikimedia.org/wiki/MediaWiki:Mwe-upwiz-license-custom

(1) As for validation: the license block as a whole is validated. That always has been the case, but I guess that wasn't as obvious in their old structure.
If that's something we want to change, let's file a new ticket!

(2) I didn't realize that these were actually a link in the design. I find it a bit confusing, though. especially since we have another link in that same line, that *is* blue. But happy to change that if needed.

Thx, @matthiasmullie! Regarding (1) and (2) - I think the input from @Sneha is needed. Moving to Design QA.

@Etonkovidova @matthiasmullie

(1) Yes the validation needs to be closer to the field or question it is referring to. We are already doing this in other parts of the upload wizard such as this AI validation error. If we don't fix it in this ticket it should definitely be captured in another ticket.

(2) The template example in the figma has links as it was meant to show users what templates are like. But since we already have the other link that shows a list of templates/copyright tags its okay to leave the example as implemented i.e. without any link. I will update in Figma to not have link for specific example.

Edit:
For #2 it seems like the link is already implemented in beta for both custom fields. Can we remove it so that it does not have underline and link. Because I agree it looks a bit confusing with another link which is blue.

Etonkovidova added a subscriber: Matthias.

@Etonkovidova @matthiasmullie

(1) Yes the validation needs to be closer to the field or question it is referring to. We are already doing this in other parts of the upload wizard such as this AI validation error. If we don't fix it in this ticket it should definitely be captured in another ticket.

@Sneha, @Matthias - I filed it as T381323: UploadWizard: Validation warning for custom license field should be placed closer to the field.

(2) The template example in the figma has links as it was meant to show users what templates are like. But since we already have the other link that shows a list of templates/copyright tags its okay to leave the example as implemented i.e. without any link. I will update in Figma to not have link for specific example.

Edit:
For #2 it seems like the link is already implemented in beta for both custom fields. Can we remove it so that it does not have underline and link. Because I agree it looks a bit confusing with another link which is blue.

It's done.

Screen Shot 2024-12-02 at 11.44.42 AM.png (511×990 px, 79 KB)