Opened 11 years ago

Closed 11 years ago

#823 closed task (fixed (in master))

Include brands on rings in various places

Reported by: magnate Owned by: magnate
Milestone: 3.1.1 beta Keywords:
Cc:

Description

  • randart generation (add_ability() and parse_frequencies())
  • decribe_combat()
  • the description of the ring itself

Change History (3)

comment:1 Changed 11 years ago by magnate

  • Status changed from new to assigned

I have a working version of randart.c that assigns brands and slays to nonweapons and launchers. If we want to restrict this specifically to brands on rings, I need to create a new ART_IDX category or two, as there are currently no other abilities which are found on jewelry but not on armour.

comment:2 Changed 11 years ago by magnate

[2c5d85e] (SVN r1326) mostly sorts this out - brands are available on all nonweapon randarts, not just rings, but they're extremely rare, so hope it is ok. Now I need to find a branded ring to check that the description and weapon description work. It would be nice if the weapon description noted when a brand came from off-weapon.

comment:3 Changed 11 years ago by magnate

  • Resolution set to fixed
  • Status changed from assigned to closed

[3d9d21e] (SVN r1327) and [0150f5e] (SVN r1328)

Note: See TracTickets for help on using tickets.