Making WordPress.org

Opened 2 years ago

Closed 2 years ago

#6013 closed defect (bug) (reported-upstream)

Wrong word-wrap on advanced page commiter widget

Reported by: sebastienserre's profile sebastienserre Owned by:
Milestone: Priority: normal
Component: Plugin Directory Keywords: close
Cc:

Description

Hello,

On the French translation of the advanced view for plugins, the Committers title is non well displayed:
https://fr.wordpress.org/plugins/wp-openagenda/advanced/
https://i.imgur.com/eVNYFOO.png

I'm not really good at CSS but it seems that adding a word-wrap solves the issue

h1, h2, h3, h4, h5, h6 {
    font-family: Open Sans,sans-serif;
    clear: both;
    line-height: 1.5;
    margin: 2rem 0 1rem;
    word-wrap: initial;
        overflow-wrap: initial;
}

}}}

https://imgur.com/a/aH5qyPK

Attachments (4)

tgyR1cr.png (198.4 KB) - added by sebastienserre 2 years ago.
Screenshot 2022-01-07 at 10.02.50.png (22.8 KB) - added by jeherve 2 years ago.
Old wp-admin Users screen, small screen size
Screenshot 2022-01-07 at 10.02.34.png (28.6 KB) - added by jeherve 2 years ago.
Newer WP-admin Users screen, small screen size
Screenshot 2022-01-07 at 10.06.33.png (238.9 KB) - added by jeherve 2 years ago.
Plugin Advanced screen in French, with added spaces

Download all attachments as: .zip

Change History (9)

@jeherve
2 years ago

Old wp-admin Users screen, small screen size

@jeherve
2 years ago

Newer WP-admin Users screen, small screen size

@jeherve
2 years ago

Plugin Advanced screen in French, with added spaces

#1 @jeherve
2 years ago

We used to have a similar issue in WP-Admin, on the Users screen: user roles such as "Administrator" used to be translated as "Administrateur/administratrice", and that used to cause issues on smaller screen widths.

https://meta.trac.wordpress.org/raw-attachment/ticket/6013/Screenshot%202022-01-07%20at%2010.02.50.png

A few weeks ago, and following discussions in the WPFR Slack team, @audrasjb fixed the issue in wp-admin by adding a non-breaking space before the slash, and a space after. Here is an example:
https://translate.wordpress.org/projects/wp/dev/fr/default/?filters%5Bstatus%5D=either&filters%5Boriginal_id%5D=7578405&sort%5Bby%5D=translation_date_added&sort%5Bhow%5D=asc&historypage

While it may be not entirely typographically correct to add in spaces around the / in French, it does allow the role to wrap properly on smaller screens.

https://meta.trac.wordpress.org/raw-attachment/ticket/6013/Screenshot%202022-01-07%20at%2010.02.34.png

Maybe we could do the same here in Meta? Here is how it would look like:

https://meta.trac.wordpress.org/raw-attachment/ticket/6013/Screenshot%202022-01-07%20at%2010.06.33.png

I've suggested a new translation following a similar pattern here:
https://translate.wordpress.org/projects/meta/plugins-v3/fr/default/?filters%5Bstatus%5D=either&filters%5Boriginal_id%5D=3472711&filters%5Btranslation_id%5D=91534531

#2 follow-up: @audrasjb
2 years ago

Translation proposal approved, I should fix the issue :)

#3 @jeherve
2 years ago

  • Keywords close added

#4 in reply to: ↑ 2 @jeherve
2 years ago

Replying to audrasjb:

Translation proposal approved, I should fix the issue :)

Thanks! :)

#5 @dd32
2 years ago

  • Resolution set to reported-upstream
  • Status changed from new to closed

Closing as reported-upstream as it was fixed "upstream", and I don't want to wontfix/worksforme/invalid/fixed :)

Note: See TracTickets for help on using tickets.