WordPress.org
Get WordPress

Make WordPress Core

Opened 14 years ago

Closed 14 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 Owned by: lancewillett
Priority: normal Milestone: 3.6
Component: Bundled Theme Version:
Severity: normal Keywords: has-patch
Cc: Focuses:

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 14 years ago.
23549.2.diff​ (2.9 KB ) - added by lancewillett 14 years ago.

Download all attachments as: .zip

Change History (8)

@sabreuse
14 years ago

#1 @sabreuse
14 years ago

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

#2 @sabreuse
14 years ago

  • Keywords has-patch added; needs-patch removed

#3 @kwight
14 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.

@lancewillett
14 years ago

#4 @lancewillett
14 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
14 years ago

  • Cc mdhansen@… added

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

#6 @lancewillett
14 years ago

  • Owner set to lancewillett
  • Resolution → fixed
  • Status new → 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.