[Résolu] Menu fixe en mode connecté

[Gilux 176]

Le 24/09/2020 à 19:49, Fred a dit :

vu trop tard

 

Super pour la sortie de cette nouvelle version, mais, vu trop tard moi aussi, en connexion en tant que membre avec le menu en position fixe, celui-ci occupe une position non conforme :

 

L'espace correspond à l'emplacement de la barre d'admin :

 

Et comme le menu est fixe, cet espace le précède en permanence :

 

 

 

Modifié 26 septembre 2020 par Gilux
Passe en résolu

[Fred 1 530]

Il y a 8 heures, Gilux a dit :

 

Super pour la sortie de cette nouvelle version, mais, vu trop tard moi aussi, en connexion en tant que membre avec le menu en position fixe, celui-ci occupe une position non conforme :

 

L'espace correspond à l'emplacement de la barre d'admin :

 

Et comme le menu est fixe, cet espace le précède en permanence :

 

 

 

Un des sites me faisait ça. Comme il a connu différentes versions avant d’être shooté, je n’ai pas approfondi. 
En tout cas, c’est plus ancien que la 10.3. 
Je déplace le commentaire dans le bon fil. 

[Fred 1 530]

Peux-tu m’adresser un export du thème par MP ?

[Gilux 176]

Il y a 5 heures, Fred a dit :

Peux-tu m’adresser un export du thème par MP ?

 

C'est fait 😉

 

[Fred 1 530]

Voir pour essai  la branche https://github.com/fredtempez/ZwiiCMS/tree/10301 qui corrige ce problème.

[Gilux 176]

Pas d'amélioration avec la V 10.3.01 😞

 

[Fred 1 530]

C’est que ton erreur est différente. F5 remet le menu en place ?

[Gilux 176]

Il y a 15 heures, Fred a dit :

C’est que ton erreur est différente. F5 remet le menu en place ?

 

Je ne sais pas si c'est mon erreur, ou si elle se reproduit pour toute personne se connectant en tant que membre ?

F5 : l'actualisation, même répétée, n'a aucun effet ni le vidage du cache, le menu fixe membre connecté est toujours décalé tel que le montre les captures d'écran ci-dessus.

 

J'ai trouvé une parade :

 

Constat dans common.css, le menu fixe est placé à 45px du haut de la page :

#navfixedconnected {
	top:45px;
	z-index:10;
	position: -webkit-sticky; /* Safari */
	position: sticky;
}

Donc pour tous, admin, éditeur et membre, mais il y a un nouveau menu membre, ou plutôt il n'y a plus de barre d'admin et les 2 seuls items du membre sont intégrés au menu, ce qui est beaucoup mieux, mais l'emplacement de la barre d'admin est toujours prévu, ce qui crée ce vide au dessus du menu, il suffit donc de le supprimer, comme ceci :

if ($this->getData(['user', $userId, 'group']) >= self::GROUP_MEMBER) {
echo '<style>
#navfixedconnected {
	top:0;
}
</style>';
}

Résultat :

 

Avant :

 

Après :

 

Ce code PHP de dépannage est provisoirement placé dans le fichier head.inc.html, mais je compte sur toi pour le placer dans le bon fichier du core

 

Par ailleurs, j'ai vu la modif de la ligne 2207 de core.php, je m'appretais à t'en faire part, mais tu as déjà corrigé le code

[Fred 1 530]

Je ne peux pas corriger un bug que je n'arrive pas à reproduire. Il faudra un backup de ton site et des infos sur le type d'hébergement, en local, en ligne réécriture, pas de réécriture, etc..

 

La partie du code qui gère cela est le mai.php :

 

				<!-- Détermine si le menu est fixe en haut de page lorsque l'utilisateur n'est pas connecté -->
				<?php
				if ( $this->getData(['theme', 'menu', 'position']) === 'top'
					AND $this->getData(['theme', 'menu', 'fixed']) === true
					AND $this->getUser('password') === $this->getInput('ZWII_USER_PASSWORD') ) {
						echo '<nav id="navfixedconnected" >';
					} else {
						echo '<nav id="navfixedlogout" >';
					}
				?>

Si le test considère que l'utilisateur est connecté, c'est que le mot de passe est toujours valide. C'est d'ailleurs le même test que celui qui gère l'affichage de la barre de membre sauf qu'il est placé dans core.php ce qui faut une sacrée différence.

 

