WordPress.org

Make WordPress Core

Opened 4 weeks ago

Last modified 4 weeks ago

#46032 new defect (bug)

Plugin details text direction layout (RTL)

Reported by: nabilmoqbel Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 5.0.3
Component: Plugins Keywords: ui-feedback has-patch has-screenshots
Focuses: ui, rtl, administration Cc:

Description

Hi,

This RTL CSS minor was noticed before 5.0 release, but with increasing number of plugins that have been translated to RTL languages! It would be nice fix it at present.

In the plugin details screen, when site language or admin choosing RTL panel:

Text direction for plugin Description, Changelog, Screenshots captions, and Reviews tabs appears LTR and some phrases become not quickly understood.

(Screenshot attached https://moqbel.net/s/WP-admin-plugins-description-RTL-PrtScn).

In this case (especially for those plugins that have been translated to any RTL language) I think the following needs to be fixed:

direction: ltr;

#plugin-information .section {
    direction: rtl;
}

And for section ul class:

margin-left: 24px;

#plugin-information .section ol, #plugin-information .section ul {
    list-style-type: disc;
    margin-right: 24px;
}

For Reviews tab (RTL):

  1. star-rating:

float: left;

#plugin-information .reviewer-info .star-rating {
    float: right;
}
  1. review-title-section h4

float: left;

margin: 0 6px 0 0;

#plugin-information .review-title-section h4 {
    display: inline-block;
    float: right;
    margin: 0 0 0 6px;
}
  1. reviewer-info .avatar

float: left;

margin: 4px 6px 0 0;

#plugin-information .reviewer-info .avatar {
    float: right;
    margin: 4px 0 0 6px;
}

Finally will be displayed correctly as this PrtScn:

https://moqbel.net/s/WP-admin-plugins-description-RTL-PrtScn-fixed

Many thanks,

Attachments (3)

plugins-description-RTL.png (458.8 KB) - added by nabilmoqbel 4 weeks ago.
Plugins description RTL
46032.diff (1.0 KB) - added by mukesh27 4 weeks ago.
Patch.
46032-common.css.patch (923 bytes) - added by man4toman 4 weeks ago.

Download all attachments as: .zip

Change History (6)

@nabilmoqbel
4 weeks ago

Plugins description RTL

@mukesh27
4 weeks ago

Patch.

#1 @mukesh27
4 weeks ago

  • Component changed from Administration to Plugins
  • Keywords has-screenshots added

Hi @nabilmoqbel, Welcome to WordPres Trac! Thanks for the report.

Good catch.

#2 @nabilmoqbel
4 weeks ago

Hi @mukesh27, You're welcome.
Thank you for that!

#3 @man4toman
4 weeks ago

Hi everyone,
The patch must be on none-rtl version of CSS file.
I attach new one.

Last edited 4 weeks ago by man4toman (previous) (diff)
Note: See TracTickets for help on using tickets.