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

Retroday outline #36402

Closed
wants to merge 22 commits into from
Closed

Retroday outline #36402

wants to merge 22 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Dec 24, 2019

Summary

SUMMARY: Interface "add transparency to retrodays"

Purpose of change

retrodays tileset is my favourite but, all the tiles have a black background thus rendering all objects like a black square with the object drawn on that.
the problem is, when you have and object on top of a larger one, like a lighter on a table,
the lighter gfx and its black square background, prevent you to see the table underneath.
it make it look like the table is missing a whole chunk. to the point that if there is few objects (3 or 4) on the table you cannot distinguish there's a table at all.

so i removed the black squares, and left instead a black outline around the contours of the objects.
other tilesets have transparency, so i made it for retrodays too.

Describe the solution

i removed black squares around tiles, and left a black outline instead.

Testing

i have been playing with that for quite some time.

Additional context

in general look for objects on top of bigger objects, like the tables and sofas:
book_desc

before:
retrodays-before

after:
retrodays-after

tiles:
Untitled

@ghost
Copy link
Author

ghost commented Dec 24, 2019

oh, it seems i mixed the git commit ?
that's stran

@ghost ghost closed this Dec 24, 2019
@ghost ghost deleted the retroday_outline branch December 24, 2019 12:39
This pull request was closed.
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.

1 participant