Kodi-vStream / venom-xbmc-addons

Addon de Streaming français pour Xbmc
GNU General Public License v3.0
497 stars 276 forks source link

Soucis touche retour info film-serie #905

Closed ghost closed 7 years ago

ghost commented 7 years ago

Quand on affiche les informations (via popup menu) d'un film/série, ca affiche en plein ecran avec bouton si résumé/casting/bande Annonce/ et tout à droite bouton ( ya rien marquer dessus) pour fermer la popup , plus image en fond si dispo. Avant ou pouvait fermer l'ex popup avec touche retour ( sous tel android ou box android avec telecommande pour ma part). La, la touche retour ne fonctionne plus pour fermer la popup, faut se déplacer sur le bouton pour fermer, pas pratique ... Mais si on remet '92' dans le code suivant ( config.py) def onAction( self, action ): if action.getId() in ( 9, 10, 11, 30, 247, 257, 275, 61467, 61448, ): self.close() La touche retour ferme à nouveau la popup information. Ya eu un commit ou a été enlevé les nombres (92,216). On peut les remettre en place ou ca va poser problème quelque part?

ghost commented 7 years ago

chez moi ça fonctionne la touche retour tu as quel version de kodi ? screenshot001

touche echap ?

ghost commented 7 years ago

Version Kodi 15.2 Pas de touche Echap sous Android/kodi dispo pour ma part , pour naviguer obliger d'utiliser retour arrière sur tel ou box telecommande. Et si on remet 92( = ACTION_NAV_BACK) , c'est ok. Le mode plein écran des infos ca doit jouer. Et le bouton 'fermer' , j'ai rien sur le bouton écrit de mon côté. Ex: http://img11.hostingpics.net/pics/402237fotonoexif.png

TmpName commented 7 years ago

Si le prb c'est la touche esc (qui sert beacoup sur kodi) pkoi tu la configures pas sur ta telecommande simplement ? plus de touches dispos, ou non detectees par kodi ? Apres je sais pas pkoi ca a été supprime, mais doit y avoir une raison, esc et back n'ont pas le meme effet sur kodi.

Edit: J'ai trouve https://github.com/Kodi-vStream/venom-xbmc-addons/commit/b08e233c1ac48365374b7e2becc5c010e44a6f7e Si @LordVenom passe par la ...

ghost commented 7 years ago

C'est pas la touche esc... j'en ai pas sous android. C'est retour arrière qu'on utilise via tel ou box. Ma telecommande fournie ya que retour arrière en plus. Après je rajoute juste ' 92 ' de mon côté et c'est ok de nouveau. C'était pour savoir si on peut le remettre dans le code ou si je fais la modif juste chez moi xD edit: oui c'est ce commit ^^

LordVenom commented 7 years ago

Pas de raison particuliere je vais poster la modif juste avant tu peux tester ligne 625 de skin/default/720/dialoginfo.xml

""

par

""

Voir si le texter Fermer s'affiche

Note comptend de voir que la popup fonctionne pas trop mal j'avais des doutes.

ghost commented 7 years ago

@LordVenom Testé mais ca n'affiche toujours pas 'Fermer' Si je change la valeur en 20410 ( = Bande Annonce) ou 31127 (TvTunes) ca s'affiche ses valeurs. Mais il va chercher ou ce 15067 ? ca vient avec les infos tmdb ?

LordVenom commented 7 years ago

https://github.com/xbmc/translations/blob/master/kodi-translations/kodi-main/merged-langfiles/kodi.core/resource.language.fr_fr/resources/strings.po

LordVenom commented 7 years ago

Je comprend pas ça devrais fonctionner, si tu met 30033 ça affiche Outil ? si oui je modifirais pour creer un language dans l'addon directement.

ghost commented 7 years ago

Ca doit kodi 15.2 , j'ai réinstallé la langue fr , il me telecharge la version 1.0.6 , et vu ton lien la version actuelle est en 3.0.11 ... Sur toutes mes installs kodi 15.2 , il me telecharge la 1.0.6 ou ya pas la valeur label 15067 dans strings.po Doit se baser sur un ancien repo pour telecharger sous 15.2. Ou rajouter cette valeur en dur dans le string.xml Fr/En pêtre? Je vais encore bidouiller voir le lien qu'il telecharge en installant la langue, bizarre ce bug. Edit: Non 30033 donne rien , donc passer par le strings.xml ca passe pas ou faut faire autrement.

ghost commented 7 years ago

ok en fait sous kodi 15.2 ( version android , autre os pareil surement) il prend le dépot suivant : http://mirrors.kodi.tv/addons/isengard/resource.language.fr_fr/ Et on est en version 1.0.6 max en language fr ou ya pas la valeur 15067 (Fermer). On peut le mettre en dur dans le strings.xml pour Fr/En et dans le label on met:

Ca affiche bien 'Outil' là. Pas génant pour moi le soucis 'fermer' perso, après si faut garder une certaine rétro compatibilité , à voir. C'est plus la touche retour qui manque ^^ ps: Ya pas dans tes stats le pourcentage des différentes versions kodi utilisés via vStream?

LordVenom commented 7 years ago

Résolut dans la prochaine version.

ghost commented 7 years ago

Testé ok, merci xD

ghost commented 7 years ago

par contre des fois la nouvelle fenetre ne fonctionne pas et affiche l'ancienne fenetre mais vide

LordVenom commented 7 years ago

