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 preview for building floors #107

Merged
merged 1 commit into from
Aug 17, 2016
Merged

Add preview for building floors #107

merged 1 commit into from
Aug 17, 2016

Conversation

SquareMan
Copy link
Contributor

@SquareMan SquareMan commented Aug 17, 2016

Since floors require the player to build them I decided to add in the preview sprite. Unfortunately since the job system isn't fleshed out yet, the approach is a bit hackish. However, it works perfectly fine and can easily be updated once the job system is redone (specifically the job sprite system)

@vogonistic
Copy link
Collaborator

Can you add a screenshot?

@SquareMan
Copy link
Contributor Author

It's just normal "greenified" version of the normal sprite. Give me a minute and I'll get a screenshot.

@SquareMan
Copy link
Contributor Author

SquareMan commented Aug 17, 2016

floor sprite preview

It's a little darker than I would personally like, but until there is a better, more customizable system for handling preview sprites it will have to do. Definitely better than no sprite at all.

@Tranberry
Copy link
Contributor

Might be a number of different reasons (nightmode, laptop screen etc..) but I can't see any difference.

@vogonistic
Copy link
Collaborator

I assume it's more visible while running it locally. Definitely agree that it's better than no sprite at all. 👍

@quill18 quill18 merged commit 329271b into TeamPorcupine:master Aug 17, 2016
@SquareMan SquareMan deleted the Add-preview-for-building-floors branch August 17, 2016 23:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants