Masquer le bouton Télécharger si Bigup est bien actif #4483

Closed
opened 3 years ago by rastapopoulos · 13 comments
Owner

Si Bigup est bien lancé, c'est que le JS marche bien etc. On a donc la zone de drop, et même si on veut pas drop, ya le bouton pour ouvrir la boite de fichiers. Ensuite que l'on drop ou que l'on fasse pas Parcourir, dans les deux cas ça lance l'upload immédiatement. On ne devrait donc jamais voire le bouton Télécharger, puisque ça incite à cliquer dessus, ce qui fait tout planter non ? (si on clique pendant que ça progresse en JS)

Vu que là on s'apprête à l'ajouter dans la distrib officielle, ça serait bien de le corriger avant car c'est un gros truc d'UI qui se voit et qui vaut pour tout le monde.

Si Bigup est bien lancé, c'est que le JS marche bien etc. On a donc la zone de drop, et même si on veut pas drop, ya le bouton pour ouvrir la boite de fichiers. Ensuite que l'on drop ou que l'on fasse pas Parcourir, dans les deux cas ça lance l'upload immédiatement. On ne devrait donc jamais voire le bouton Télécharger, puisque ça incite à cliquer dessus, ce qui fait tout planter non ? (si on clique pendant que ça progresse en JS) Vu que là on s'apprête à l'ajouter dans la distrib officielle, ça serait bien de le corriger avant car c'est un gros truc d'UI qui se voit et qui vaut pour tout le monde.
Et le patch est ici : https://github.com/marcimat/bigup/pull/13

J'ai fait un fork et une branche ici https://git.spip.net/RealET/bigup/src/branch/issue%234483

J'ai pu fusionner cette branche avec mon fork (https://git.spip.net/RealET/bigup)
Mais je n'ai pas accès à la demande de fusion de cette branche avec https://git.spip.net/spip/bigup

J'ai fait un fork et une branche ici https://git.spip.net/RealET/bigup/src/branch/issue%234483 J'ai pu fusionner cette branche avec mon fork (https://git.spip.net/RealET/bigup) Mais je n'ai pas accès à la demande de fusion de cette branche avec https://git.spip.net/spip/bigup

Et maintenant que https://git.spip.net/spip/bigup est l'original et non plus le miroir de github, je ne peut toujours pas faire la PR dessus.

Et maintenant que https://git.spip.net/spip/bigup est l'original et non plus le miroir de github, je ne peut toujours pas faire la PR dessus.
Poster
Owner

parce que quoi ? ya pas le bouton ?

parce que quoi ? ya pas le bouton ?

Y'a le bouton, je peux dire que je choisi spip:master dans la liste déroulante.
Et là, je tombe sur une 404 : https://git.spip.net/spip/bigup/compare/master...RealET/bigup:issue%234483

Y'a le bouton, je peux dire que je choisi spip:master dans la liste déroulante. Et là, je tombe sur une 404 : https://git.spip.net/spip/bigup/compare/master...RealET/bigup:issue%234483
Poster
Owner

mais t'as vraiment nommé ta branche avec un # ? Le nom de la branche est utilisée dans l'URL ensuite, et donc forcément # ça veut dire une ancre dans une URL… donc du coup l'URL générée est en fait "bigup/issue" à la fin, ce qui n'existe pas, non ?

mais t'as vraiment nommé ta branche avec un # ? Le nom de la branche est utilisée dans l'URL ensuite, et donc forcément # ça veut dire une ancre dans une URL… donc du coup l'URL générée est en fait "bigup/issue" à la fin, ce qui n'existe pas, non ?

Ben oui, personne ne m'a dit que c'était impossible, alors je l'ai fait ;-)

Ben oui, personne ne m'a dit que c'était impossible, alors je l'ai fait ;-)
Poster
Owner

Bé faut que tu la renommes donc

Bé faut que tu la renommes donc

J'ai renommé
Je vais dans https://git.spip.net/RealET/bigup/compare/master...issue4483
Je choisi de faire une nouvelle demande de fusion.
J'indique comme destination : spip:Master (j'ai pas d'autres choix pour SPIP).
Et je tombe sur une 404
https://git.spip.net/spip/bigup/compare/master...RealET/bigup:issue4483 : La page que vous essayez d'atteindre n'existe pas ou vous n'êtes pas autorisé à la voir.

J'ai renommé Je vais dans https://git.spip.net/RealET/bigup/compare/master...issue4483 Je choisi de faire une nouvelle demande de fusion. J'indique comme destination : spip:Master (j'ai pas d'autres choix pour SPIP). Et je tombe sur une 404 https://git.spip.net/spip/bigup/compare/master...RealET/bigup:issue4483 : La page que vous essayez d'atteindre n'existe pas ou *vous n'êtes pas autorisé* à la voir.

`RastaPopoulos : est-ce que tu peux essayer toi, qui doit être "Autorisé à la voir" ?

`RastaPopoulos : est-ce que tu peux essayer toi, qui doit être "Autorisé à la voir" ?
Owner

Bonjour

La fonctionnalité PR n'avait pas été activée suite à l'indépendance du projet sur la forge.
Cela est fait.

La demande de fusion devrait être disponible maintenant.

Bonjour La fonctionnalité PR n'avait pas été activée suite à l'indépendance du projet sur la forge. Cela est fait. La demande de fusion devrait être disponible maintenant.
Owner

La PR est donc là, à valider #1
Statut changé à En cours

La PR est donc là, à valider https://git.spip.net/spip/bigup/pulls/1 **Statut changé à En cours**
Owner

Intégré
Statut changé à Fermé

Intégré **Statut changé à Fermé**
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.