Normal si pas d info trouver c est la popup de base vide . mais ça serais plus logique de faire un message pas d info trouver c est vrais.

TmpName commented 7 years ago

Question ! Est qu'il y a des cas ou ca affche la vielle fenetre mais pleine ?

Il suffirait de remplacer tout les "xbmc.executebuiltin("Action(Info)")" par un popup, mais j'aimerais etre sur qu'il n'y ait pas de prbs (eg si meta active ou pas)

ghost commented 7 years ago

J'ai eu des fois pas pleine juste la thumb du film peut etre il affiche l'ancienne quand il trouve pas le synopsis ou autre

TmpName commented 7 years ago

Ben vla c'est pour ca que j esite. Tu peux l'avoir vide avec jsute le thumb et un lien au format pluggin. Mais si les infos sont deja dans kodi, tu peux aussi avoir la vielle version mais pleine, mais ca depend de tellement de trucs que je ne sais pas comment ca se produit.

Et il y a aucun moyen de savoir si elle va etre pleine ou vide vu que c'est kodi qui gere tout.

LordVenom commented 7 years ago

Click sur info de la télécommande ou I du clavier = fenêtre d'info du skin peux importe si meta activer ou pas donc vide ou pleine.

Click sur le context information peux importe si meta activer ou pas il vas cherche les infos sur TMDB et affiche la popup de l'addon ou extendedinfo si activer. Si TMDB plante pour une raison indeterminer except: affichage de la popup skin si pas imdb_id un popup d'information "erreur pas d'info".

TmpName commented 7 years ago

Et on peut pas avoir le cas ou tmdb plante, mais que les infos soit deja dans kodi, et donc affichage possible ? Quoique statistiquement, avant que ca arrive ....

Moi je propose une modif qui affiche message d'erreur si pas de reponse TMDB (dans tout les cas). Je pense que l'user qui utilise cette fonction pour avoir l'info n'a pas deja les infos dans kodi, sinon c'est qu'il connait le film et donc n'a pas besoin des infos.

LordVenom commented 7 years ago

Hum... Il vas chercher dans la base de donner avant d'allais sur l'api.Et dans tous les cas imdb_id seras vide et ça retourneras "erreur d'info" mais si tu as trouver un code plus propre pas de soucis pour moi fait toi plaisir :)

Note : Perso j'utilise cette fonction pour les résumer et le fanart.

reNote: extend info et vStream n'utilise pas la même base de donner de Film.

TmpName commented 7 years ago

J'ai mit une modif en test. Mais pr les tests... j'ai des soucis d'affichage du dialog, mise en page, mais ca a l'air de marcher.

ghost commented 7 years ago

dites on pourrait pas mettre en veilleuse le son pour les erreur comme j'ai fait sur siteuptobox ? popup silencieuse les popup d'erreur et autres il y en a souvent avec vstream je dois pas etre le seul a qui ça prend la tete le son de confluence(pas tester avec le nouveau theme)

zombiB commented 7 years ago

dans parametres systeme -audio-tu peux désactiver le son de l'interface graphique ou gui sounds

ghost commented 7 years ago

@zombiB ça desactive tous les sons ce n'est pas ce que je veut je parle des popup erreur

TmpName commented 7 years ago

Edit: J'ai vu que tu connaissais deja la soluce ^^.

Apres je sais pas, moi ca me derange pas, tant qu'il n'y pas de film qui tournent.

On peut rajouter une option dans l'api de Vstream pr activer/desactiver le son, mais dans quel cas on le laisse ou on le vire ? Mettre un checkbox dans les parametres ?

ghost commented 7 years ago

Par default c'est optionnelle le son et si il n'y a pas l'argument c'est toujours true Donc oui ça serait possible avec checkbox activer desactiver dans settings Bah moi si il y avait l'option dans vstream je désactiverai le son de toutes les popups que je trouve inutile vu qu'ont regarde l'ecran quand on clique donc

TmpName commented 7 years ago

J'attend l avis du boss. Et perso je partirais sur la chackbox a cocher ou pas, je vois pas pkoi certaines notifications aurait plus besoin d'un son qu'une autre, meme si perso moi ca me derange pas.

LordVenom commented 7 years ago

ça me dérange pas non plus avec ou sans pas de soucis.

TmpName commented 7 years ago

J'ai rajoute une section divers dans le menu de config du coup. A tester.

ghost commented 7 years ago

j'étais dessus aussi je cherché comment convertir true en True ou 0 en 1 sans ajout de code

fonctionne aussi avec def error xbmcgui.Dialog().notification('Vstream', ('Erreur: '+str(e)),self.__sIcon,10000,sound)

TmpName commented 7 years ago

Ben pour les erreurs le son me semble normal, en temps normal pas d'erreurs > pas de sons.

TmpName commented 7 years ago

Bon j'ai remodifie le code. Meme si l'user a tout laisse par defaut (son on), il y a moyen de desactiver les son en mettant flase dans la fonction, pour les notifications inutile (sous titre, authentification, ect ...)

ghost commented 7 years ago

un bon exemple de prise de tete avec le son de confluence pour moi c'est de regarder plusieurs épisodes d'une série a la suite ou le volume doit etre augmenté car bas donc on le fait au premier épisode ensuite on oublie de le baisser(ou alors c'est chiant monté et descendre le volume a chaque fois) et au prochain episode resolve ouverture du flux le son de confluence a fond!

TmpName commented 7 years ago

XD, je comprend.