Changeset 43246

Timestamp:

05/12/2018 05:28:30 PM (6 years ago)

Author:

iandunn

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.
Fixes #44045.

Location:

trunk/src

Files:

• wp-admin/includes/class-wp-internal-pointers.php (modified) (2 diffs)
• wp-includes/css/wp-pointer.css (modified) (1 diff)

trunk/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 );

trunk/src/wp-includes/css/wp-pointer.css

@@ -208 +208 @@
 }
 
+.wp-pointer.arrow-bottom .wp-pointer-content {
+    margin-bottom: -45px;
+}
+
+.wp-pointer.arrow-bottom .wp-pointer-arrow {
+    top: 100%;
+    margin-top: -30px;
+}
+
 /* Disable pointers at responsive sizes */
 @media screen and ( max-width: 782px ) {