Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#22468 closed defect (bug) (fixed)

Twenty Twelve: arrow in reply text renders poorly

Reported by: miqrogroove's profile miqrogroove Owned by: lancewillett's profile lancewillett
Milestone: 3.5 Priority: normal
Severity: normal Version: 3.5
Component: Bundled Theme Keywords: has-patch
Focuses: Cc:

Description

The "Reply" links in the Twenty Twelve theme all look like they are followed by some random garbage characters. They look more like arrows in Twenty Eleven, and they don't appear at all in my own theme. I'll attach some screen shots.

Attachments (7)

twenty-twelve-chrome.png (13.9 KB) - added by miqrogroove 12 years ago.
WinXP Screen Shots
twenty-twelve-firefox.png (18.8 KB) - added by miqrogroove 12 years ago.
twenty-twelve-ie8.png (13.3 KB) - added by miqrogroove 12 years ago.
twenty-twelve-opera.png (13.5 KB) - added by miqrogroove 12 years ago.
twenty-eleven-chrome.png (13.3 KB) - added by miqrogroove 12 years ago.
At least it looks like an arrow here.
image.jpg (231.0 KB) - added by miqrogroove 12 years ago.
Looks weird on iPad.
miqro-22468.diff (800 bytes) - added by miqrogroove 12 years ago.

Download all attachments as: .zip

Change History (23)

@miqrogroove
12 years ago

WinXP Screen Shots

#1 @nacin
12 years ago

Are you using Windows XP font smoothing? Or rather, why aren't you? (And, why XP?)

#2 @miqrogroove
12 years ago

The &darr code can be found in the twentytwelve_comment() callback where it uses comment_reply_link(). Currently, http://core.trac.wordpress.org/browser/trunk/wp-content/themes/twentytwelve/functions.php#L311

I am using XP because I like it, and my computer is ancient. Font smoothing makes the text blurry and annoys me.

@miqrogroove
12 years ago

At least it looks like an arrow here.

@miqrogroove
12 years ago

Looks weird on iPad.

#3 @kovshenin
12 years ago

  • Cc kovshenin added

#4 @lancewillett
12 years ago

I vote "wontfix" on this, works fine in the vast majority of browsers and OS versions.

#5 @helenyhou
12 years ago

I think it's more that the underline combined with the down arrow, which descends below the baseline, looks a little weird, even in more modern combinations.

Last edited 12 years ago by helenyhou (previous) (diff)

#6 @miqrogroove
12 years ago

helenhou are there other browsers/OS examples you could screen shot?

#7 @helenyhou
12 years ago

I'm on OSX. It looks like it does in your iPad screenshot, except not HiDPI.

#8 @miqrogroove
12 years ago

On OSX does it look the same in all browsers? Maybe 1 or 2 screen shots would help.

#9 @lancewillett
12 years ago

  • Cc drewstrojny added

@drewstrojny What do you think about removing the underline or arrow here?

#10 @drewstrojny
12 years ago

I'd say remove the arrow if it's causing issues. The basic link works just fine.

#11 @miqrogroove
12 years ago

Just a thought: comment_reply_link() offers an "after" argument. Move the arrow after the link, as seen here on Trac?

#12 @miqrogroove
12 years ago

Patch attached for clarity. Uncertain if arrow requires translation though.

#13 @lancewillett
12 years ago

  • Milestone changed from Awaiting Review to 3.5

Nope, since it's a down arrow it doesn't need to be translated. (Left or right would for RTL languages.)

#14 @miqrogroove
12 years ago

  • Keywords has-patch added

That should do it then.

#15 @lancewillett
12 years ago

  • Summary changed from Twenty Twelve's &darr Renders Poorly to Twenty Twelve: arrow in reply text renders poorly

#16 @lancewillett
12 years ago

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

In 22614:

Twenty Twelve: improve arrow display in comment reply text, props miqrogroove. Also consistent styles for edit link to match reply link in color and size. Closes #22468.

Note: See TracTickets for help on using tickets.