Make WordPress Core

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#29753 closed defect (bug) (fixed)

CSS rules removing strong in string (plugin card)

Reported by: pavelevap Owned by: SergeyBiryukov
Priority: normal Milestone: 4.1
Component: Plugins Version: 4.0
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

We are using strong in following string:

<strong>Untested</strong> with your version of WordPress

But then removing it by CSS. I am not sure if I am missing something, but CSS part should be removed, I guess (or strong from string). See attached patch...

Attachments (1)

css_strong.patch​ (393 bytes ) - added by pavelevap 12 years ago.

Download all attachments as: .zip

Change History (7)

#1 @voldemortensen
12 years ago

  • Keywords has-patch added

#2 @SergeyBiryukov
12 years ago

I think this is intentional, so that the emphasis is only on "Compatible" or "Incompatible" strings.

#3 @pavelevap
12 years ago

Sergey: OK, but why we have CSS rule and strong tag in this string? We can remove both CSS rule and strong tags?

#4 @SergeyBiryukov
12 years ago

  • Milestone Awaiting Review → 4.1

Yeah, I guess we can.

#5 @SergeyBiryukov
12 years ago

  • Owner set to SergeyBiryukov
  • Resolution → fixed
  • Status new → closed

In 29775:

Remove unnecessary CSS overriding on Add Plugins screen.

props pavelevap.
fixes #29753.

#6 @nacin
12 years ago

The intention here, FWIW, was so we could introduce the strings, get them translated, and then decide on the exact styling. These strings came pretty late in the 4.0 cycle, and helen and I weren't sure how they'd be styled, but we knew that they'd say.

Changing it now is no problem.

Note: See TracTickets for help on using tickets.