Simplifier dcPage::breadcrumb

[bruno]

• Enlever l'option "echo" (l'appelant sait en général s'il doit faire un echo ou non)
• Mettre automatiquement la dernière entrée en span class="page-title", plutôt que de le faire faire par l'appelant. Le cas écheant, permettre de le désactiver via un paramètre optionnel.

[kozlika]

Dois-je ouvrir un ticket séparé ?

Le breadcrumb des permissions est double :

• 1. Go to dashboard › Système : Utilisateurs › Actions
• 2. Utilisateurs › Permissions

(et du coup je vois que le alt du lien de retour au dashboard n'est pas localisé)

[bruno]

Replying to kozlika:

Dois-je ouvrir un ticket séparé ?

Le breadcrumb des permissions est double :

• 1. Go to dashboard › Système : Utilisateurs › Actions
• 2. Utilisateurs › Permissions

Je regarde ça.

Pas de véto sur l'implémentation du ticket ? (J'ai peut-être raté quelques pages particulières dans l'admin qui ont des besoins tels qu'implémenté).

[franck]

Perso j'attendrai de voir si certains plugins tiers n'auraient pas un besoin spécifique de ce côté.

[franck]

Pour le paramètre echo, suis d'ac, personne ne s'en sert, on peut le virer.

Pour le span on risque de se retrouver avec des plugins qui provoqueront un double span, c'est moche. De plus je me demande si c'est à cette fonction de gérer la présence d'un span comme celui-ci dans le dernier élément du breadcrumb. Pour moi c'est à l'appelant de décider ce que doit être les éléments de son fil d'Ariane.

On pourrait même concevoir qu'un plugin complète ce fil d'Ariane avec un ou plusieurs éléments qui ne sont pas la page courante, mais autre chose…

[bruno]

Pour le span on risque de se retrouver avec des plugins qui provoqueront un double span, c'est moche. De plus je me demande si c'est à cette fonction de gérer la présence d'un span comme celui-ci dans le dernier élément du breadcrumb. Pour moi c'est à l'appelant de décider ce que doit être les éléments de son fil d'Ariane.

dcPage::breadcrumb n'a été introduit qu'avec la non-encore officielle 2.6. Le risque d'avoir le double span est réduit.

Pour ce qui est de laisser le choix au plugin, d'accord. Mais alors il ne doit pas fournir de code html dans le paramètre, du moins pas celui-là qui à mon sens relève du core (quid du jour où on renommera la classe du span?). Je le verrais plutôt comme un paramètre de l'entrée du tableau (formalisme à définir). du genre :

		dcPage::breadcrumb(
			array(
				__('System') => '',
				__('Blogs') => 'blogs.php',
				__('New blog') => array('url' => '', "page-title" => true)
			))

[franck]

Au temps pour moi pour le double span, j'étais persuadé que ça datait déjà de la 2.5.2 ou 2.5.3. Donc oui, on peut faire ce qu'on veut.

Et ok pour le reste, ça me convient.

[franck]

Au moment de la simplification, penser à adapter simpleMenu qui met dans certain cas un dernier item qui n'est pas "page-title".

Je m'en suis servi pour indiquer le numéro d'étape au moment de l'ajout d'un élément de menu.

[Dsls]

(In [6f409aa25386]) revamped dcPage::breadcrumb, closes #1559