The path of the background icon for the "dismiss" link of WP Pointer is broken

[jamalorg]

The URL of the background icon (x) for the "dismiss" link in WP Pointer returns 404 error because it calls the icon from ../../wp-admin/images/xit.gif.

Logically the path is not wrong. The stylesheet of WP Pointer is located at wp-includes/css/wp-pointer.css, and calling xit.gif path from two directories up is the correct way to do it, but since all admin styles were loaded via wp-admin/load-styles.php the actual path relatively changes.

[duck_]

This must vary in different browsers as I don't get a 404 in Chromium or Firefox. Or maybe it depends on where your WordPress files are, i.e. installed in a subdirectory. What's your setup to reproduce this?

Obviously we cannot just change it to work for load-styles.php because then in would break when SCRIPT_DEBUG == true. In fact it breaks for me with SCRIPT_DEBUG false and the CSS changed to reference ../images/xit.gif (as used by the colors CSS files).

[jamalorg]

Replying to duck_:

This must vary in different browsers as I don't get a 404 in Chromium or Firefox. Or maybe it depends on where your WordPress files are, i.e. installed in a subdirectory. What's your setup to reproduce this?

It works when WordPress is installed in the root directory (e.g. site.com/), but 404 is returned when WP is installed in sub-directory.

What about dropping a copy of xit.gif icon in wp-includes/images/ and updating the icon URL to this path?

[duck_]

Replying to jamalorg:

It works when WordPress is installed in the root directory (e.g. site.com/), but 404 is returned when WP is installed in sub-directory.

As I suspected. Thanks.

[nacin]

Yeah, I've pointed out elsewhere that we can't do this. xit.gif needs to be copied to wp-includes/images/xit.gif. Then ../images/xit.gif will always work.

[nacin]

We could also add mapping to load-styles.php, but I'd rather not, since the pointers code is designed to hypothetically work on the frontend, and therefore should not rely on admin-only assets (which may be unreachable).

[ryan]

Fine with putting a copy in wp-includes.

[nacin]

In [19557]:

Copy xit.gif to wp-includes/images for work with pointers CSS. fixes #19441.