search

GlowstoneMC / Glowstone-Legacy

An open-source server for the Bukkit Minecraft modding interface
Other
363 stars 122 forks source link

Inventory and Crafting #1

Open SpaceManiac opened 13 years ago

SpaceManiac commented 13 years ago

Basic inventory is already implemented. Get crafting working as well as smelting and chests.

Edit by turt2live (thanks to @dequis): GUIs:

Other issues:

SpaceManiac commented 13 years ago

CraftingManager is done a lot of the way. Finishing it and adding the ability to craft in the inventory are next up.

SpaceManiac commented 13 years ago

CraftingManager is now completely done aside from any bugs that may be lurking and a tiny amount of testing code. Just need to get inventory support for it working. Workbench support will likely come later.

dequis commented 10 years ago

Maybe update title/description of this one to mention it's about block GUIs in general?

dequis commented 9 years ago

Since this ticket is mostly for inventories and GUIs, I think it would make sense to leave out of the scope of this ticket features such as smelting, and make separate tickets for them, because each one of those GUIs implements a rather large feature set.

turt2live commented 9 years ago

I see no problem in referencing this ticket for every inventory change, meaning the PRs would be limited in scope. There may be 3 PRs (workbench, furnace, and enchanting), for example, that all reference this ticket and complete a component of it.

dequis commented 9 years ago

Ehhh well. I'm just saying we should leave out implementations of smelting, brewing, enchanting, repairing, beacon effects, etc. Dispenser/hopper should definitely be out, since redstone and entities. Horse/villager are entities too. And so on.

If we add all that stuff to this ticket we're going to have it open for a very long while, covering pretty much all of the "game completeness" milestone.

turt2live commented 9 years ago

It doesn't really matter to me personally. I do prefer a fewer number of tickets, but I can survive with 20 extra ones.

dequis commented 9 years ago 
< dx> let's start a bounty for the brave soul that fixes window click handling completely
<@turt2live> I'll throw in $10 via PayPal
<@turt2live> Can quote me on that.
<@turt2live> Whoever fixes all of window clicking: $10.
turt2live commented 9 years ago

I can confirm I said that

On Sun, Oct 19, 2014 at 10:21 PM, dx notifications@github.com wrote:

01:20 < dx> let's start a bounty for the brave soul that fixes window click handling completely 01:21 @turt2live I'll throw in $10 via PayPal 01:21 @turt2live Can quote me on that. 01:21 @turt2live Whoever fixes all of window clicking: $10.

— Reply to this email directly or view it on GitHub https://github.com/GlowstoneMC/Glowstone/issues/1#issuecomment-59682304.

Sincerely, Travis R

(Turt2Live http://turt2live.com/)

mastercoms commented 9 years ago

I believe we've fixed window click handling completely, in Glowstone++. I'll try to figure out how to make a pull request with our changes.

@deathcap merged #484, and I fixed a bunch of bugs related to it.

mastercoms commented 8 years ago

Glowstone++ has finished crafting, and has added dispensers/droppers, double chests, enchantment tables and furnaces.

Beacons, horse, and villager inventories are missing.