Skip to content

[5.x] Navbarextensions improvements#10879

Merged
spalger merged 1 commit into5.xfrom
jasper/backport/9871/5.x
Mar 24, 2017
Merged

[5.x] Navbarextensions improvements#10879
spalger merged 1 commit into5.xfrom
jasper/backport/9871/5.x

Conversation

@elastic-jasper
Copy link
Copy Markdown
Contributor

Navbarextensions improvements

Backports PR #9871

Commit 1:

  • added controller option
  • added $location service to determine on which the user is at currently

Commit 2:
added test for the hideButton method that is dependent on the location path

Commit 3:
removed argument

Commit 4:
use locals object to pass variables to the template's scope

Commit 5:
use map object

Backports PR #9871

**Commit 1:**
- added controller option
- added $location service to determine on which the user is at currently

* Original sha: ddc2eb5
* Authored by Stéphane Campinas <stephane.campinas@gmail.com> on 2017-01-13T17:29:50Z

**Commit 2:**
added test for the hideButton method that is dependent on the location path

* Original sha: f27a535
* Authored by Stéphane Campinas <stephane.campinas@gmail.com> on 2017-01-13T23:43:14Z

**Commit 3:**
removed  argument

* Original sha: c9d5c0c
* Authored by Stéphane Campinas <stephane.campinas@gmail.com> on 2017-01-14T13:44:35Z

**Commit 4:**
use locals object to pass variables to the template's scope

* Original sha: 07b7549
* Authored by Stéphane Campinas <stephane.campinas@gmail.com> on 2017-03-04T21:26:33Z

**Commit 5:**
use map object

* Original sha: f9ee311
* Authored by Stéphane Campinas <stephane.campinas@gmail.com> on 2017-03-04T21:33:48Z
@elastic-jasper elastic-jasper added the backport This PR is a backport of another PR label Mar 24, 2017
@spalger spalger merged commit 45263a4 into 5.x Mar 24, 2017
@spalger spalger deleted the jasper/backport/9871/5.x branch March 24, 2017 02:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants