Allow nested lists in MediaWiki:Sidebar
Closed, DeclinedPublic
Actions

Assigned To

Authored By

	• bzimport
	Oct 4 2005, 1:02 PM

Description

Author: wikipedia

Description:
It would be nice, if the sidebar could deal with lists that are nested more than
once.

So that something like the following works:

Blockheading
- mainpage|mainpage
- artikel-url|Artikel
  - sub-artikel-url|Sub-Artikel
  - sub-artikel-url|Sub-Artikel
- next-listpoint-url|next-listpoint
Next-Block-Heading
- ...
- ...

Version: 1.5.x
Severity: enhancement
URL: http://207.175.61.212

Details

Reference: bz3597

Revisions and Commits

Unknown Object (Diffusion Commit)

Event Timeline

• bzimport raised the priority of this task from to Lowest.Nov 21 2014, 8:50 PM

• bzimport added a project: MediaWiki-Parser.

• bzimport set Reference to bz3597.

• bzimport added a subscriber: Unknown Object (MLST).

• bzimport created this task.Oct 4 2005, 1:02 PM

rjf wrote:

Here goes, this appears to work but has some bugs,
like it doesn't deal with special characters very well,
and could probebly be improved quite a bit...

It expands the entire sub-branch for menus like:
Main Menu
Main Item2+
-Sub_Item
--Sub Sub Item
-Sub Item2
Main Item3
Main Item4

Uses "-" characters to "indent" items to their depth
Uses "+" characters to indicate where you are currently sitting.

In includes/Skin.php
Replace existing buildSidebar function with this one

function buildSidebar() {
        global $wgOut; // new required by next line
        $whereami = htmlspecialchars( $wgOut->getPageTitle() ); // new

determine what article I am
Notice: Article names in menus with "_" instead of " " won't match later!
Bug: Other special characters in acticle names will cause broken links!

                                                               

$fname = 'SkinTemplate::buildSidebar';
wfProfileIn( $fname );
                                                               

$bar = array();
$lines = explode( "\n", wfMsgForContent( 'sidebar' ) );
$majorlevel = -1; // new
foreach ($lines as $line) {

// This might be better as a case statement

$minorlevel = 0; // new reinit minorlevel each time arround
if (strpos($line, '*') !== 0) continue; // compressed

this line

if (strpos($line, '********') !== 0)   $minorlevel = 6;

// new

if (strpos($line, '*******') !== 0)   $minorlevel = 5;

// new

if (strpos($line, '******') !== 0)   $minorlevel = 4; // new
if (strpos($line, '*****') !== 0)   $minorlevel = 3; // new
if (strpos($line, '****') !== 0)   $minorlevel = 2; // new
if (strpos($line, '***') !== 0) { $minorlevel = 1;

$majorlevel++; } // new

if (strpos($line, '**') !== 0) {
        $line = trim($line, '* ');
        $heading = $line;
} else {
        if (strpos($line, '|') !== false) { // sanity check
                $line = explode( '|' , trim($line, '*

'), 2 );

$link = wfMsgForContent( $line[0] );
if ($link == '-')
        continue;
if (wfNoMsg($line[1], $text =

wfMsg($line[1])))

        $text = $line[1];
if (wfNoMsg($line[0], $link))
        $link = $line[0];
if ( $whereami == $link ) $text = "+" .

$text . "+"; // new modify whereami's link text

if ( $whereami == $link )

$mymajorlevel=$majorlevel; // new set mymajorlevel for export

$bar[$heading][] = array(
        'text' => $text,
        'majorlevel' => $majorlevel, // new
        'minorlevel' => $minorlevel, // new
        'mymajorlevel' => $mymajorlevel,

// new

'href' =>

$this->makeInternalOrExternalUrl( $link ),

'id' => 'n-' . strtr($line[1], '

', '-'),

                                );
                        } else { continue; }
                }
        }
  
        wfProfileOut( $fname );
        return $bar;
}

In skins/MonoBook.php
Change line ?132-134

<?php foreach($cont as $key => $val) { ?>
  <li id="<?php echo htmlspecialchars($val['id']) ?>"><a href="<?php

echo htmlspecialchars($val['href']) ?>"><?php echo
htmlspecialchars($val['text'])?></a></li>

<?php } ?>

To

<?php foreach($cont as $key => $val) { ?><?php $mymajorlevel =

$val['mymajorlevel'] ?><?php } ?>

<?php foreach($cont as $key => $val) { ?>

<?php if($val['minorlevel'] == "1" ) {?><?php $text = "" .