Fix navbar's active class.

Use a new variable based on `req.originalUrl`.
jsdelivr · May 26, 2018 · beb59b8 · beb59b8
1 parent 90cf9f4
commit beb59b8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/routes/appendLocals.js b/routes/appendLocals.js
@@ -19,6 +19,7 @@ function appendLocals(req, res) {
     res.locals.canonicalUrl = `${req.config.siteurl}${req.path}`;
 
     res.locals.siteUrl = `${proto}://${req.hostname}`;
+    res.locals.pageUrl = req.originalUrl;
 
     res.locals.theme = req.query.theme < totalThemes ?
         req.query.theme :

diff --git a/views/_mixins.pug b/views/_mixins.pug
@@ -6,7 +6,7 @@ mixin menuItem(name, url)
             classes.push(attributes['extra-class']);
         }
 
-        if (title === name) {
+        if (pageUrl === url) {
             classes.push('active');
         }