Search results

:) While I have no idea what this does, I took a stab at fixing it because I was already messing with CharPaneDecorator.java. Attached patch is against r10887 and builds/runs/fixes the javascript error.

Another feature suggest by a friend that I thought would be nifty and whipped up - changing the class listed in the character pane in the relay browser to add a link to the appropriate wiki page for the class. Patch is against r10887, builds/runs/works. Minor, but has been helpful for me as I...

per the wiki, indeed he does. added init as well, while I was at it. patch is against r10855. :) Probably faster to just make the changes than it would be to apply my patch, though...

Any chance one of the developer folk could take a look at this? I apologize for the impatience on my part, but the suspense is killing me. :) Worst case, I'll take a stab at finishing this by moving the new URLLabel to it's own java file in the net/java/dev/spellcast/utilities/ library, and...

I'm not familiar with moods/shelves, so I will go away and do more research before commenting. Is this the "Mood Setup" panel that you're referring to, or some other part of the UI I'm unfamiliar with? dave

So to play it back to make sure I'm on the same page (and then I'll go whip up a patch) - kill separator. check. :) - if mrulength = 0, expression pulldown works as a static list (of 34 items, fwiw). - if mrulength <= 34 (:) I counted), then the expression pulldown displays "mrulength" items...

The intention is to solve for the causal user who isn't putting in their own expressions, they're just picking from the pulldown - to have the out of the box behavior "just work" for everyone. With the existing behavior, you either a. have the mrulist turned off, and then have to either scroll...

I did clear out the maximizerMRUList - that was the intention. With an empty MRU list, it behaves as if you had maximizerMRULength set to zero - it just displays the default list of expressions. If the MRUList was (or is) left as it was before my patch, i.e., with the same list as the default...

Attached patch improves (at least, from my perspective) how the maximizer pulldown works, and specifically the maximizer "most recently used" preferences. Before, if you had maximizerMRULength set to non-zero, you'd get X number of most recently used expressions in the list, but the defaults...

Per GValko's excellent suggestion above, a patch to add a decorator for when you get the Stone-Faced effect from wool, that links back to the hidden temple. Patch is against r10837, builds/runs/seems to work... :) Link shows up, but I'm drunk, so I can't click on it.

Fixed up some monster/item stats based on the wiki, added the choiceadventures to the manager except for the hidden heart of the hidden temple adventures, since that requires some more custom work. Not done by any means, but here's some progress. Wasn't sure if these should be choiceadventures...

Alright! well, that was vastly more complicated than I expected, but there you go. So, I've got a working patch for this (see attached). The patch is definitely not ready for prime-time - there's a bunch of hacky bits just to get it working. But, given my serious lack of java experience, I...

:) I know you already committed it, but I did have a chance today to test, and indeed it works as advertised for the Knob Shaft clancy adventure. So, that's good.

Okay, so it looks like there's three parts to this: - part one (patch attached) - change the name to "Typical Tavern 511" (or the like), i.e., append the choice adventure number to the name. This was simple, but I'm concerned - from what I can tell, there are two places that uses...

I'm going to take a stab at this. There seem to be a variety of different tradeoffs here, all with different downsides, which of course is likely why no one's made any changes to the existing setup. :) I can see two broad choices - either first, including the name of the adventure on the same...

In the main avatar of boris thread, there is a request to have the clancy adventures auto-stop: http://kolmafia.us/showthread.php?9205-New-Challenge-Path-Avatar-of-Boris/page12 Made sense to me, so I did. The patch is against svn head, r10822. I think it's right, but unfortunately I'm not at a...

Well, I could go down two routes - convert the healing counter to a float, or change it to two values, min and max, to allow for a more "true" representation of the healing - and then the appropriate changes to the actual recovery algs to match. Both of which are a *tad* more invasive, and thus...

This is an incredibly minor bug... but it was an easy fix, so off I went. Currently when doing the NS familiar fights in axecore, you get the warning messages about what familiar you need to equip. In axecore, no familiars needed, but you still get the warning, and I fixed it. Patch is against...

I hope I'm not being impatient, but... *bump*. Any feedback? :) My hope is this is an easy one, as it's an incredibly simple patch. If there's a better/preferred way to do this, let me know, I'm happy to redo it appropriately.

stdoodle: :) Exactly why I tried, figured someone was going to ask. xkiv: shrug. happy to rework the patch further if it makes sense, I just noticed the STP with the insult and figured I'd fix it.