Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

ModuleCustomnav sollte CSS-Klasse "trail" unterstützen #1691

Closed
aschempp opened this issue Nov 29, 2011 · 9 comments
Closed

ModuleCustomnav sollte CSS-Klasse "trail" unterstützen #1691

aschempp opened this issue Nov 29, 2011 · 9 comments
Assignees
Labels
Milestone

Comments

@aschempp
Copy link
Member

In einigen Fällen wäre es praktisch die CSS-Klasse "trail" auch im ModuleCustomnav zu haben. Schaden kann sie eigentlich nicht. Hier eine Umsetzung für 2.8.1 (Zeile 183+). Geändert hat sich nur der "class" Parameter.

$items[] = array
(
    'isActive' => false,
    'class' => trim((strlen($arrPage['cssClass']) ? $arrPage['cssClass'] : '') . (in_array($arrPage['id'], $objPage->trail) ? ' trail' : '')),
    'pageTitle' => specialchars($arrPage['pageTitle']),
    'title' => specialchars($arrPage['title']),
    'link' => $arrPage['title'],
    'href' => $href,
    'target' => (($arrPage['type'] == 'redirect' && $arrPage['target']) ? LINK_NEW_WINDOW : ''),
    'description' => str_replace(array("\n", "\r"), array(' ' , ''), $arrPage['description']),
    'accesskey' => $arrPage['accesskey'],
    'tabindex' => $arrPage['tabindex'],
    'alias' => $arrPage['alias']
);

Download the attachments

--- Originally created on March 10th, 2010, at 10:40am (ID 1691)

@ghost ghost assigned leofeyer Nov 29, 2011
@leofeyer
Copy link
Member

Da das Modul keine Hierarchien berücksichtigt, macht die Klasse "trail" nicht wirklich Sinn. Welche übergeordneten Seiten möchtest Du in einem nicht verschachtelten Menü markieren?

--- Originally created on March 10th, 2010, at 12:42pm

@aschempp
Copy link
Member Author

Das habe ich mir auch immer gedacht. Im Anhang ein Screenshot einer Seite. Die Menüpunkte ganz oben rechts ist eine generische Navigation, die gleich darunter die normale. Links kommt dann eine Unternavigation der ersten Ebene. Die Customnav-Punkte befinden sich auf demselben Level wie die Hauptpunkte, sind aber "im Menü versteckt". Das muss so sein damit links das Menü entsprechend mitspielt. Der Menüpunkt oben sollte aber aktiv (d.h. trail) sein, wenn eine Unterteite Links ausgewählt wird.

Trail ist ja nicht nur im selben Menü-Modul, sondern auch wenn in einem anderen Menü-Modul eine Unterseite derselbigen ausgewählt ist.

--- Originally created on March 10th, 2010, at 12:47pm

@leofeyer
Copy link
Member

Die Navigation oben rechts ist vom Typ "ModuleCustomnav"?

--- Originally created on March 10th, 2010, at 12:57pm

@aschempp
Copy link
Member Author

Ja genau, und die Menüpunkte sollten aktiv (rot) sein wenn links ein Unterpunkt gewählt wird.

--- Originally created on March 10th, 2010, at 12:59pm

@aschempp
Copy link
Member Author

Bezüglich der Darstellung, die Seiten haben eine CSS-Klasse und der Klasse ist ein Background-Image zugewiesen ;-)

--- Originally created on March 10th, 2010, at 01:00pm

@leofeyer
Copy link
Member

Ja genau, und die Menüpunkte sollten aktiv (rot) sein wenn links ein Unterpunkt gewählt wird.

Ja, das macht Sinn :) Jetzt frage ich mich nur, ob die Änderung Auswirkungen auf bestehende Webseiten haben könnte. Wenn dem so wäre, müssten wir bis zur 2.9 warten.

--- Originally created on March 10th, 2010, at 01:02pm

@aschempp
Copy link
Member Author

Das ist ne gute Frage. Wenn jemand aus versehen die Trail-Klasse für alle Module (also nicht nur für mod_navigation) definiert hat, könnte das Einfluss haben. Für mich wäre es nicht sehr schlimm wenn es erst in 2.9 kommt, ich habe ein SVN für diese Seite und muss das dann halt selber pflegen bei einem Update. Aber da das ModuleCustomnav vor 2.9 nicht geändert wird, wird das auch nie ein Problem sein.

--- Originally created on March 10th, 2010, at 01:06pm

@leofeyer
Copy link
Member

Implementiert in 3d7aafa.

--- Originally created on April 8th, 2010, at 11:07am

@leofeyer
Copy link
Member

--- Originally completed on April 8th, 2010, at 11:07am

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants