Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#20837 closed defect (bug) (worksforme)

About WordPress: Wrap class/function names into code tags

Reported by: ocean90's profile ocean90 Owned by: nacin's profile nacin
Milestone: Priority: low
Severity: minor Version: 3.4
Component: Help/About Keywords: has-patch rtl-feedback
Focuses: Cc:

Description

Just needs some magic from Nacin for GlotPress.

Attachments (4)

20837.patch (721 bytes) - added by ocean90 11 years ago.
20837.2.patch (1.1 KB) - added by ocean90 11 years ago.
under-the-hood-code.png (78.2 KB) - added by ryan 11 years ago.
20837.3.patch (1.6 KB) - added by SergeyBiryukov 11 years ago.

Download all attachments as: .zip

Change History (16)

@ocean90
11 years ago

#1 @ocean90
11 years ago

  • Owner set to nacin
  • Status changed from new to assigned

#2 @nacin
11 years ago

<code>WP_Query</code> also.

@ocean90
11 years ago

#3 @ryan
11 years ago

code in the h4 doesn't look quite nice, to me.

#4 @ocean90
11 years ago

In 3.3:

http://f.cl.ly/items/310b1X3U133c2F3o3K0q/Bildschirmfoto%202012-06-04%20um%2022.40.55.png

#5 follow-up: @nacin
11 years ago

  • Keywords rtl-feedback added

We just did this in 3.3: <h4 style="direction:ltr">is_main_query()</h4>

We can't do that so easily with the use of "API" there.

Perhaps we can just "unstyle" <code> inside an <h4>. We're mostly worried about the semantic value as well as RTL.

#6 @azaozz
11 years ago

Looks like the compromise is to use a css class that would keep certain text blocks like headers in LTR. That gives some flexibility. The other option would be to style places we don't want monotype fonts inside "code":

h1 code,
h2 code,
h3 code,
h4 code,
h5 code,
h6 code {
  font-family: sans-serif;
}
Last edited 11 years ago by azaozz (previous) (diff)

#7 in reply to: ↑ 5 @SergeyBiryukov
11 years ago

Replying to nacin:

We just did this in 3.3: <h4 style="direction:ltr">is_main_query()</h4>

Also <h4><?php _e( 'WP_Screen API' ); ?></h4>.

#8 @ryan
11 years ago

Or say "Faster Queries" and avoid tedium no one will appreciate.

#9 @nacin
11 years ago

  • Milestone changed from 3.4 to Awaiting Review
  • Priority changed from normal to low

Going to look into this, but not a priority if we don't get to it for 3.4 final.

#10 @SergeyBiryukov
11 years ago

Maybe "<h4>Faster Post Queries</h4>", then "<code>WP_Query</code> has been optimized..."

#11 @ocean90
11 years ago

  • Milestone changed from Awaiting Review to 3.5
  • Severity changed from normal to minor

We should look at it again when we do a 3.5 about page.

#12 @nacin
11 years ago

  • Milestone 3.5 deleted
  • Resolution set to worksforme
  • Status changed from assigned to closed

Closing. We'll get it to our liking when we build the about page in November.

Note: See TracTickets for help on using tickets.