voxel / ideas

issue tracker for new voxel.js plugin ideas
4 stars 1 forks source link

[voxel-inventory-dialog] Generic player inventory transfer dialog #8

Closed deathcap closed 10 years ago

deathcap commented 10 years ago

Currently, the transferrable (linked) 'player inventory' GUI dialog is duplicated across several plugins:

https://github.com/deathcap/voxel-inventory-dialog https://github.com/deathcap/voxel-furnace https://github.com/deathcap/voxel-workbench

could be useful if this code was all factored out into a reusable module. Most other machine GUIs would make use of this same UI.

deathcap commented 10 years ago

Maybe change https://github.com/deathcap/voxel-inventory-dialog to include only the player inventory, have other plugins reuse it, then add a new voxel-inventory-crafting plugin for inventory + 2x2 crafting grid to replace the current voxel-inventory-dialog.

deathcap commented 10 years ago

Added new plugin voxel-inventory-crafting, inherits from voxel-inventory-dialog, but still need to refactor voxel-furnace, voxel-workbench, and clean up

deathcap commented 10 years ago

Complete; reusable base class is now: https://github.com/deathcap/voxel-inventory-dialog