Page 1 sur 1

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

Publié : 06 mai 2014, 09:06
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);


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

Publié : 14 juin 2014, 17:03
par Ackak
Modification mise en place sur le svn.

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

Publié : 14 juin 2014, 18:10
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.

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

Publié : 06 nov. 2014, 01:46
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;

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

Publié : 11 nov. 2014, 16:43
par Ackak
Modification prise sur le svn.