Toolbar JS with jQuery doesn't match non-jQuery behavior with hashes (e.g. read more link)

[helenyhou]

In /wp-includes/js/admin-bar.js, we have a block that uses jQuery if available and then falls back to another block of plain JS if not. The JS version does the following to help prevent the toolbar from covering up content when a hash is present in the URL (w = window):

if ( w.location.hash )
	w.scrollBy(0,-32);

The jQuery version does not seem to do the same, but it should. Patch is forthcoming.

[helenyhou]

Related: #18299

[tlovett1]

Patches problem in wp-includes/js/admin-bar.js

[tlovett1]

This version has the if statement within the jquery exists if

[helenyhou]

patch-more-toolbar-offset-v1.diff​ works for me. patch-more-toolbar-offset.diff​ appears more ideal on first glance (moving it out of the if/else block since it's used in both places) but doesn't actually work, I suppose because it no longer waits until load.

[azaozz]

In 22728:

Toolbar: sync jQuery and non-jQuery JS behaviour when there's a hash in the url, props tlovett1, fixes #22520