Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
wants to merge 1 commit into from

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