Malaclypse Report post Posted March 6, 2005 As it looks to me it is necessary to do the step wytter said. I noticed yesterday, that key don't work accurately on depending where the cursor is. For example, if I had my cursor on the frame border (where the hud, icons and the like are) and I press either of TAB or F1 the interface didn't switch, but nothing occurs. I first had to move the cursor into the game window. The same is true, if the cursor is inside some other window (manu, spell, inventory, no matter which one), the keys don't work as expected all the time. Share this post Link to post Share on other sites
crusadingknight Report post Posted March 6, 2005 Another problem with the cursor, it that when you click an item to move it, instead of being drawn with the cursor, it is instead drawn in the root window, becoming semi-invisible behind the current (Read- Inv, Drop, etc.) window. Share this post Link to post Share on other sites
Grum Report post Posted March 6, 2005 (edited) Another problem with the cursor, it that when you click an item to move it, instead of being drawn with the cursor, it is instead drawn in the root window, becoming semi-invisible behind the current (Read- Inv, Drop, etc.) window. Yes, I know about that one. I have to look into it, but creating all root windows was the first priotity (one or two to go, I'm not sure). As to the problem with other windows taking over keypresses: they shouldn't, and it's a bug. Since no window apart from the root windows has a keypress handler yet, all key presses should fall through to the root window. I'll try to fix that. [EDIT: can't reproduce, it Works For Me ] Edited March 6, 2005 by Grum Share this post Link to post Share on other sites
Grum Report post Posted March 7, 2005 (edited) BTW, you should make the client force tabs in 640x480. Done Another problem with the cursor, it that when you click an item to move it, instead of being drawn with the cursor, it is instead drawn in the root window, becoming semi-invisible behind the current (Read- Inv, Drop, etc.) window. Fixed. As for unifying the keypress handlers for the various windows: yeas, that needs to be done. I'll look into it when I'm done with the last root window (rules). EDIT: Gah, broke zooming again. Also the frigging hud windows don't show up when compiling without -DWINDOW_CHAT. I should've stayed in bed this morning EDIT 2: fixed zooming. Really stupid mistake. EDIT 3: fixed the hud too. Damn, I wish this cold was over. Edited March 7, 2005 by Grum Share this post Link to post Share on other sites
Specter Report post Posted March 7, 2005 Grum I think I speak for everyone when I say "Good job, keep up the good work G-man" Share this post Link to post Share on other sites
duran Report post Posted March 7, 2005 your codeing this while haveing a cold? Share this post Link to post Share on other sites
Grum Report post Posted March 8, 2005 Specter, thanks, I appreciate it. duran, yes. Going to bed early tonight and hope it'll be better tomorrow. Anyway, between sneezes I managed to finish the last root window. Next is cleaning up, and then I'll switch the code to using the new windows by default insetad of having to define WINDOW_CHAT. Share this post Link to post Share on other sites
Specter Report post Posted March 8, 2005 Excellent cant wait to go bug hunting Share this post Link to post Share on other sites
Grum Report post Posted March 8, 2005 Excellent cant wait to go bug hunting Have fun The new root windows are now the default. To use the old event loop, add -DOLD_EVENT_HANDLER to your compile flags. Share this post Link to post Share on other sites
Malaclypse Report post Posted March 9, 2005 The bug related to several keypresses (esp. TAB/F1) I reported in this thread has gone :/ At least I cannot reproduce it. Maybe it was only introduced intermediary while you were working on it. Thanks Grum for this excellent work Share this post Link to post Share on other sites
Grum Report post Posted March 15, 2005 Okay, I'm going to keep the old event loop one more week. If noone objects before then, I'll remove it. Share this post Link to post Share on other sites