"Configure" link for dashboard widgets no longer displayed.

[catchmyfame]

The "configure" links for dashboard widgets are now hidden by CSS. The HTML, which is generated automatically by wp_add_dashboard_widget when the $control_callback argument is provided is:

<span class="postbox-title-action"><a href="/wp-admin/index.php?edit=widget#" class="edit-box open-box">Configure</a></span>

The CSS being applied from:

wp-admin/load-styles.php?c=0&dir=ltr&load=dashicons,admin-bar,wp-admin,buttons,wp-auth-check&ver=4.4

Creates the rule online 7518:

.edit-box {
     display: none; 
}

Which hides the anchor element as it has

class="edit-box open-box"

Another issue is that even when the rule is removed, the "configure" link is no longer right justified.

[swissspidy]

The problem is not the display: none, but the incorrect line afterwards: h3:hover .edit-box.

It's now a h2 (introduced in #33557), that's why it doesn't work anymore. We should change that to .hndle (or .handle, see #32017)

[catchmyfame]

After I submitted this ticket I remembered that the configure link wasn't always visible and became visible on hover of the title. Thanks @swissspidy for updating that.

[afercia]

Refreshed patch to make the Configure link appear also when tabbing with the keyboard.
Side note: actually, there are 2 "Configure" link, we should probably completely hide the first one. Some testing would be nice :)

[swissspidy]

After the discussion on Slack, 34987.4.patch​ is an updated patch that also removes the unneeded right: 33px; rule. Successfully tested on Firefox, Chrome, Safari, Opera, Internet Explorer with both JS enabled and disabled. Tested with RTL and LTR.

What the patch does:

• Shows the Configure link again on :hover
• Improves accessibility by also showing the link on :focus (using opacity instead of display: none)
• Hides the second Configure link that is inside the toggle button
• Fixes vertical alignment by not using absolute positioning but floats instead

[swissspidy]

Configure link in WordPress 4.3

[swissspidy]

Configure link with patch applied

[afercia]

Tested on a few more Dashboard widgets, on hover and focus. Looks good to me.
About the doubled "Configure" link we should probably open a separate ticket to avoid to output redundant markup.

[tywayne]

The vertically alignment of the Configure link doesn't look quite right to me - seems to sit lower than the other elements. Adjusting the line height on the #dashboard-widgets .hndle .postbox-title-action selector should fix it. Maybe at line-height: 1.2?

[swissspidy]

Replying to tywayne:

The vertically alignment of the Configure link doesn't look quite right to me - seems to sit lower than the other elements. Adjusting the line height on the #dashboard-widgets .hndle .postbox-title-action selector should fix it. Maybe at line-height: 1.2?

You're right, after looking at it again I realized that the alignment is not ideal.

In 34987.5.patch​ I decided to not change absolute positioning to floats. So basically this patch just re-introduces the Configure link.

[afercia]

For reference, this is how the (mis)alignment looked like in WordPress 4.2. Also, the float wasn't a bad idea since a very long Widget title might have some issues if the link is positioned absolutely (see the screenshot second part):

[swissspidy]

@afercia Good point!

I guess under that circumstances 34987.4.patch​ makes more sense with a line-height of 1.2, as suggested by @tywayne.

Here's how a long title looks like when using floats:

Short title, using adjusted line-height:

[swissspidy]

Pending final review on 34987.6.patch​

[afercia]

Looks good to me. The only thing to be aware of is that opacity won't work in IE 8 (still supported in the admin) and the link would need ​proprietary filters and to have hasLayout. The link is always visible in IE 8 and I'm perfectly fine with that.

[swissspidy]

Update against trunk

[swissspidy]

In 35896:

Admin: After [35128], make the 'Configure' link work again for dashboard widgets.

See #34987.

[SergeyBiryukov]

#35162 was marked as a duplicate.

[SergeyBiryukov]

#35202 was marked as a duplicate.

[dd32]

34987.4-4-branch.diff​ is an attempt at inlining the required CSS here to work around #35229 - assuming that no other changes in 4.4.1 result in wp-admin.min.css changes.

[dd32]

In 36153:

Admin: After [35128], make the 'Configure' link work again for dashboard widgets.

This merges the CSS changes from [35896] into an inline function in formatting.php to avoid a wp-admin.min.css rebuild.

Fixes #34987.