Twenty Thirteen: some stylesheet cleanup

[gorgoglionemeister]

In style.css:

line 863: rgba(247, 245, 231, 1) is the same as #f7f5e7 (am I wrong?), so we should use the second notation.

line 865: color: #fff; can be removed because it is defined below (line 884) and does nothing.

line 867: float: left; has no effect since we are using position: absolute; and must be removed.

line 871: top: 45px; is not necessary since it is the used value for top (every li element is 45 pixels high).

[alexvorn2]

I think rgba(247, 245, 231, 1) is better because of opacity.

[gorgoglionemeister]

Replying to alexvorn2:

I think rgba(247, 245, 231, 1) is better because of opacity.

rgba(247, 245, 231, 1) is the same as rgb(247, 245, 231) and, according to the ​CSS Coding Standards, we must use the hex instead of the functional notation for colors.

[obenland]

It indeed looks like the styles can be safely removed. Still need to make IE tests, but if it breaks there (which I doubt), these styles would belong in /css/ie.css anyway.

gorgoglionemeister, could you refresh your patch and make the path relative to WordPress' root?

[gorgoglionemeister]

Replying to obenland:

It indeed looks like the styles can be safely removed. Still need to make IE tests, but if it breaks there (which I doubt), these styles would belong in /css/ie.css anyway.

gorgoglionemeister, could you refresh your patch and make the path relative to WordPress' root?

Yes, sure. Sorry.

[lancewillett]

Yes, the changes do break things in IE8 and 7, the dropdown is off by a few pixels. But we can remove the following from the ie.css file:

.nav-menu .sub-menu,
.nav-menu .children {
	left: 0;
}

[lancewillett]

In 24542:

Twenty Thirteen: minor CSS cleanup for navigation, props gorgoglionemeister. Closes #24651.