Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#23549 closed enhancement (fixed)

Twenty Thirteen: header area with no site title or tagline should still be clickable (link to home)

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

Description

Header area should probably link to home_url() even if no title or tagline.

Attachments (2)

23549.diff (1.1 KB) - added by sabreuse 12 years ago.
23549.2.diff (2.9 KB) - added by lancewillett 12 years ago.

Download all attachments as: .zip

Change History (8)

@sabreuse
12 years ago

#1 @sabreuse
12 years ago

23549.diff moves the home_url link to include the entire hgroup. Clicky goodness!

#2 @sabreuse
12 years ago

  • Keywords has-patch added; needs-patch removed

#3 @kwight
12 years ago

That's what I was thinking too – you beat me to it.

Patch works for me.

It's odd though, seeing the :hover underline when mousing over anywhere in the header. Hm.

#4 @lancewillett
12 years ago

One thing that would break with the first patch would be any CSS selectors that expected the anchor around the site title. Things like changing Header Text Color.

2nd patch addresses that as well as the hover style kwight mentioned.

#5 @MikeHansenMe
12 years ago

  • Cc mdhansen@… added

Tested patch .2 and it works as expected. Also tested the customizer, it also works as expected.

#6 @lancewillett
12 years ago

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

In 23492:

Twenty Thirteen: header area with no site title or tagline should still be clickable to home. Fixes #23549.

Fix also removes several !important flags that aren't necessary, relating to header text color.

Note: See TracTickets for help on using tickets.