Skip to content

Conversation

Thumperrr
Copy link
Member

  • graphics.json renamed to resources.json
  • graphics.json and board.json moved into a chespp folder.
  • "chesspp" node removed from both json files
  • GraphicsConfig class renamed to ResourceConfig
  • spritePath(s) funcitons renamed to filePath(s)
  • filePath(s) functions no longer append "chesspp" to the beginning of a path

resources.json is now used for any resource that requires a file path (not just sprites).
Thus, GraphicsConfig class and spritePath(s) functions were promptly renamed.

Fixes #67 and part of a discussion in #65

@LB--
Copy link
Member

LB-- commented Dec 28, 2013

Some of these changes are in my resource-refactor branch - looks like I'm going to have to do some manual merging. I think we should wait to merge this until we choose a new resource loading technique.

@Thumperrr
Copy link
Member Author

Agree. Merge conflicts are no fun. This will sit idly for the time being. Actually it might conflict with my own resource manager refactor, so yeah.

@LB-- LB-- mentioned this pull request Dec 28, 2013
@ghost ghost assigned LB-- Dec 28, 2013
LB-- added a commit to LB--/ChessPlusPlus that referenced this pull request Dec 31, 2013
@LB--
Copy link
Member

LB-- commented Dec 31, 2013

I merged this into #72, so this PR should close automatically if #72 is merged.

@LB-- LB-- merged commit f191f41 into cpluspluscom:master Jan 2, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GraphicsConfig spritePath() hidden behavior
2 participants