spip
/
bigup
15
0
Fork 3
Code Issues 8 Pull Requests Releases Wiki Activity
Labels Milestones
New Issue

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.
tcharlss commented 3 years ago
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.
tcharlss commented 3 years ago
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.
miros commented 3 years ago

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.
tcharlss commented 3 years ago
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.
tcharlss commented 3 years ago
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.
maieul commented 2 years ago
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)
cerdic commented 2 years ago
Owner

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

ah ben j'ai pas le pouvoir de fermer ici :p
miros commented 2 years ago

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

maieul a dit résolu **Statut changé à Résolu**
miros commented 2 years ago

Statut changé à Fermé

**Statut changé à Fermé**
miros commented 2 years ago
There is no content yet.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
1.0
1.1
2.0
2.1
3.0
3.1
3.2
master
Labels
Apply labels
Clear labels
amélioration
Amélioration, nouvelle fonctionnalité bug
Ca ne fonctionne pas documentation
Documentation souhaite doublon
Ce ticket est un doublon invalide
Ticket invalide refusé
Ignoré, c'est comme Ca...
No Label amélioration bug documentation doublon invalide refusé
Milestone
Set milestone
Clear milestone
No items
No Milestone
Assignees
Assign users
Clear assignees
No Assignees
4 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch '%!s(MISSING)'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes