Page 1 of 1

TMWA Dye System Revisted. [DIS]

Posted: 04 Feb 2015, 22:38
by wushin
Selim the dyer, hairstles, barbers, etc. generally cause text clutter in code. It could be much much much much more concise.

We have in no particular order:
  • TMWA needs an edit to adopt manaplus Client-data color dye system
  • TMWA needs Own color method and someone makes a PR to Manaplus+
  • TMWA scripts the item color via formula & auto-generated data.
Please keep topics on discussion of Item (sprites; hairstyles, mobs, equipments) Dyes (no tileset dye discussion here please, thats a different thread)

Re: TMWA Dye System Revisted. [DIS]

Posted: 05 Feb 2015, 02:38
by o11c
Attack plan:
  • Add the `color` field to the internal stuff and send it in the packets.
  • Add a temporary GM command that switches some item's dye temporarily.
  • Add palettes to items.xml
  • Pray.
  • Curse.
  • Make sure the `color` field is saved/loaded in all savefiles correctly (IIRC previously it was not for all subfields, but I'm not sure which).
  • Refactor all the code that needs item keys.
  • Make all items stack; if any scripts are still broken by now-full inventory on delete of previous stackable, screw them, it will just drop on the ground.
  • Migrate the item IDs (how?). Should I also autostack or let that wait?
If option A fails (see step 5), then we'll have to do option C, which is basically "option A internally but current status quo externally".