WordPress.org

Make WordPress Core

Opened 2 years ago

Closed 13 months ago

Last modified 13 months ago

#17842 closed defect (bug) (fixed)

Style var tag in Twenty Eleven

Reported by: mfields Owned by: nacin
Priority: normal Milestone: 3.4
Component: Bundled Theme Version: 3.2
Severity: normal Keywords: has-patch dev-feedback
Cc: michael@…, peterhol

Description

var tags are currently not styled differently than other tags. Would it be a good idea to style them like code and kbd?

Attachments (1)

17842.diff (929 bytes) - added by mfields 19 months ago.
Add var and samp elements to style.css and editor-style.css.

Download all attachments as: .zip

Change History (9)

comment:1 mfields2 years ago

  • Cc michael@… added

comment:2 PeteMall2 years ago

  • Summary changed from Style var tag in wenty Eleven to Style var tag in Twenty Eleven

comment:3 peterhol23 months ago

  • Cc peterhol added

Should perhaps be added to editor-style.css as well?

comment:4 mfields23 months ago

Actually, the var tag seems to be accounted for in Twenty Eleven's editor-style.css, but only when used as a child of a "code" element.

comment:5 nacin19 months ago

Looks good. Also samp could be added. Let's cover editor-style.css too.

mfields19 months ago

Add var and samp elements to style.css and editor-style.css.

comment:6 mfields19 months ago

I've updated the patch adding the samp element and accounting for editor-style.css.

comment:7 nacin13 months ago

  • Owner set to nacin
  • Resolution set to fixed
  • Status changed from new to closed

In [20906]:

Twenty Eleven: Style the samp and var elements the same way we style the code and kbd elements. props mfields, fixes #17842.

comment:8 SergeyBiryukov13 months ago

  • Milestone changed from Awaiting Review to 3.4
Note: See TracTickets for help on using tickets.