WordPress.org

Make WordPress Core

Opened 2 years ago

Last modified 9 months ago

#42599 new defect (bug)

Design changes in Tools Categories and Tags Converter box and Tools Import box

Reported by: Shital Patel Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 4.9
Component: Import Keywords: has-patch needs-testing
Focuses: ui, administration Cc:
PR Number:

Description

Some Design changes in Categories and Tags Converter box and Import box
Go to wp-admin

  1. Tools > Available Tools
  2. Tools > Import

Attachments (7)

before_categories_tags_converter_box.png (37.4 KB) - added by Shital Patel 2 years ago.
before_import_box.png (53.6 KB) - added by Shital Patel 2 years ago.
after_categories_tags_converter_box.png (36.5 KB) - added by Shital Patel 2 years ago.
after_import_box.png (53.9 KB) - added by Shital Patel 2 years ago.
42599.patch (752 bytes) - added by Shital Patel 2 years ago.
42599.1.patch (854 bytes) - added by Shital Patel 2 years ago.
src level pathch
42599.2.patch (1.2 KB) - added by Shital Patel 23 months ago.
Responsive Tools Categories and Tags Converter box and Tools Import box

Download all attachments as: .zip

Change History (18)

@Shital Patel
2 years ago

#1 @Shital Patel
2 years ago

Hello,
after change in css I added patch 42599.patch.​ Please check and let me know your views.

#2 @Shital Patel
2 years ago

  • Focuses administration added

@Shital Patel
2 years ago

src level pathch

#3 @Shital Patel
2 years ago

  • Focuses ui added
  • Keywords has-patch added

#4 follow-up: @welcher
23 months ago

  • Keywords ui-feedback needs-testing added

@Shital Patel thanks for the ticket and patch. How is this change reflected in multiple screen sizes?

@Shital Patel
23 months ago

Responsive Tools Categories and Tags Converter box and Tools Import box

#5 in reply to: ↑ 4 @Shital Patel
23 months ago

Replying to welcher:

@Shital Patel thanks for the ticket and patch. How is this change reflected in multiple screen sizes?

I have been added new patch for Responsive CSS. Sorry, I don't have an iphone. So I checked with web developer tool for responsive design. Let me know if any changes in responsive design.

This ticket was mentioned in Slack in #design by boemedia. View the logs.


15 months ago

#7 @boemedia
15 months ago

Hey @Shital Patel,

We discussed your ticket today in the design team's ticket triage and I need to find a moment to test your patch. But I was curious what your reason is for the text box to go full width.

#8 @boemedia
15 months ago

  • Keywords ui-feedback needs-testing removed

I tested the patch, it works for me! See this screen recording I did: https://cl.ly/tADD

#9 @karmatosed
15 months ago

Just noting a little point to consider is character length per line. With these changes it pushes it to just over the maximum and may go even further with translations.

#10 @calebj7
15 months ago

This comment is in response to @karmatosed point about length and readability in the description.
In the example above it has "Categories and Tags Converter" mentioned a second time in the description but as a link to the tool; I like that it makes finding it easy by having the link provided, but having the link be the exact name of the Tool in the description seems redundant and wordy. Having the title of the tool (Categories and Tags Converter) as the link (flashes blue as you mouse over it or just a cursor change), maintains the accessibility of the link and allows the description to be much shorter and to the point.
For example:

Categories and Tags Converter

Easily convert categories to tags (or vice versa).

(Exactly 50 characters)

Taking out the redundancy and allowing the tool title to be the link should, theoretically, help keep the description shorter in any language.

Also, it does seem odd that there is a page for available tools that force you to go somewhere else to activate the said tool. An idea would be to have an activator next to each tool that you could switch on and off as desired.

Last edited 15 months ago by calebj7 (previous) (diff)

#11 @Shital Patel
12 months ago

  • Keywords needs-testing added
Note: See TracTickets for help on using tickets.