Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add an inventory system to the game #9

Draft
wants to merge 29 commits into
base: main
Choose a base branch
from
Draft

Conversation

https123456789
Copy link
Member

@https123456789 https123456789 commented Aug 24, 2022

I'm just making this so we can track the progress of the system's development.

Linting status:
flake8 Lint

@https123456789 https123456789 added the enhancement New feature or request label Aug 24, 2022
@https123456789 https123456789 added this to the Inventory System milestone Aug 24, 2022
@https123456789 https123456789 self-assigned this Aug 24, 2022
@https123456789
Copy link
Member Author

Progress Update

I have redone the inventory system to have a less complicated design which should make development of new items easier.

The next things I plan to work on are:

  1. Add the ability to remove items from an inventory
  2. Add the ability to serialize/deserialize both an entire inventory and individual items.

The player can now change their equipped weapon in the inventory overlay by clicking on the item.
Add a bit of text just below the item to let the user know that the item is equipped.
@https123456789
Copy link
Member Author

https123456789 commented Dec 20, 2022

Progress Update:

Note:
✔️ -> In progress
✅ -> Done
❌ -> Not started

While I want to finish off the inventory system, I think getting a functional UI for the system is also important so I am going to (most likely) devote all of today working on things that are not directly related to the inventory system. The things I plan on working on include:

  • ✅ Add a tool-tip when the user hovers over an item, displaying more information about the item. This may require a few tweaks to the inventory system, but they should be minor.
  • ✅ Sprucing up the individual item slots to fit the theme of the game a bit better.
  • ✅ Add some more enemies to the boss level for the first floor to make it harder.

Once I get back to working on the actual system, I will probably work on these things:

  • ✅ Add a better method for query information about a particular inventory item.
  • ✔️ Add a very rudimentary chest to see how the current design of the inventory system can be applied to non-player inventories. I probably will change the system to make it more of a generally applicable system.
  • ❌ Add a way to transfer items to another inventory instance.
  • ❌ Add a better method for removing items from the inventory.
  • ✅ Add methods to serialize and deserialize an instance.

This will most certainly span many days, but I plan to get them all done. I'll update the post as I go.

Add a simple tooltip to the inventory overlay. An improved version of the tooltip can be created once some addition methods are added to the inventory system API to simplify access to the inventory registry.
@https123456789 https123456789 linked an issue Dec 21, 2022 that may be closed by this pull request
@https123456789 https123456789 linked an issue Dec 21, 2022 that may be closed by this pull request
https123456789 and others added 5 commits December 21, 2022 16:22
Added these things:
- Item descriptions to the inventory tooltips.
- Removed random print statements.
- Changed the filename of the game's save file.
- Added the game's save file to the .gitignore
Added inventory serializations methods.
Added the basics of a simple chest
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PEP8 Compliance For All Overlays Add serialization for inventories
2 participants