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

Proper and visually nice asset management #44

Open
3 of 7 tasks
Kruptein opened this issue Jul 6, 2018 · 3 comments
Open
3 of 7 tasks

Proper and visually nice asset management #44

Kruptein opened this issue Jul 6, 2018 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@Kruptein
Copy link
Owner

Kruptein commented Jul 6, 2018

Asset management is currently very barebones and frankly just horrible albeit functional.

Desired changes

  • Adding/Moving/Removing assets should all be possible from inside the client, whereas this currently is only possible from the filesystem.
  • The asset menu should be completely overhauled in regards to its visuals and usability
  • Ability to filter your assets
  • Ability to favourite/lock assets that are frequently used within a session
  • Proper player characters that appear to players and can be modified by them
  • Edit asset information to provide defaults
    • e.g. A size field, so that if a large monster is dragged on the field it will automatically assume that
    • e.g. default vision for monsters
  • Sharing system
    • Hard copy: changes are reflected in both directions (e.g. player token shared with DM)
    • Soft copy: changes are not reflected. (e.g. monster tokens shared among DMs)
      size.
@Kruptein Kruptein added this to the 1.0 milestone Jul 6, 2018
@Kruptein Kruptein added the enhancement New feature or request label Jul 6, 2018
@Kruptein Kruptein self-assigned this Jul 6, 2018
@Kruptein
Copy link
Owner Author

Kruptein commented Aug 1, 2018

Work has started on overhauling the asset management system.

In commit 6df4cc0 the first steps have been taken to change the appearance of the asset list in game.

An out of game interface will be created to do more advanced asset management changes like creating folders, moving assets, giving assets metadata (e.g. size, HP, AC, ...).
No in game support will be provided for these things initially as they are typically only needed during preparation and not during actual gameplay which means these features would only add clutter to the game interface. A button to open the asset management interface in a new tab/window should be readily available though.

Another big change will be how assets are actually stored on disk. The current implementation simply mimics the file directory under /static/img/assets/ and is public for all users.
The current plan is to change this making it so that the assets folder will be a flat file directory with all assets (with their names renamed to their hash). The game server should maintain a datastructure that is able to associate folder hierarchies and matching files with the actual files on disk, using hashes to relate them.
This allows for a flexible system in which file separation/permissions/sharing are all easy to implement all while not wasting disk space to duplicate files.

@Kruptein
Copy link
Owner Author

Kruptein commented Aug 5, 2018

The assetmgmt branch now uses the earlier proposed system to keep track of assets, whereby all data is stored in a flat directory (currently /static/assets/ and each user having a personal directory/file structure that links to the actual image along with metadata.

The in-game assets list gained a small icon to open a full asset manager in a new tab. In this asset manager some common operations from a file browser are mimicked like: creating directories, renaming, removing and moving folders/files as well as uploading files from disk.

This means users are now able to upload images themselves and no longer have to rely on the server owner to put the images in the correct folder.

This also means that you can no longer access your assets through command line or file browser as all actions have to go through the new asset manager

@Kruptein
Copy link
Owner Author

This ticket needs to be split up in smaller digestible parts.

@Kruptein Kruptein removed this from the 1.0 milestone Mar 16, 2020
@dthv dthv mentioned this issue Dec 28, 2020
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

No branches or pull requests

1 participant