Make WordPress Core

Opened 15 months ago

Last modified 15 months ago

#58722 new defect (bug)

Fix RTL display in Optional grouping on Permalinks Settings page

Reported by: sabernhardt's profile sabernhardt Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Permalinks Keywords: has-patch needs-testing has-testing-info
Focuses: rtl, multisite Cc:

Description

splitting from #47755 (props rachid84, johnbillion, SergeyBiryukov, costdev, joedolson, afercia, ryokuhi, oglekler)

For the Category/Tag Base fields, subdirectory network installations can show a /blog prefix before the input field. In RTL languages, the slash is on the wrong side. Also, the example URL can wrap awkwardly to the next line. In any language, the text is stacked above the field with little space between them on smaller screens.

  1. Create a multisite network with subdirectories.
  2. Open the main site.
  3. Visit Settings -> Permalinks.
  4. Scroll down to the Optional section.

Change History (4)

This ticket was mentioned in PR #2955 on WordPress/wordpress-develop by @sabernhardt.


15 months ago
#1

  • Keywords has-patch added

In the main site of subdirectory network installations, the blog prefix does not display correctly in RTL.

LTR (at 820 pixels wide and at 600):

https://i0.wp.com/user-images.githubusercontent.com/17100257/177835146-9eec7d5c-8cf5-424e-bc22-77f89cbedb7b.png

https://i0.wp.com/user-images.githubusercontent.com/17100257/177835149-89e628d9-b00b-49c1-b7ee-3e57697fa8e3.png

In RTL, the slash displays on the wrong side of "blog" and the prefix appears on the wrong side of the input field. Plus, the example URL can wrap awkwardly to the next line.

https://i0.wp.com/user-images.githubusercontent.com/17100257/177835151-82d8d5f4-1d0d-4ea1-9aa3-6766f2b9cf55.png

https://i0.wp.com/user-images.githubusercontent.com/17100257/177835150-4036b888-f931-41d4-abdf-0c365d0a0cf0.png

This wraps the prefix and field in a span with the code class and adds CSS to keep the prefix next to the field on smaller screens. In the paragraph, code elements are set to inline-block to keep them on one line (or wrap better).

Trac ticket: https://core.trac.wordpress.org/ticket/58722

#3 @costdev
15 months ago

  • Keywords has-testing-info added

#4 @sabernhardt
15 months ago

  • Milestone changed from 6.3 to Future Release
Note: See TracTickets for help on using tickets.