|Anonymous | Login | Signup for a new account||2019-08-24 07:12 CEST|
|Main | My View | View Issues | Change Log | Wiki | Tavern | News|
|Viewing Issue Simple Details|
|ID||Category||Severity||Reproducibility||Date Submitted||Last Update|
|0007171||[DCSS] Patches||minor||N/A||2013-06-07 02:40||2013-12-11 09:32|
|Status||closed||Product Branch||0.13 ancient branch|
|Summary||0007171: Allow potions/cards of experience train restricted skills|
Allow potions/cards of experience train restricted skills. These are magical items and shouldn't require you to have the right type of item in your inventory.
Potentially invocations and/or spell schools should still be restricted.
Interface clutter is also an issue, but I think it's bearable.
0001-Allow-potions-cards-of-exp-train-restricted-skills.patch [^] (2,159 bytes) 2013-06-07 02:40 [Show Content]
exp-hide-untrainable.diff [^] (2,043 bytes) 2013-09-12 22:00 [Show Content]
|In trunk (0.13-a0-1481-g48b0c40), thanks!|
|elliptic has suggested hiding the untrainable skills by default, so that (for example) Nemelexites don't select Invocations under the mistaken impression that it is useful. * would display and allow you to select everything, as it does in the reskill-to menu. I am attaching a patch that implements that; but it's buggy: you can end up with a skill selected but 0% of the XP devoted to it.|
Doesn't sound like a good idea to me: you typically do such things upon returning to your stash, and at that moment your shield is often unworn, weapon unwielded, etc. Blocking a huge chunk of experience (a potion gives lots early on) just because you didn't suspect it's experience -- or worse, because you were unspoiled or unwary, doesn't make for good gameplay.
Even hiding the skill but allowing selecting it with some magic command wouldn't be good as most players don't read docs.
Thus, I'd leave status quo as is.
This has been in for a while now.
The interface surely could be improved, but no need to keep this report open for that.
|2013-06-07 02:40||Medar||New Issue|
|2013-06-07 02:40||Medar||File Added: 0001-Allow-potions-cards-of-exp-train-restricted-skills.patch|
|2013-06-07 02:41||Medar||Tag Attached: patch|
|2013-06-07 03:17||neil||Note Added: 0023089|
|2013-06-07 03:17||neil||Status||new => resolved|
|2013-06-07 03:17||neil||Fixed in Branch||=> 0.13 development branch|
|2013-06-07 03:17||neil||Resolution||open => done|
|2013-06-07 03:17||neil||Assigned To||=> neil|
|2013-06-07 03:17||Medar||Status||resolved => closed|
|2013-09-12 21:59||neil||Note Added: 0024006|
|2013-09-12 22:00||neil||File Added: exp-hide-untrainable.diff|
|2013-09-12 22:00||neil||Status||closed => new|
|2013-09-12 22:00||neil||Resolution||done => open|
|2013-09-13 06:06||KiloByte||Note Added: 0024009|
|2013-12-11 09:31||Medar||Note Added: 0024684|
|2013-12-11 09:31||Medar||Status||new => resolved|
|2013-12-11 09:31||Medar||Resolution||open => done|
|2013-12-11 09:32||Medar||Status||resolved => closed|
|Mantis 1.1.8[^] Copyright © 2000 - 2009 Mantis Group|