Skip to content
Permalink
Browse files

MDL-57075 message: notification popover see all link

The "see all" link in the notification popover should link to the full
notifications page similar to how the "see all" link in the message
popover links to the messaging page.
  • Loading branch information...
ryanwyllie committed Nov 22, 2016
1 parent cd4a6b8 commit a6a277f06ce39ba5de507924ec68217d325a4ac3
Showing with 9 additions and 6 deletions.
  1. +8 −6 lib/templates/popover_region.mustache
  2. +1 −0 message/output/popup/lib.php
@@ -64,12 +64,14 @@
{{> core/loading }}
</div>
{{$anchor}}
<a class="see-all-link"
href="{{{urls.seeall}}}">
<div class="popover-region-footer-container">
<div class="popover-region-seeall-text">{{#str}} seeall, message {{/str}}</div>
</div>
</a>
{{#urls.seeall}}
<a class="see-all-link"
href="{{{.}}}">
<div class="popover-region-footer-container">
<div class="popover-region-seeall-text">{{#str}} seeall, message {{/str}}</div>
</div>
</a>
{{/urls.seeall}}
{{/anchor}}
</div>
</div>
@@ -61,6 +61,7 @@ function message_popup_render_navbar_output(\renderer_base $renderer) {
$context = [
'userid' => $USER->id,
'urls' => [
'seeall' => (new moodle_url('/message/output/popup/notifications.php'))->out(),
'preferences' => (new moodle_url('/message/notificationpreferences.php', ['userid' => $USER->id]))->out(),
],
];

0 comments on commit a6a277f

Please sign in to comment.
You can’t perform that action at this time.