Making WordPress.org

#7182 closed enhancement (fixed)

Alphabetize rejection list in Photos

Reported by: topher1kenobe's profile topher1kenobe Owned by: coffee2code's profile coffee2code
Milestone: Priority: normal
Component: Photo Directory Keywords:
Cc:

Description

When you go to reject a photo there's a drop down list of reasons. It would be great if they were alphabetical

Attachments (1)

CleanShot 2023-08-05 at 14.11.06@2x.png (507.7 KB) - added by topher1kenobe 10 months ago.
Screenshot of Rejection reasons not alphabetized.

Download all attachments as: .zip

Change History (7)

@topher1kenobe
10 months ago

Screenshot of Rejection reasons not alphabetized.

#1 @paulkevan
10 months ago

I guess there is a reason for ordering them in this way (or maybe it's just how they have been added) - not sure if making them alphabetic is better, but they are just set within code here: https://github.com/WordPress/wordpress.org/blob/91fc0d424e513dc5bba990f3547fb8d1d0dff8be/wordpress.org/public_html/wp-content/plugins/photo-directory/inc/rejection.php#L30-L107.

#2 @coffee2code
10 months ago

@topher1kenobe: Do you think it would be better to alphabetize the existing list, or to alphabetize the list after prepending each with a one-word token. The former would look like:

Do not reject
Collage or composite image
Contains human face(s)
Duplicate or minor variation of submitted photo
Image subject matter
Insufficient image quality (e.g. blurriness, composition, lighting, lens issues)
Not a photo (e.g. screenshot, digital art)
Overlays, watermark, borders, or other additions
Overprocessed
Potential copyright/ownership infringement
Potentially violates privacy
Predominantly another piece of art
Predominantly text
Prominent branding
Reason specified below
Submission error
Violence, gore, hate, or sexual content

And the latter would look like:

Do not reject
[Anothers Art] Predominantly another piece of art
[Branding] Prominent branding
[Collage] Collage or composite image
[Copyright] Potential copyright/ownership infringement
[Error] Submission error
[Extreme] Violence, gore, hate, or sexual content
[Faces] Contains human face(s)
[Not Photo] Not a photo (e.g. screenshot, digital art)
[Other]Reason specified below
[Overlays] Overlays, watermark, borders, or other additions
[Overprocessed] Overprocessed
[Privacy] Potentially violates privacy
[Quality] Insufficient image quality (e.g. blurriness, composition, lighting, lens issues)
[Subject] Image subject matter
[Text] Predominantly text
[Variation] Duplicate or minor variation of submitted photo

And if the brackets are too much noise, this example show colons and dashes:

Do not reject
Anothers Art: Predominantly another piece of art
Branding: Prominent branding
Collage: Collage or composite image
Copyright: Potential copyright/ownership infringement
Error: Submission error
Extreme: Violence, gore, hate, or sexual content
Faces: Contains human face(s)
Not Photo: Not a photo (e.g. screenshot, digital art)
Other - Reason specified below
Overlays - Overlays, watermark, borders, or other additions
Overprocessed - Overprocessed
Privacy - Potentially violates privacy
Quality - Insufficient image quality (e.g. blurriness, composition, lighting, lens issues)
Subject - Image subject matter
Text - Predominantly text
Variation - Duplicate or minor variation of submitted photo

#3 @topher1kenobe
10 months ago

I think I like the third option quite a lot.

#4 @coffee2code
10 months ago

@topher1kenobe: With colons or dashes?

#5 @topher1kenobe
10 months ago

I can't actually think of a reason it matters. Visually I like colons, but that's completely subjective.

#6 @coffee2code
10 months ago

  • Owner set to coffee2code
  • Resolution set to fixed
  • Status changed from new to closed

In 12840:

Photo Directory, Rejection: Add prefix label to each rejection reason in dropdown and alphabetize the list.

Props topher1kenobe, coffee2code.
Fixes #7182.

Note: See TracTickets for help on using tickets.