La modification apportée évite le flottement du menu après une déco automatique :

		// Force la déconnexion des membres bannis ou d'une seconde session
		if (
			$this->getUser('password') === $this->getInput('ZWII_USER_PASSWORD')
			AND ( $this->getUser('group') === self::GROUP_BANNED
				  OR $_SESSION['csrf'] !== $this->getData(['user',$this->getUser('id'),'accessCsrf']) )
		) {
			$user = new user;
			$user->logout();
			// Recharge la classe
			$core = new core;
			$core->router();

Pour le coup cette partie fonctionne, j'ai été capable de reproduire le bug et de le corriger. Cela veut dire que ton bug apparaît dans une autre circonstance.

 

 

 

 

[Fred 1 530]

Avant tout envoi je vais tester une autre solution, je te tiens au courant par MP.

[Gilux 176]

il y a 13 minutes, Fred a dit :

Je ne peux pas corriger un bug que je n'arrive pas à reproduire

 

C'est surprenant, le bug se produit en se connectant en tant que membre, sans rien faire d'autre, donc personne d'autre que moi n'a rencontré ce bug ? J'ai peine à y croire !

[roger 296]

il y a 42 minutes, Gilux a dit :

C'est surprenant, le bug se produit en se connectant en tant que membre, sans rien faire d'autre, donc personne d'autre que moi n'a rencontré ce bug ? J'ai peine à y croire !

Je viens de vérifier sur une version 10.3.00 en production....je confirme le "blème"

 

Un membre avec droit éditeur a sa barre admin tout en haut (comme pour l'admin d'un zwii), là c'est OK

Un simple membre à sa barre d’administration (gérer son compte et déconnexion) dans la barre de menu du site, c'est comme pour @Gilux pas au bon endroit (comme pour un admin ou un membre éditeur qui eux l'ont en haut du site.)

 

[Gilux 176]

il y a 41 minutes, roger a dit :

.je confirme le "blème"

 

 

Ah ! Tout de même ! Je me sens moins seul 😉

 

Le bug montré ci-dessous avec une install fraîche de la dernière version, téléchargée sur le git et installée à l'instant :

 

Configuration du menu :

 

Connecté en tant que membre :

 

 

@Fred Le zip de sauvegarde du thème, si besoin, t'es transmis en MP

 

[Fred 1 530]

il y a une heure, roger a dit :

Un simple membre à sa barre d’administration

Là oui, je reproduis le bug.

Mais @gilux, tu ne me donnes pas assez d'élément qui me permette de reproduire la mauvaise position. Est-ce un compte éditeur ?

[Fred 1 530]

Bug corrigé, la barre de membre simple reste bien en haut de page.

 

Je n'avais pas capté qu'il s'agissait d'une connexion en tant que membre simple 🥴

 

[Gilux 176]

Il y a 2 heures, Fred a dit :

Je n'avais pas capté qu'il s'agissait d'une connexion en tant que membre simple 🥴

 

Je ne pensais pas qu'il fallait préciser "simple", car sur 3 connexions possibles, une seule est nommée membre, et la barre de menu avec les 2 icônes supplémentaires est bien différente de celle de l’administrateur ou de celle de l’éditeur...

 

Le 24/09/2020 à 22:53, Gilux a dit :

Super pour la sortie de cette nouvelle version, mais, vu trop tard moi aussi, en connexion en tant que membre avec le menu en position fixe, celui-ci occupe une position non conforme :

 

Peu importe, le bug est fixé, je passe le sujet en résolu 😉

 

[Fred 1 530]

Il y a 1 heure, Gilux a dit :

Je ne pensais pas qu'il fallait préciser "simple", car sur 3 connexions possibles, une seule est nommée membre, et la barre de menu avec les 2 icônes supplémentaires est bien différente de celle de l’administrateur ou de celle de l’éditeur.

Laisse, c’est moi les captures étaient explicites mais je me suis focalisé sur l’une. La solution était simple une fois le problème identifié :

 

 <?php
                if ( $this->getData(['theme', 'menu', 'position']) === 'top'
                    AND $this->getData(['theme', 'menu', 'fixed']) === true
                    AND $this->getUser('password') === $this->getInput('ZWII_USER_PASSWORD',true) 
                    AND $this->getUser('group') > self::GROUP_MEMBER) {
                        echo '<nav id="navfixedconnected" >';
                    } else {
                        echo '<nav id="navfixedlogout" >';
                    }
  ?>