You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@orc.apache.org by do...@apache.org on 2022/08/20 01:16:04 UTC

[orc] branch main updated: ORC-1255: Fix ORC website navbar highlight

This is an automated email from the ASF dual-hosted git repository.

dongjoon pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/orc.git


The following commit(s) were added to refs/heads/main by this push:
     new 9e2fa879c ORC-1255: Fix ORC website navbar highlight
9e2fa879c is described below

commit 9e2fa879c6ff259a8152948b8c9e11157638d713
Author: William Hyun <wi...@apache.org>
AuthorDate: Fri Aug 19 18:15:56 2022 -0700

    ORC-1255: Fix ORC website navbar highlight
    
    ### What changes were proposed in this pull request?
    This PR aims to fix the ORC website navbar highlight issue.
    
    ### Why are the changes needed?
    **BEFORE**
    <img width="717" alt="Screen Shot 2022-08-19 at 5 53 16 PM" src="https://user-images.githubusercontent.com/62487364/185723100-572ed69a-3400-4d53-a5b1-894fbeb68860.png">
    
    **AFTER**
    <img width="717" alt="Screen Shot 2022-08-19 at 5 53 23 PM" src="https://user-images.githubusercontent.com/62487364/185723102-a138eb61-2e21-4792-a419-e5ffff58e1ff.png">
    
    ### How was this patch tested?
    Manually build and check.
    
    Closes #1226 from williamhyun/highlight.
    
    Authored-by: William Hyun <wi...@apache.org>
    Signed-off-by: Dongjoon Hyun <do...@apache.org>
---
 site/_includes/primary-nav-items.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/site/_includes/primary-nav-items.html b/site/_includes/primary-nav-items.html
index c69578eca..b5cb0c4d8 100644
--- a/site/_includes/primary-nav-items.html
+++ b/site/_includes/primary-nav-items.html
@@ -2,7 +2,7 @@
   <li class="{% if page.overview %}current{% endif %}">
     <a href="/">Home</a>
   </li>
-  <li class="{% if page.url contains '/releases/' %}current{% endif %}">
+  <li class="{% if page.url == '/releases/' %}current{% endif %}">
     <a href="/releases/"><span class="show-on-mobiles">Rel</span>
                          <span class="hide-on-mobiles">Releases</span></a>
   </li>