00:21:07 Unstable branch on CRAWL.XTAHUA.COM updated to: 0.22-a0-858-g985ef60 (34) 00:59:24 !lg . fedhas won 00:59:25 1. ebering the Slayer (L27 CeHu of Fedhas), escaped with the Orb and 5 runes on 2015-11-12 17:16:21, with 2505733 points after 98781 turns and 12:17:09. 01:27:58 Unstable branch on crawl.develz.org updated to: 0.22-a0-858-g985ef60 (34) 02:59:11 Monster database of master branch on crawl.develz.org updated to: 0.22-a0-858-g985ef60 03:01:47 -!- OICU812 is now known as Tabmasher 03:19:48 Unstable branch on crawl.beRotato.org updated to: 0.22-a0-858-g985ef60 (34) 04:11:13 -!- amalloy is now known as amalloy_ 06:32:22 advil: you'd make the scroller widget expand vertically, to push it to the bottom of the screen 06:44:59 aidanh: It looks like some popups are missing a css rule for text in latest webtiles. 06:45:10 The descriptions of monsters, spells, items, etc. (except for gods) have different text colour in local tiles and webtiles. 06:45:20 Previously that text was light grey, fg7, but now it's white in webtiles and light grey in local tiles. 06:47:51 Ge0ff: thanks, I'll take a look at that tonight 07:40:47 aidanh: One more thing: after commit aabaeaee60 tiles in popups are hidden not only for tile_display_mode=glyph, but for all other modes too (webtiles, 0.22-a0-858-g985ef60). 09:55:35 03gammafunk02 07* 0.22-a0-859-g47ae1cb: Prevent ordering allies while confused (RoGGa, ebering, 11505) 10(16 minutes ago, 1 file, 3+ 3-) 13https://github.com/crawl/crawl/commit/47ae1cb34437 10:10:21 Unstable branch on crawl.jorgrun.rocks updated to: 0.22-a0-859-g47ae1cb344 (34) 11:46:32 03advil02 07* 0.22-a0-860-ga17920c: Optimize memorize menu a bit 10(9 minutes ago, 1 file, 112+ 87-) 13https://github.com/crawl/crawl/commit/a17920cad9f3 11:46:32 03advil02 07* 0.22-a0-861-g3db0934: Add MF_NOWRAP to spell menu 10(82 seconds ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/3db09340a758 11:46:57 man, the xcode profiler is amazing now that I know how to use it 11:47:03 it's just so easy to use 11:48:54 aidanh: the wrapping behavior of the memorize menu was a bit odd(/buggy?) before 3db09340a758, see the note in that commit message -- not sure if this only applies here / if it was working as intended but otherwise you may want to take a look 12:10:27 Unstable branch on crawl.jorgrun.rocks updated to: 0.22-a0-861-g3db09340a7 (34) 12:17:07 hm 12:17:25 did you know that summons with SPWPN pain use the summoner's necro skill 12:17:33 instead of their own 12:18:31 The build was broken. (master - 3db0934 #10012 : advil): https://travis-ci.org/crawl/crawl/builds/409000824 12:19:00 oops 12:19:02 it compiles for me! 12:20:11 advil: I think wrapping only ever worked for pgdn; it's not a new change 12:20:21 <|amethyst> ebering: it does? 12:20:25 ah ok 12:20:31 <|amethyst> ebering: check the first if in _pain_weapon_user 12:20:43 <|amethyst> ebering: all that following logic applies only to spectral weapon 12:21:56 oh 12:22:37 ok that makes more sense 12:30:41 03advil02 07* 0.22-a0-862-g98e63d6: Set expand_v on memorize menu 10(13 minutes ago, 1 file, 1+ 0-) 13https://github.com/crawl/crawl/commit/98e63d6bbd13 12:30:41 03advil02 07* 0.22-a0-863-g005b4bd: Make _sort_mem_spells arugments const 10(31 seconds ago, 1 file, 8+ 7-) 13https://github.com/crawl/crawl/commit/005b4bd22859 12:31:04 arugments 12:31:34 <|amethyst> urugments 12:33:51 wouldn't that be fuzzing? 12:33:58 Igyb shouts: "Dungeon const! All const!" 12:34:13 *Ijyb 12:34:24 lol 12:48:08 advil: your expand_v change looks like it'll affect local tiles as well; is that intentional? 12:48:18 yes 12:48:50 it would look better on all platforms to keep the window size fixed as search changes, and that commit was the quickest way to do so on the two that it works on 12:49:11 it might produce oddly large windows sometimes but I think fixing that is a further refinement 12:49:14 ah; I'm working on a number of UI tweaks, one of which is moving local tiles menus to the top-centre 12:49:42 the status line is on the bottom so I want to keep height fixed 12:49:47 not just position 12:50:08 it doesn't look as bad on local tiles to not keep it fixed, but it looks better to do so 12:51:33 a more refined solution for local tiles (not console) might do something like set the max window height when the menu is first populated, maybe with a but of extra padding if it's not maxed, but I wasn't quite sure what the proper way to do that would be 12:51:46 s/but/bit 12:54:30 that wouldn't work, because some users have spells hidden by default as they're collected, so the initial menu is (mostly) empty 12:55:31 oh right it does expand 12:55:42 I was forgetting about that 12:56:56 it looked basically fine for me as-is with a mostly empty spell menu; another possible tweak would be to just use 2/3 of the space for large screens instead of all the space, but that I also didn't know how to do (seems harder, would need to not do that below a certain screen size I think) 13:01:50 there's also the switcher widget, which is very handy for that sort of thing 13:02:40 it switches between showing any single child at once, but requests enough size to show the largest of them 13:02:45 it's used on e.g. the ^ screen so the popup doesn't resize when switching between panes 13:02:52 ah, yeah at some point converting the ! thing to that would be good 13:03:28 might wait until 0.23 for that 13:04:14 btw I really appreciate how much easier it has been to tweak this menu than it would be have been before the ui changes 13:10:23 Unstable branch on crawl.jorgrun.rocks updated to: 0.22-a0-863-g005b4bd228 (34) 13:12:52 :) glad to hear that 13:18:52 Unstable branch on crawl.akrasiac.org updated to: 0.22-a0-863-g005b4bd (34) 13:20:19 do we have any sort of escaping function for formatted_string codes? 13:22:32 replace_all(foo, "<", "<<") 13:22:36 The build was fixed. (master - 005b4bd #10013 : advil): https://travis-ci.org/crawl/crawl/builds/409017777 13:25:55 thx 13:27:18 aidanh: I can now activate wizard commands from &? 13:27:28 yep, should work 13:27:28 however, it parses my arrow key as a command and I can't scroll :/ 13:27:46 oh, that's no good 13:28:19 !tell gammafunk oh yeah, i did take a peek at that doc when you linked it the other day, it reminded me of a couple of uniques i was surprised still existed (one being harold so the recent changes to him look good!) 13:28:20 MarvinPA: You have 2 messages. Use !messages to read them. 13:28:20 MarvinPA: OK, I'll let gammafunk know. 13:30:48 !tell gammafunk but in terms of jorgrun i'm not sure - late D/early lair branches with shatter gone is the first thing that comes to mind 13:30:49 MarvinPA: OK, I'll let gammafunk know. 13:32:11 !tell gammafunk frances was one of the others that jumped out to me as being not very notable, so he could maybe replace her there 13:32:12 MarvinPA: OK, I'll let gammafunk know. 14:22:29 aidanh: on local tiles, when the number of items is 0 and m_ui.scroller->expand_v = true, update_entries doesn't seem to do anything, any ideas? 14:22:39 console is fine 14:22:43 haven't tested webtiles 14:25:25 in current trunk you should be able to replicate this by searching for a non-matching string in the memorize menu in local tiles 14:32:16 -!- Bammboobies is now known as Bammboo 15:04:28 -!- amalloy_ is now known as amalloy 15:10:31 advil: might be the command repeat code that was changed recently; I remember debugging something like that before 15:26:24 !seen Lasty 15:26:24 I last saw Lasty at Wed Jul 25 02:23:54 2018 UTC (2d 17h 2m 30s ago) quitting, saying 'Quit: Leaving.'. 16:00:37 -!- aidanh_ is now known as aidanh 16:19:35 huh, I hadn't noticed before but it does update when you move the mouse over the window 17:10:41 03advil02 07* 0.22-a0-864-g692e097: Rework memorize menu layout 10(3 hours ago, 1 file, 34+ 27-) 13https://github.com/crawl/crawl/commit/692e0977ac3c 17:10:41 03advil02 07* 0.22-a0-865-g16ba858: Ensure that clearing a menu triggers a layout 10(10 minutes ago, 2 files, 2+ 1-) 13https://github.com/crawl/crawl/commit/16ba858c1f63 17:11:29 aidanh: you may want to glance at 16ba858, I found the bug well enough but I'm not sure the fix is done in the proper way for the API 17:28:41 03advil02 07* 0.22-a0-866-geee5034: Further memorization layout tweaks 10(77 seconds ago, 1 file, 8+ 7-) 13https://github.com/crawl/crawl/commit/eee50340df1e 18:10:21 Unstable branch on crawl.jorgrun.rocks updated to: 0.22-a0-866-geee50340df (34) 18:22:14 Unstable branch on underhound.eu updated to: 0.22-a0-866-geee50340df (34) 20:03:56 -!- ProzacElf_ is now known as ProzacElf 20:30:51 -!- ProzacElf_ is now known as ProzacElf 21:22:38 hot new pr commin at you 21:22:58 might be a bit big and you might not want to merge all of it (I still believe in Harm Morg) 21:27:07 New branch created: pull/828 (5 commits) 13https://github.com/crawl/crawl/pull/828 21:27:08 03ebering02 07https://github.com/crawl/crawl/pull/828 * 0.22-a0-862-g19e7351: Allow SH specification for unrands. 10(8 hours ago, 1 file, 2+ 1-) 13https://github.com/crawl/crawl/commit/19e735185926 21:27:08 03ebering02 07https://github.com/crawl/crawl/pull/828 * 0.22-a0-863-g8c537b4: Allow tweaking more artps in wizmode 10(8 hours ago, 1 file, 12+ 7-) 13https://github.com/crawl/crawl/commit/8c537b41aa26 21:27:08 03ebering02 07https://github.com/crawl/crawl/pull/828 * 0.22-a0-864-g7fd23a0: Revise Morg (mikee) 10(6 hours ago, 4 files, 16+ 6-) 13https://github.com/crawl/crawl/commit/7fd23a0426e4 21:27:08 03ebering02 07https://github.com/crawl/crawl/pull/828 * 0.22-a0-865-g4786ac1: Revise the Spriggan's Knife (mikee) 10(5 hours ago, 3 files, 12+ 6-) 13https://github.com/crawl/crawl/commit/4786ac1b58a1 21:27:08 03ebering02 07https://github.com/crawl/crawl/pull/828 * 0.22-a0-866-gd4bd2a7: Revise the Singing Sword 10(60 minutes ago, 2 files, 32+ 24-) 13https://github.com/crawl/crawl/commit/d4bd2a7a3217 22:39:19 yess better unrands 22:47:37 <|amethyst> ebering: huh, just noticed that ARTP_DRAIN and AMU_HARM have a slight difference in messaging (the former gives a message even if you fully resist) 22:49:01 <|amethyst> ebering: oh, but I guess the amulet has its own message separate from the draining message 22:50:05 <|amethyst> hm, but it seems like the amulet behaviour is better, because the ARTP I guess just gives "You resist." 22:51:05 <|amethyst> could make _remove_amulet_of_harm take an item (or just a string) to use as the subject, and rename it appropriately 23:40:22 |amethyst: ARTP doesn't just give you "You resist." because it's irresistable 23:42:23 but, harm does give you an extra message and not just "You feel drained.", which could be abstracted and used for ARTP_DRAIN and Harm 23:53:34 hm. my cao trunk game appears to be in a sad place: when I try to play it says another game is using the save 23:53:40 however, there are no running games for me 23:55:24 probably the process is running with no websocket 23:55:37 should time out for input and then get killed