Changeset 43253 for branches/4.9/src/wp-admin/includes/class-wp-internal-pointers.php

Timestamp:

05/14/2018 02:00:01 PM (8 years ago)

Author:

SergeyBiryukov

Message:

Privacy: Reposition pointer to ensure dismiss link is always visible.

r43158 introduced a new admin pointer for the privacy tools added in 4.9.6. With the previous positioning, though, sometimes the Dismiss link would be fixed off screen, making it impossible for the user to dismiss the pointer. This happened when there were enough extra menu items, or when the viewport height was short enough.

This commit repositions the pointer to work around that problem. One down side of this workaround is that the arrow will not always be positioned next to the Tools menu, where it should be. That's an acceptable compromise given the current time constraints, though. A long term solution would be to make WP_Pointer robust enough to handle this use case.

Props imath, audrasjb, desrosj.
Merges [43246] to the 4.9 branch.
Fixes #44045.

Location:

branches/4.9

Files:

• . (modified) (1 prop)
• src/wp-admin/includes/class-wp-internal-pointers.php (modified) (2 diffs)

branches/4.9/src/wp-admin/includes/class-wp-internal-pointers.php

@@ -171 +171 @@
                         $position = array(
                                 'edge'  => 'right',
-                                'align' => 'left',
+                                'align' => 'bottom',
                         );
                 } else {
                         $position = array(
                                 'edge'  => 'left',
-                                'align' => 'right',
+                                'align' => 'bottom',
                         );
                 }
@@ -183 +183 @@
                         'content'  => $content,
                         'position' => $position,
+                        'pointerClass' => 'wp-pointer arrow-bottom',
+                        'pointerWidth' => 420,
                 );
                 self::print_js( 'wp496_privacy', '#menu-tools', $js_args );