[patch] ne pas changer le curseur à cause de la barre rapide

Crayona
Messages : 457
Inscription : 24 juin 2012, 18:53

[patch] ne pas changer le curseur à cause de la barre rapide

Message par Crayona »

J'ai remarqué que si l'on utilise un objet dans la barre rapide avec la souris, le curseur (ou "mode d'action") se change en doigt et on ne peut plus cliquer sur un adversaire pour l'attaquer (même avec ALT + clic gauche). On est obligé de faire itérer le curseur, ce qui est bien inconvenant pendant le combat.

Dans le code source j'ai enlevé deux lignes qui me semblent illogiques, et après tout se passe comme je voulais dans mes testes, mais il est possible que ce changement casse quelque chose subtilement.

En tout cas voici le patch si vous voulez essayer vous-mêmes:

Code : Tout sélectionner

diff --git a/fr_quickitems.c b/fr_quickitems.c
index 152dc2f..6dcb9b2 100644
--- a/fr_quickitems.c
+++ b/fr_quickitems.c
@@ -738,10 +738,6 @@ int click_fr_quickitems_handler(window_info *win, int mx, int m
                return 1;
        }
 
-       if (qb_action_mode==ACTION_USE_WITEM) action_mode=ACTION_USE_WITEM;
-       if (qb_action_mode==ACTION_USE) action_mode=ACTION_USE;
-
-
        // Recherche l'id de la case cliqu<E9>e dans la quickbar
        id_quick = get_quickitem_from_mouse(mx, my);


Ackak
Messages : 7348
Inscription : 18 déc. 2005, 19:31

Re: [patch] ne pas changer le curseur à cause de la barre ra

Message par Ackak »

Modification mise en place sur le svn.
Celui qui apprend quelque chose de moi enrichit son savoir sans réduire le mien, tout comme celui qui allume sa chandelle à la mienne se donne de la lumière sans me plonger dans l'obscurité.
-- Thomas Jefferson

Crayona
Messages : 457
Inscription : 24 juin 2012, 18:53

Re: [patch] ne pas changer le curseur à cause de la barre ra

Message par Crayona »

Un avertissement mineur que j'ai récemment découvert: avec ce patch il n'est plus possible de changer le curseur en poing ("utiliser avec") sur la barre rapide et après cliquer sur un objet 3D de la carte, si le curseur de l'inventaire n'est pas aussi en poing. Il est toujours possible de le faire utilisant l'inventaire alors ce n'est pas trop gênant.

Crayona
Messages : 457
Inscription : 24 juin 2012, 18:53

Re: [patch] ne pas changer le curseur à cause de la barre ra

Message par Crayona »

Voici une petite amélioration: si l'on fait un clic avec le poing sur la barre rapide, le curseur primaire se change en poing aussi. Donc on peut de nouveau faire un "utiliser-avec" en prenant directement un objet de la barre.

Code : Tout sélectionner

diff --git a/fr_quickitems.c b/fr_quickitems.c
index 6dcb9b2..7c77442 100644
--- a/fr_quickitems.c
+++ b/fr_quickitems.c
@@ -995,7 +995,12 @@ int click_fr_quickitems_handler(window_info *win, int mx, int my, Uint32 flags)
                        return 1;
 
                case ACTION_USE_WITEM :
-                       if (use_item < 0) { use_item = id_item; return 1;}
+                       if (use_item < 0)
+                       {
+                               action_mode = ACTION_USE_WITEM;
+                               use_item = id_item;
+                               return 1;
+                       }
                        str[0] = ITEM_ON_ITEM;
                        str[1] = item_list[use_item].pos;
                        str[2] = item_list[id_item].pos;

Ackak
Messages : 7348
Inscription : 18 déc. 2005, 19:31

Re: [patch] ne pas changer le curseur à cause de la barre ra

Message par Ackak »

Modification prise sur le svn.
Celui qui apprend quelque chose de moi enrichit son savoir sans réduire le mien, tout comme celui qui allume sa chandelle à la mienne se donne de la lumière sans me plonger dans l'obscurité.
-- Thomas Jefferson

Répondre