From 33a0f309ff70c408bafa3912b86801a1856db745 Mon Sep 17 00:00:00 2001
From: Endilie Yacop Sucipto <endiliey@gmail.com>
Date: Sun, 10 Jun 2018 17:56:49 +0800
Subject: [PATCH] Fix HeaderNav linking to versions page to depends on cleanUrl
 (#748)

---
 lib/core/nav/HeaderNav.js | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/lib/core/nav/HeaderNav.js b/lib/core/nav/HeaderNav.js
index 52a4ff5fdb..97bf6187e4 100644
--- a/lib/core/nav/HeaderNav.js
+++ b/lib/core/nav/HeaderNav.js
@@ -21,6 +21,7 @@ const readMetadata = require('../../server/readMetadata.js');
 readMetadata.generateMetadataDocs();
 const Metadata = require('../metadata.js');
 const utils = require('../utils.js');
+const extension = siteConfig.cleanUrl ? '' : '.html';
 
 // language dropdown nav item for when translations are enabled
 class LanguageDropDown extends React.Component {
@@ -200,7 +201,6 @@ class HeaderNav extends React.Component {
     } else if (link.page) {
       // set link to page with current page's language if appropriate
       const language = this.props.language || '';
-      const extension = siteConfig.cleanUrl ? '' : '.html';
       if (fs.existsSync(CWD + '/pages/en/' + link.page + '.js')) {
         href =
           siteConfig.baseUrl +
@@ -243,8 +243,8 @@ class HeaderNav extends React.Component {
     const versionsLink =
       this.props.baseUrl +
       (env.translation.enabled
-        ? this.props.language + '/versions.html'
-        : 'versions.html');
+        ? this.props.language + '/versions' + extension
+        : 'versions' + extension);
     return (
       <div className="fixedHeaderContainer">
         <div className="headerWrapper wrapper">