From d5bbc8d68722d1464fa198cc04339cc0b35ecbf8 Mon Sep 17 00:00:00 2001
From: Liam Brenner <liam.brenner@gmail.com>
Date: Mon, 4 Apr 2016 13:37:34 +1000
Subject: [PATCH] Added titles to action buttons, made them display in the
 dropdown menu

---
 .../pages/listing/_button_with_dropdown.html    |  2 +-
 wagtail/wagtailadmin/wagtail_hooks.py           | 17 +++++++++++------
 2 files changed, 12 insertions(+), 7 deletions(-)

diff --git a/wagtail/wagtailadmin/templates/wagtailadmin/pages/listing/_button_with_dropdown.html b/wagtail/wagtailadmin/templates/wagtailadmin/pages/listing/_button_with_dropdown.html
index 825ec282a3..2fcfd805a5 100644
--- a/wagtail/wagtailadmin/templates/wagtailadmin/pages/listing/_button_with_dropdown.html
+++ b/wagtail/wagtailadmin/templates/wagtailadmin/pages/listing/_button_with_dropdown.html
@@ -7,7 +7,7 @@
         <ul role="menu" class="c-dropdown__menu  u-toggle  u-arrow u-arrow--tl u-background">
         {% for button in buttons %}
             <li class="c-dropdown__item ">
-                <a href="{{ button.url }}" class="u-link is-live {{ button.classes|join:' ' }}">
+                <a href="{{ button.url }}" title="{{ button.attrs.title }}" class="u-link is-live {{ button.classes|join:' ' }}">
                     {{ button.label }}
                 </a>
             </li>
diff --git a/wagtail/wagtailadmin/wagtail_hooks.py b/wagtail/wagtailadmin/wagtail_hooks.py
index 87e36c54b2..76cdc6cfc1 100644
--- a/wagtail/wagtailadmin/wagtail_hooks.py
+++ b/wagtail/wagtailadmin/wagtail_hooks.py
@@ -69,7 +69,7 @@ def page_listing_buttons(page, page_perms, is_parent=False):
         yield PageListingButton(_('Draft'), reverse('wagtailadmin_pages:view_draft', args=[page.id]),
                                 attrs={'title': _('Preview draft'), 'target': '_blank'}, priority=20)
     if page.live and page.url:
-        yield PageListingButton(_('Live'), page.url, attrs={'target': "_blank"}, priority=30)
+        yield PageListingButton(_('Live'), page.url, attrs={'target': "_blank", 'title': 'View live'}, priority=30)
     if page_perms.can_add_subpage():
         if is_parent:
             yield Button(_('Add child page'), reverse('wagtailadmin_pages:add_subpage', args=[page.id]),
@@ -89,11 +89,16 @@ def page_listing_buttons(page, page_perms, is_parent=False):
 @hooks.register('register_page_listing_more_buttons')
 def page_listing_more_buttons(page, page_perms, is_parent=False):
     if page_perms.can_move():
-        yield Button(_('Move'), reverse('wagtailadmin_pages:move', args=[page.id]), priority=10)
+        yield Button(_('Move'), reverse('wagtailadmin_pages:move', args=[page.id]),
+                     attrs={"title": 'Move this page'}, priority=10)
     if not page.is_root():
-        yield Button(_('Copy'), reverse('wagtailadmin_pages:copy', args=[page.id]), priority=20)
+        yield Button(_('Copy'), reverse('wagtailadmin_pages:copy', args=[page.id]),
+                     attrs={'title': 'Copy this page'}, priority=20)
     if page_perms.can_delete():
-        yield Button(_('Delete'), reverse('wagtailadmin_pages:delete', args=[page.id]), priority=30)
+        yield Button(_('Delete'), reverse('wagtailadmin_pages:delete', args=[page.id]),
+                     attrs={'title': 'Delete this page'}, priority=30)
     if page_perms.can_unpublish():
-        yield Button(_('Unpublish'), reverse('wagtailadmin_pages:unpublish', args=[page.id]), priority=40)
-    yield Button(_('Revisions'), reverse('wagtailadmin_pages:revisions_index', args=[page.id]), priority=50)
+        yield Button(_('Unpublish'), reverse('wagtailadmin_pages:unpublish', args=[page.id]),
+                     attrs={'title': 'Unpublish this page'}, priority=40)
+    yield Button(_('Revisions'), reverse('wagtailadmin_pages:revisions_index', args=[page.id]),
+                 attrs={'title': _("View this page's revision history")}, priority=50)