WordPress.org

Making WordPress.org

Opened 7 months ago

Closed 7 months ago

Last modified 6 months ago

#2628 closed defect (fixed)

Plugin Directory: Too short plugin description with letter count locale

Reported by: mayukojpn Owned by: obenland
Milestone: Plugin Directory v3.0 Priority: normal
Component: Plugin Directory Keywords: has-patch commit
Cc:

Description

I found new ja localized plugin directory has too short plugin description in a front page / archives. It might come from letter count system instead of word count.

https://ja.wordpress.org/plugins/

Attachments (3)

貼り付けた画像_2017_03_29_16_09.png (140.4 KB) - added by mayukojpn 7 months ago.
2628.patch (1.1 KB) - added by SergeyBiryukov 7 months ago.
2628.2.patch (888 bytes) - added by SergeyBiryukov 7 months ago.

Download all attachments as: .zip

Change History (17)

This ticket was mentioned in Slack in #polyglots by mayuko. View the logs.


7 months ago

#2 in reply to: ↑ 1 ; follow-up: @afzalmultani
7 months ago

Can we change the link text that currently says "Translation Contributors"? It leads to the translation stats and doesn't immediately show contributors?

#3 in reply to: ↑ 2 ; follow-up: @mayukojpn
7 months ago

Replying to afzalmultani:
Hey, let's make separate ticket for different problem 😄 Are you going to make it or @petya?

#4 @mayukojpn
7 months ago

  • Milestone set to Plugin Directory v3.0

#5 in reply to: ↑ 3 @afzalmultani
7 months ago

Replying to mayukojpn:

Replying to afzalmultani:
Hey, let's make separate ticket for different problem 😄 Are you going to make it or @petya?

okey i'm created ticket #2629

#6 @SergeyBiryukov
7 months ago

  • Summary changed from Too short plugin description with letter count locale to Plugin Directory: Too short plugin description with letter count locale

#7 @obenland
7 months ago

  • Keywords close added

The excerpt is very short because there is only room for three short lines. It uses `wp_trim_words()` which is supposed to account for letter count locales

#8 follow-up: @SergeyBiryukov
7 months ago

  • Keywords needs-patch added; close removed

wporg-plugins theme is supposed to limit the excerpt length to 15 words, see the excerpt_length() function.

In Japanese and Chinese (and perhaps other East Asian locales), wp_trim_words() counts characters instead of words.

Seems like the limit should be increased for those locales. If the current UI can account for three lines, there's no need to overzealously limit the excerpt to just 15 characters.

#9 @SergeyBiryukov
7 months ago

  • Keywords has-patch commit added; needs-patch removed

#10 in reply to: ↑ 8 @SergeyBiryukov
7 months ago

Replying to SergeyBiryukov:

Seems like the limit should be increased for those locales.

2628.patch implements that, using the same approach as in WordPress core.

#11 @obenland
7 months ago

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

In 5196:

Plugin Directory: Adjust exerpt length by locale

Allows for a longer excerpt in letter count locales.

Props SergeyBiryukov.
Fixes #2628.

#12 @SergeyBiryukov
7 months ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

@obenland: Sorry, missed text domain in my initial patch, see 2628.2.patch.

[5196] should still work, as it would use the core translation, but we should still add the text domain for clarity and best practices.

#13 @obenland
7 months ago

  • Resolution set to fixed
  • Status changed from reopened to closed

In 5212:

Plugin Directory: Use textdomain in i18n function

Props SergeyBiryukov.
Closes #2628.

This ticket was mentioned in Slack in #meta-i18n by mayuko. View the logs.


6 months ago

Note: See TracTickets for help on using tickets.