Skip to content

Prevent Admin Bar Items From Wrapping if There's More Items Than the Available Adminbar Width#352

Closed
KZeni wants to merge 1 commit intoWordPress:masterfrom
KZeni:patch-3
Closed

Prevent Admin Bar Items From Wrapping if There's More Items Than the Available Adminbar Width#352
KZeni wants to merge 1 commit intoWordPress:masterfrom
KZeni:patch-3

Conversation

@KZeni
Copy link

@KZeni KZeni commented Jun 23, 2020

Trac ticket: https://core.trac.wordpress.org/ticket/44438 (has been open for some time without being really followed up on so hopefully this PR breathes some life back into this.)

The Trac ticket has a bunch of additional details regarding the issue being resolved here, screenshots, etc.

Code here is from version 1.0.3 of https://wordpress.org/plugins/admin-bar-wrap-fix/ (June 22, 2020) per that plugin ideally not being necessary to prevent issues that this resolves (with WordPress just preventing it from the start.) Somewhat of a "patch plugin" equivalent to a "feature plugin" WordPress has for forthcoming features, and it might be good to include it in the WP core at some point (if not now).

Edit: Also see https://core.trac.wordpress.org/ticket/28983 as this ticket may take precedent as it's older than my ticket mentioned above while discussing the same issue (though the actual method/implementation to proceed with is yet to be decided, it seems.)

…Available Adminbar Width

Code from version 1.0.3 of https://wordpress.org/plugins/admin-bar-wrap-fix/ per that plugin ideally not being necessary to prevent issues that this resolves (with WordPress just preventing it from the start.)
@KZeni
Copy link
Author

KZeni commented Mar 9, 2021

See #1082 as the trac ticket this was attached to (https://core.trac.wordpress.org/ticket/44438) was seen as a duplicate of https://core.trac.wordpress.org/ticket/28983. As such, the PR has been moved over to the active ticket involved here.

@KZeni KZeni closed this Mar 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments