Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Normalize copy around Image Processing functions #280

Closed
jeffpaul opened this issue May 19, 2021 · 6 comments · Fixed by #325
Closed

Normalize copy around Image Processing functions #280

jeffpaul opened this issue May 19, 2021 · 6 comments · Fixed by #325
Assignees
Labels
help wanted Extra attention is needed needs:engineering This requires engineering to resolve. needs:ux This requires user experience to resolve. type:enhancement New feature or request. type:good-first-issue Good for newcomers.
Milestone

Comments

@jeffpaul
Copy link
Member

jeffpaul commented May 19, 2021

Is your enhancement related to a problem? Please describe.
I received the following feedback via email:

I believe it should be called "Use OCR to scan for text in the image" to explain what it actually does and connect back to the setting itself. The same goes for Rescan Alt Text. The setting of managing this is called "Automatically Caption Images". Not a very obvious connection. On top, you know there is actually another media meta field called Caption. To make the chance of confusion even bigger the setting is explained with "Images will be captioned with alt text upon upload". Does this mean that the Alt Text will be put as Caption as well? It seems that the Settings dialogue and the meta box was written by two completely different departments who never spoke to each other.

Describe the solution you'd like
There's a solid point here where we should look at normalizing labels and helper text in different places:

ALT TEXT

  • Setting label: Automatically Caption Images
  • Setting helper text: Images will be captioned with alt text upon upload
  • Threshold setting label: Caption Confidence Threshold
  • Threshold setting helper text: Minimum confidence score for automatically applied image captions, numeric value from 0-100. Recommended to be set to at least 75.
  • Data currently flows into the Alternative Text field
  • Media Library image processing option label: Scan Alt Text and Rescan Alt Text

TAGGING

  • Setting label: Automatically Tag Images
  • Setting helper text: Images will be tagged upon upload
  • Threshold label: Tag Confidence Threshold
  • Threshold helper text: Minimum confidence score for automatically applied image tags, numeric value from 0-100. Recommended to be set to at least 70.
  • Media Library image processing option label: Generate Tags and Regenerate Tags

SMART CROPPING

  • Setting label: Enable smart cropping
  • Setting helper text: Crop images around a region of interest identified by ComputerVision
  • Media Library image processing option label: Generate Smart Thumbnail and Regenerate Smart Thumbnail

OCR

  • Setting label: Enable OCR
  • Setting helper text: Detect text in an image and store that as post content
  • Media Library image processing option label: Scan Text and Rescan Text

Designs

Describe alternatives you've considered

Additional context

@jeffpaul jeffpaul added type:enhancement New feature or request. help wanted Extra attention is needed type:good-first-issue Good for newcomers. needs:ux This requires user experience to resolve. labels May 19, 2021
@jeffpaul jeffpaul added this to the 1.8.0 milestone May 19, 2021
@jeffpaul
Copy link
Member Author

Following on from this is additional feedback on the Computer Vision endpoint setting field:

Also, in the Settings, mentioning the Endpoint URL the e.g. is "https://REGION.api.cognitive.microsoft.com/" and that is not at all how the endpoint looks, at least not for me. Mine is "https://REDACTED.cognitiveservices.azure.com/" where the "REDACTED" was designed by my choices setting it up in Azure.

So we may want to further generalize the helper text for the Endpoint URL field or add a secondary example there (e.g., https://EXAMPLE.cognitiveservices.azure.com).

@jeffpaul
Copy link
Member Author

Noting that we'll want to credit https://github.com/myBCN for the original issue report on this.

@vikrampm1 vikrampm1 added needs:engineering This requires engineering to resolve. and removed hacktoberfest labels Dec 17, 2021
@ajmaurya99 ajmaurya99 self-assigned this Jan 6, 2022
@ajmaurya99
Copy link

Hey @jeffpaul I have started a WIP PR for the above change - #322
Could you please have a look, Currently I have updated all the ALT TEXT labels and helper text with the TAGGING labels and helper text.

I am not sure if the SMART CROPPING and the OCR section also requires any change.

@jeffpaul
Copy link
Member Author

jeffpaul commented Jan 6, 2022

@ajmaurya99 #322 is an incorrect approach. We need to work to make the copy in each section more normalized/similar so that we reference things within the Alt Text, Tagging, Smart Cropping, and OCR sections all reference terms in the same manner. I'm going to work on getting some copyediting help on this as that will help outline the changes needed more precisely. Until then, you can probably close that WIP PR and I'll ensure this issue is updated when we have better copy available.

@ajmaurya99 ajmaurya99 removed their assignment Jan 7, 2022
@ajmaurya99
Copy link

Thanks @jeffpaul, I have closed the WIP PR.

@jeffpaul
Copy link
Member Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed needs:engineering This requires engineering to resolve. needs:ux This requires user experience to resolve. type:enhancement New feature or request. type:good-first-issue Good for newcomers.
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants