Conflit bigup+ espace privé fluide + insérer modèle #4485

Closed
opened 3 years ago by maieul · 10 comments
maieul commented 3 years ago
Collaborator

Ce ticket est la reprise de https://github.com/marcimat/bigup/issues/12, maintenant que bigup est commutaire.

Avec SPIP 3.3 (48c029a85) et bigup (cdb4188), bigup empeche le clique sur le formulaire "insérer_modele" situé sur la colonne de droite.

Il semblerait après dichtomie que ce soit cette ligne https://github.com/marcimat/bigup/blob/v1.0.6/prive/style_prive_plugin_bigup.html#L94 qui pose problème.

Mais après je suis pas assez calé css pour comprendre le pourquoi du comment.

Ce ticket est la reprise de https://github.com/marcimat/bigup/issues/12, maintenant que bigup est commutaire. Avec SPIP 3.3 (48c029a85) et bigup (cdb4188), bigup empeche le clique sur le formulaire "insérer_modele" situé sur la colonne de droite. Il semblerait après dichtomie que ce soit cette ligne https://github.com/marcimat/bigup/blob/v1.0.6/prive/style_prive_plugin_bigup.html#L94 qui pose problème. Mais après je suis pas assez calé css pour comprendre le pourquoi du comment.
Owner

Lié au ticket #4464 , effectivement c'est en rapport avec ce sélecteur CSS.

Ça a été traité sur le dépôt de github, je ne sais pas trop où en est la synchro avec gitea mais pas encore reporté dessus visiblement : cdb4188db4

Dans le patch j'ai enlevé le "débordement" de 10px en haut et en bas, je me dis qu'il faudrait peut-être carrément ajouter un z-index: -1 pour éviter tout problème.

Lié au ticket #4464 , effectivement c'est en rapport avec ce sélecteur CSS. Ça a été traité sur le dépôt de github, je ne sais pas trop où en est la synchro avec gitea mais pas encore reporté dessus visiblement : https://github.com/marcimat/bigup/commit/cdb4188db466a8ba5afe9dc66dd866d2ae5a555a Dans le patch j'ai enlevé le "débordement" de 10px en haut et en bas, je me dis qu'il faudrait peut-être carrément ajouter un z-index: -1 pour éviter tout problème.
Owner

Du coup je ne sais pas où on doit faire les PRs suivants pour Bigup.
Est-ce qu'on pourrait faire un point sur la migration ? Où cela en est-il, est-ce qu'il y a des choses qui bloquent ? Cf. #4482

`maieul je pense qu'il faudrait passer ce ticket en priorité haute parceque potentiellement ça impacte tous les contenus qui sont dans les colonnes latérales, pas juste insérer modèles.

Du coup je ne sais pas où on doit faire les PRs suivants pour Bigup. Est-ce qu'on pourrait faire un point sur la migration ? Où cela en est-il, est-ce qu'il y a des choses qui bloquent ? Cf. #4482 `maieul je pense qu'il faudrait passer ce ticket en priorité haute parceque potentiellement ça impacte tous les contenus qui sont dans les colonnes latérales, pas juste insérer modèles.

tcharlss 🐽 a écrit :

Du coup je ne sais pas où on doit faire les PRs suivants pour Bigup.
Est-ce qu'on pourrait faire un point sur la migration ? Où cela en est-il, est-ce qu'il y a des choses qui bloquent ? Cf. #4482

`maieul je pense qu'il faudrait passer ce ticket en priorité haute parceque potentiellement ça impacte tous les contenus qui sont dans les colonnes latérales, pas juste insérer modèles.

`tcharlss le correctif proposé n'est pas suffisant chez moi. J'ai du rajouter un #extra{z-index:100} dans un css pour que cela fonctionne.... pas reporté dans le core car je ne sais pas ou il faut le placer. et si c'est correct.

tcharlss 🐽 a écrit : > Du coup je ne sais pas où on doit faire les PRs suivants pour Bigup. > Est-ce qu'on pourrait faire un point sur la migration ? Où cela en est-il, est-ce qu'il y a des choses qui bloquent ? Cf. #4482 > > `maieul je pense qu'il faudrait passer ce ticket en priorité haute parceque potentiellement ça impacte tous les contenus qui sont dans les colonnes latérales, pas juste insérer modèles. `tcharlss le correctif proposé n'est pas suffisant chez moi. J'ai du rajouter un #extra{z-index:100} dans un css pour que cela fonctionne.... pas reporté dans le core car je ne sais pas ou il faut le placer. et si c'est correct.
Owner

Autre manifestation du problème : impossible de cliquer sur les onglets situés avant #contenu, cf. spip-contrib-extensions/roles_documents#10

Et donc en ajoutant z-index: -1 sur .bigup-extended-drop-zone::before ça semble suffire à régler le problème dans tous les cas. À confirmer avec insérer modèles.

Nb : m'est avis qu'il faudrait augmenter la priorité de ce ticket parceque c'est assez bloquant, ça rend des portions entières de l'interface inutilisables.

Autre manifestation du problème : impossible de cliquer sur les onglets situés avant #contenu, cf. https://git.spip.net/spip-contrib-extensions/roles_documents/issues/10 Et donc en ajoutant `z-index: -1` sur `.bigup-extended-drop-zone::before` ça semble suffire à régler le problème dans tous les cas. À confirmer avec insérer modèles. Nb : m'est avis qu'il faudrait augmenter la priorité de ce ticket parceque c'est assez bloquant, ça rend des portions entières de l'interface inutilisables.
Owner

Ps. : `maieul On pourrait également changer le titre de ce ticket pour le retrouver plus facilement : ça concerne spécifiquement l'extended-drop-zone, et rien à voir spécifiquement avec prive_fluide ou inserer_modeles.

Ps. : `maieul On pourrait également changer le titre de ce ticket pour le retrouver plus facilement : ça concerne spécifiquement l'extended-drop-zone, et rien à voir spécifiquement avec prive_fluide ou inserer_modeles.
Poster
Collaborator

Avant que cedric ne me le demande : on peut fermer ce ticket, visiblement ca a été intégré dans bigup. Pb résolu (le mien + celui de tcharlss)

Avant que cedric ne me le demande : on peut fermer ce ticket, visiblement ca a été intégré dans bigup. Pb résolu (le mien + celui de tcharlss)
Owner

ah ben j'ai pas le pouvoir de fermer ici :p

ah ben j'ai pas le pouvoir de fermer ici :p

maieul a dit résolu
Statut changé à Résolu

maieul a dit résolu **Statut changé à Résolu**

Statut changé à Fermé

**Statut changé à Fermé**
There is no content yet.
Sign in to join this conversation.
No Milestone
No Assignees
4 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.