From 5ded18d5a81de913f2aab36ebd1f21a79d9ae974 Mon Sep 17 00:00:00 2001 From: Andrew Nicols Date: Wed, 6 Sep 2017 10:23:48 +0800 Subject: [PATCH] MDL-60031 admin: Move risk longtext to pix title --- admin/roles/classes/capability_table_with_risks.php | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/admin/roles/classes/capability_table_with_risks.php b/admin/roles/classes/capability_table_with_risks.php index b62ad19c409..db7e16bc83f 100644 --- a/admin/roles/classes/capability_table_with_risks.php +++ b/admin/roles/classes/capability_table_with_risks.php @@ -183,9 +183,14 @@ abstract class core_role_capability_table_with_risks extends core_role_capabilit public function get_risk_icon($type) { global $OUTPUT; - $text = $OUTPUT->pix_icon('i/' . str_replace('risk', 'risk_', $type), get_string($type . 'short', 'admin')); + $alt = get_string("{$type}short", "admin"); + $title = get_string($type, "admin"); + + $text = $OUTPUT->pix_icon('i/' . str_replace('risk', 'risk_', $type), $alt, 'moodle', [ + 'title' => $title, + ]); $action = new popup_action('click', $this->risksurl, 'docspopup'); - $riskicon = $OUTPUT->action_link($this->risksurl, $text, $action, array('title'=>get_string($type, 'admin'))); + $riskicon = $OUTPUT->action_link($this->risksurl, $text, $action); return $riskicon; }