Ian Dickinson wrote:
>> What do you think?
> 
> Thanks for looking into this. I've been dealing with some difficult
> family issues, together with heavy day-job project deadlines, so my
> bandwidth for Jena has been rather severely curtailed lately. I'll leave
> it on my todo list to take a look at what you've done, but for now if it
> fixes the presenting problem then that's good enough for me.

Hi Ian
re: family, I did not know, sorry. Forget about Jena website (and don't
ready the following on this email!

Paolo



PS:
For others (or myself in future), in case you need to do something similar
or a similar issue reappears in future.

I think the '_' or '-' appear because of the markdown filter, but we have
no control on it. Therefore we need to adapt and go around the issue via
jena-navigation.js.

Here is how I got around it (it's only a few places in jena-nativation.js...
but not being familiar with it, it took me sometime to find the right ones).

This also taught the lesson that with SVN + buildbots combination changes
are not always reversible.


svn diff -r 1175246:1303991 content/jena/js/jena-navigation.js
Index: content/jena/js/jena-navigation.js
===================================================================
--- content/jena/js/jena-navigation.js  (revision 1175246)
+++ content/jena/js/jena-navigation.js  (revision 1303991)
@@ -38,7 +38,7 @@
    * @param {Object} fn
    */
   var findTopMenu = function( fileName, fn ) {
-    var expr = (fileName == 'about_jena') ? "#home_menu" : "#topmenu a[href*='"
+ fileName + "']";
+    var expr = (fileName == 'about_jena') ? "#home-menu" : "#topmenu a[href*='"
+ fileName + "']";
     $(expr).first().each( function( n ) {
         fn( this );
     } );
@@ -50,7 +50,7 @@
   var getLocation = function() {
     var url = checkForDefaultPage();
     if (url.match( /\/jena\/index.html$/ )) {
-        currentSection.fileName = "about_jena";
+        currentSection.fileName = "about-jena";
         currentPage.fileName = "index.html";
     }
     else {
@@ -89,10 +89,10 @@
   var setSideNavigationVisibility = function() {
     $("#navigation>h1").hide();
     $("#navigation>ul").hide();
-    showMenu( "#quick_links" );
-    showMenu( "#asf_links" );
+    showMenu( "#quick-links" );
+    showMenu( "#asf-links" );

-    var currentTopicMenu = "#" + currentSection.fileName;
+    var currentTopicMenu = "#" + currentSection.fileName.replace( /\_/g, "-" );
     showMenu( currentTopicMenu );

     // make the current topic link styled as 'current'

Reply via email to