[Toaster] [PATCH] toaster: Indicate active navigation element
Belen Barros Pena
belen.barros.pena at linux.intel.com
Tue Sep 6 03:13:10 PDT 2016
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]
Signed-off-by: Belen Barros Pena <belen.barros.pena at linux.intel.com>
---
bitbake/lib/toaster/toastergui/templates/basebuildpage.html | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/bitbake/lib/toaster/toastergui/templates/basebuildpage.html b/bitbake/lib/toaster/toastergui/templates/basebuildpage.html
index 50658e0..01d3117 100644
--- a/bitbake/lib/toaster/toastergui/templates/basebuildpage.html
+++ b/bitbake/lib/toaster/toastergui/templates/basebuildpage.html
@@ -38,10 +38,13 @@
$(this).parent().addClass("active");
} else {
/* Special case the configvar as this is part of configuration
- * page but is a separate url
+ * page but is a separate url, and the direct links to errors
+ * and warnings, which are part of the build dashboard
*/
if (window.location.pathname === configVarUrl){
$("#menu-configuration").addClass("active");
+ } else if (currentUrl.indexOf('error') > 1 || currentUrl.indexOf('warning') > 1){
+ $("#menu-dashboard").addClass("active");
} else {
$(this).parent().removeClass("active");
}
@@ -57,7 +60,7 @@
<div class="row">
<div id="nav" class="col-md-2">
<ul class="nav nav-pills nav-stacked" id="build-menu">
- <li
+ <li id="menu-dashboard"
{% if request.resolver_match.url_name == 'builddashboard' %}
class="active"
{% endif %} >
--
1.9.1
More information about the toaster
mailing list