Skip to content

Commit

Permalink
Merge branch 'MDL-61620_34' of git://github.com/dmonllao/moodle into …
Browse files Browse the repository at this point in the history
…MOODLE_34_STABLE
  • Loading branch information
stronk7 committed Mar 14, 2018
2 parents ad3fe95 + 735da22 commit 81eec77
Show file tree
Hide file tree
Showing 3 changed files with 21 additions and 4 deletions.
17 changes: 16 additions & 1 deletion lib/outputcomponents.php
Original file line number Diff line number Diff line change
Expand Up @@ -4139,6 +4139,12 @@ class action_menu implements renderable, templatable {
*/
public $menutrigger = '';

/**
* Any extra classes for toggling to the secondary menu.
* @var triggerextraclasses
*/
public $triggerextraclasses = '';

/**
* Place the action menu before all other actions.
* @var prioritise
Expand Down Expand Up @@ -4178,8 +4184,16 @@ public function __construct(array $actions = array()) {
}
}

public function set_menu_trigger($trigger) {
/**
* Sets the menu trigger text.
*
* @param string $trigger The text
* @param string $extraclasses Extra classes to style the secondary menu toggle.
* @return null
*/
public function set_menu_trigger($trigger, $extraclasses = '') {
$this->menutrigger = $trigger;
$this->triggerextraclasses = $extraclasses;
}

/**
Expand Down Expand Up @@ -4457,6 +4471,7 @@ public function export_for_template(renderer_base $output) {
$actionicon = $this->actionicon;
if (!empty($this->menutrigger)) {
$primary->menutrigger = $this->menutrigger;
$primary->triggerextraclasses = $this->triggerextraclasses;
} else {
$primary->title = get_string('actions');
$actionicon = new pix_icon('t/edit_menu', '', 'moodle', ['class' => 'iconsmall actionmenu', 'title' => '']);
Expand Down
5 changes: 3 additions & 2 deletions lib/templates/action_menu_trigger.mustache
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,8 @@
{
"instance": "1",
"title": "Trigger me!",
"menutrigger": true
"menutrigger": true,
"triggerextraclasses": ""
}
}}
<a href="#" class="toggle-display {{#menutrigger}}textmenu{{/menutrigger}}" id="action-menu-toggle-{{instance}}" title="{{title}}" role="menuitem">{{{actiontext}}}{{{menutrigger}}}{{#icon}}{{#pix}}{{key}}, {{component}}, {{title}}{{/pix}}{{/icon}}{{#rawicon}}{{{.}}}{{/rawicon}}{{#menutrigger}}<b class="caret"></b>{{/menutrigger}}</a>
<a href="#" class="{{triggerextraclasses}} toggle-display {{#menutrigger}}textmenu{{/menutrigger}}" id="action-menu-toggle-{{instance}}" title="{{title}}" role="menuitem">{{{actiontext}}}{{{menutrigger}}}{{#icon}}{{#pix}}{{key}}, {{component}}, {{title}}{{/pix}}{{/icon}}{{#rawicon}}{{{.}}}{{/rawicon}}{{#menutrigger}}<b class="caret"></b>{{/menutrigger}}</a>
3 changes: 2 additions & 1 deletion theme/boost/templates/core/action_menu_trigger.mustache
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@
"url": "http://example.com/link",
"classes": "icon menu-action",
"instance": "1",
"triggerextraclasses": "",
"attributes": [
{"name": "role", "value": "menuitem" },
{"name": "data-title", "value": "mymoodle,admin" }
Expand Down Expand Up @@ -77,7 +78,7 @@
}
}}
<div class="dropdown d-inline">
<a href="#" class="dropdown-toggle" id="dropdown-{{instance}}" title="{{title}}" role="button" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">{{{actiontext}}}{{{menutrigger}}}{{#icon}}{{#pix}}{{key}},{{component}},{{title}}{{/pix}}{{/icon}}{{#rawicon}}{{{.}}}{{/rawicon}}{{#menutrigger}}<b class="caret"></b>{{/menutrigger}}</a>
<a href="#" class="{{triggerextraclasses}} dropdown-toggle" id="dropdown-{{instance}}" title="{{title}}" role="button" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">{{{actiontext}}}{{{menutrigger}}}{{#icon}}{{#pix}}{{key}},{{component}},{{title}}{{/pix}}{{/icon}}{{#rawicon}}{{{.}}}{{/rawicon}}{{#menutrigger}}<b class="caret"></b>{{/menutrigger}}</a>
{{#secondary}}
<div class="dropdown-menu dropdown-menu-right {{classes}}"{{#attributes}} {{name}}="{{value}}"{{/attributes}}>
{{#items}}{{!
Expand Down

0 comments on commit 81eec77

Please sign in to comment.