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

Seamless windows #7928

Merged
merged 15 commits into from
Mar 31, 2022
Merged

Seamless windows #7928

merged 15 commits into from
Mar 31, 2022

Conversation

Laboredih123
Copy link
Contributor

@Laboredih123 Laboredih123 commented Mar 23, 2022

About the PR

Makes windows seamless.
Adds a new proc for seaming and seamlessing (By pgmzeta)

Why's this needed?

It looks cool. People liked it.

Images

image
image

Changelog

(u)Laboredih123 & pgmzeta
(+)Nanotrasen has now made windows fully connected, removing seams.

@boring-cyborg boring-cyborg bot added the C-Sprites Automatically applied on any .dmi or icons folder change label Mar 23, 2022
@github-actions github-actions bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Mar 23, 2022
@Laboredih123 Laboredih123 changed the title Seamwindows Seamless windows Mar 23, 2022
@ZeWaka
Copy link
Member

ZeWaka commented Mar 23, 2022

This incorrectly affects standalone windows.

@ZeWaka
Copy link
Member

ZeWaka commented Mar 23, 2022

Also definitely not a major changelog.

@Laboredih123
Copy link
Contributor Author

After some testing, It doesn't seem to affect standalones for me.
image

@Laboredih123
Copy link
Contributor Author

If you mean the single windows between walls, that is how the windows work.
image

@Eagletanker
Copy link
Contributor

Eagletanker commented Mar 23, 2022

If you mean the single windows between walls, that is how the windows work.
image

The right window looks wrong

@Laboredih123
Copy link
Contributor Author

Laboredih123 commented Mar 23, 2022

I can assure you they are the same.
image

@Laboredih123
Copy link
Contributor Author

Although I notice a little seam.

@ZeWaka
Copy link
Member

ZeWaka commented Mar 23, 2022

If you mean the single windows between walls, that is how the windows work.
image

The right window looks wrong

It's because the walls are applying the 'edge' overlays and also not smoothing, while the window is smoothing.

@Laboredih123
Copy link
Contributor Author

Should it be made that walls smooth with windows?

@ZeWaka
Copy link
Member

ZeWaka commented Mar 25, 2022

Should it be made that walls smooth with windows?

it either needs to smooth with the windows or the windows need to not smooth with the walls

@Laboredih123
Copy link
Contributor Author

I find it nicer to smooth with the windows

@Laboredih123
Copy link
Contributor Author

Laboredih123 commented Mar 25, 2022

I'll ask others.

@Laboredih123
Copy link
Contributor Author

So it turns out windows are unable to connect to walls without smoothing.

@Laboredih123
Copy link
Contributor Author

image
This looks quite nice, other than the fact it looks like the walls have a depression on top.

@pali6
Copy link
Member

pali6 commented Mar 25, 2022

I prefer having the connector overlay on walls.

@Laboredih123
Copy link
Contributor Author

People prefer the overlay over the without.

@ZeWaka
Copy link
Member

ZeWaka commented Mar 27, 2022

Q: Why aren't we just applying the icon smoothing we use for walls, with the 40 states or whatever? Then we don't get window holes in 2x2 window cubes.

@pgmzeta
Copy link
Contributor

pgmzeta commented Mar 27, 2022

Q: Why aren't we just applying the icon smoothing we use for walls, with the 40 states or whatever? Then we don't get window holes in 2x2 window cubes.

Do we have a tool to generate the sprites for this? right now it's using icon_states in a dmi for each cardinal x ordinal combination:
image

If we have sprites for those, it's a very small lift to make sure we can use them with the refactoring changes in flight.

@Laboredih123
Copy link
Contributor Author

I believe this would be a good addition to the pr. What do ya'll think? Laboredih123#1

@ZeWaka
Copy link
Member

ZeWaka commented Mar 27, 2022

Q: Why aren't we just applying the icon smoothing we use for walls, with the 40 states or whatever? Then we don't get window holes in 2x2 window cubes.

Do we have a tool to generate the sprites for this?

Yes. https://github.com/tgstation/icon-cutter

@github-actions github-actions bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Mar 31, 2022
code/obj/table.dm Outdated Show resolved Hide resolved
@pali6 pali6 merged commit a634a44 into goonstation:master Mar 31, 2022
github-actions bot pushed a commit that referenced this pull request Mar 31, 2022
@Laboredih123 Laboredih123 deleted the seamlesswindows branch April 2, 2022 05:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Sprites Automatically applied on any .dmi or icons folder change size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants