diff options
author | Belen Barros Pena <belen.barros.pena@linux.intel.com> | 2016-09-18 19:33:14 -0700 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2016-09-21 21:58:05 +0100 |
commit | 07aa966b5b7017458345bde6dbc454e1d1c25ec1 (patch) | |
tree | 5fac81a753b1e0f6608f8faf182483f77df4bbc0 /bitbake/lib/toaster/toastergui | |
parent | 67ce1ce3cbfa1977761ad668bdd548e7389211c3 (diff) | |
download | poky-07aa966b5b7017458345bde6dbc454e1d1c25ec1.tar.gz |
bitbake: toaster: Indicate active navigation element
The left navigation in the build history pages is not showing the active
item when you navigate directly to the errors or warnings information in
the build summary. Add a special case to make sure the "build summary"
item is highlighted.
[YOCTO #9864]
(Bitbake rev: f236d9ca28e45a270f50bb3edcd466b1bc8d2960)
Signed-off-by: Belen Barros Pena <belen.barros.pena@linux.intel.com>
Signed-off-by: bavery <brian.avery@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'bitbake/lib/toaster/toastergui')
-rw-r--r-- | bitbake/lib/toaster/toastergui/templates/basebuildpage.html | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/bitbake/lib/toaster/toastergui/templates/basebuildpage.html b/bitbake/lib/toaster/toastergui/templates/basebuildpage.html index 50658e0c51..01d3117e1e 100644 --- a/bitbake/lib/toaster/toastergui/templates/basebuildpage.html +++ b/bitbake/lib/toaster/toastergui/templates/basebuildpage.html | |||
@@ -38,10 +38,13 @@ | |||
38 | $(this).parent().addClass("active"); | 38 | $(this).parent().addClass("active"); |
39 | } else { | 39 | } else { |
40 | /* Special case the configvar as this is part of configuration | 40 | /* Special case the configvar as this is part of configuration |
41 | * page but is a separate url | 41 | * page but is a separate url, and the direct links to errors |
42 | * and warnings, which are part of the build dashboard | ||
42 | */ | 43 | */ |
43 | if (window.location.pathname === configVarUrl){ | 44 | if (window.location.pathname === configVarUrl){ |
44 | $("#menu-configuration").addClass("active"); | 45 | $("#menu-configuration").addClass("active"); |
46 | } else if (currentUrl.indexOf('error') > 1 || currentUrl.indexOf('warning') > 1){ | ||
47 | $("#menu-dashboard").addClass("active"); | ||
45 | } else { | 48 | } else { |
46 | $(this).parent().removeClass("active"); | 49 | $(this).parent().removeClass("active"); |
47 | } | 50 | } |
@@ -57,7 +60,7 @@ | |||
57 | <div class="row"> | 60 | <div class="row"> |
58 | <div id="nav" class="col-md-2"> | 61 | <div id="nav" class="col-md-2"> |
59 | <ul class="nav nav-pills nav-stacked" id="build-menu"> | 62 | <ul class="nav nav-pills nav-stacked" id="build-menu"> |
60 | <li | 63 | <li id="menu-dashboard" |
61 | {% if request.resolver_match.url_name == 'builddashboard' %} | 64 | {% if request.resolver_match.url_name == 'builddashboard' %} |
62 | class="active" | 65 | class="active" |
63 | {% endif %} > | 66 | {% endif %} > |