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

Adds a perspective map editor walls define to _build.dm #14355

Merged

Conversation

DisturbHerb
Copy link
Contributor

@DisturbHerb DisturbHerb commented Jun 3, 2023

[MAPPING][INPUT]

About the PR

It is now possible to define which wall icons are used in a map editor by uncommenting the USE_PERSPECTIVE_EDITOR_WALLS define in _std/_build.dm. By default, flat wall and wingrille spawner icons are used.

Why's this needed?

As someone who occasionally uses a map editor, I've found the recent default wall icon changes to, instead of improving the map editing experience, actively hinders it. The icons have significantly more going on in the perspective versions, and the introduction of so many lines and shapes makes reading maps in the editor significantly more difficult than flat icons. This matters as, if you're making large scale changes and need to use SDMM for extended periods, the new icons can cause some eye strain; at least for me.

I think the most convincing way of demonstrating my frustration would be a visual. From looking at the following screenshots, I think that the top one is a little harder to look at, it looks cluttered and the lines constantly break up my focus rather than letting me to simply gloss over them like I could with flat walls.

A section of Cogmap's security foyer with perspective wall icons.
A section of Cogmap's security foyer with flat wall icons.

I think a case can be made for the perspective wingrille spawners, they're a little less intrusive than the flat wingrille spawner icons we had previously.

Of course, my experience is not analagous to everyone's, and I think I'd need more opinions to make sure that this isn't just a me thing. The original PRs were out to solve a problem that does exist -- that of aligning wall-mounted objects properly -- but it hasn't come up regularly enough for me to see the benefits. I myself was personally happy with modifying the base icon state as necessary, or drawing from a palette of modified walls that I already had.

I think a solution that would satisfy everyone would be a native SDMM solution for switching between perspective and flat icons, but there seem to be very difficult barriers to overcome to render perspective wall outside of runtime.

These points have been since superseded by the recent changes made as of 10 June 2023. Now that any given contributor has personal agency over how their editor looks, I believe we have come to a more satisfactory solution.

@boring-cyborg boring-cyborg bot added the C-Sprites Automatically applied on any .dmi or icons folder change label Jun 3, 2023
@keywordlabeler keywordlabeler bot added A-Mapping A mapping change E-Input-Wanted Input and feedback are wanted. Also posts a discussion thread on the forums. labels Jun 3, 2023
@github-actions github-actions bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jun 3, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Jun 3, 2023

Created discussion thread: https://forum.ss13.co/showthread.php?tid=21301

@DisturbHerb DisturbHerb changed the title Revert perspective map editor icons Reverts perspective map editor icons Jun 3, 2023
@DisturbHerb
Copy link
Contributor Author

n.b. Looking at it again, reinforced and regular perspective wingrille spawners aren't very well distinguished from each other in my opinion.

@Laboredih123
Copy link
Contributor

perhaps a good idea would be to make custom simplistic icons to show where the lines are on the walls and windows?

@CheffieGithub
Copy link
Contributor

I personally prefer the perspective icons. Though im bias since I was the one who changed them. I do think the best solution to leave the icons as is or create simpler icons in perspective as Laboredih says.
Ill PR a fix for the wingrille spawner colours, I neglected to add in the colour difference between reinforced and not.

@DisturbHerb
Copy link
Contributor Author

I personally prefer the perspective icons. Though im bias since I was the one who changed them. I do think the best solution to leave the icons as is or create simpler icons in perspective as Laboredih says.

What worries me most is the inability to pick what works best for you when working in the editor. Removing either option outright's gonna leave someone dissatisfied at the end of the day. I personally think that, if you really do prefer using perspective walls everywhere for the purpose of arranging wall-mounted items, then that's your perogative; you could just make a prefab in SDMM with your desired icon state and just run with it. However, I question the need for it to be imposed on everyone for every possible situation, and the removal of the flat icons has caused me some trouble when working in the editor.

@CheffieGithub
Copy link
Contributor

CheffieGithub commented Jun 3, 2023

I think most people would find perspective icons more convenient but more voices on this matter are needed.

I made the changes not because I prefer them myself but because perspective is the standard for sprites and it made sense for the map editor to mirror how it would look in-game. The old sprites were just bugs from having no real "map" iconstate and it making a default one.

I also asked for opinions before I made the PR to get an idea of how it would be percieved, but there wasn't any strong opinions eitherway aside from BatElite. Personally I don't really mind eitherway but I do think perspective makes more sense.

@CheffieGithub
Copy link
Contributor

I also think if you do go down this route you should make every wall not perspective to avoid inconsistencies

@Laboredih123
Copy link
Contributor

Laboredih123 commented Jun 3, 2023

What worries me most is the inability to pick what works best for you when working in the editor.

Hm not exactly. Could have a define in build.dm letting us switch between the two tilesets for the map editor. Wouldn't be so hard too since Byond lets us use booleans in macro ifs. We can't do it on the go as you would need to reopen the environment every time you switch, but this is next best no?

@DisturbHerb DisturbHerb added the S-Controversial This requires a heightened standard of review before it can be merged label Jun 4, 2023
@DisturbHerb
Copy link
Contributor Author

Hm not exactly. Could have a define in build.dm letting us switch between the two tilesets for the map editor. Wouldn't be so hard too since Byond lets us use booleans in macro ifs. We can't do it on the go as you would need to reopen the environment every time you switch, but this is next best no?

This is a better idea, I'll give it a try.

@BatElite
Copy link
Contributor

BatElite commented Jun 4, 2023

To me it comes down to noise, and the perspective graphics are considerably noisier and have more internal contrast than the flat icons so I do prefer the old sprites.

@Sord213
Copy link
Contributor

Sord213 commented Jun 5, 2023

Yeah I'm not sure if it's just "new thing bad" but man I don't like the perspective sprites in editor.

@github-actions github-actions bot added the S-Merge-Conflict Applied and removed when a PR has or no longer has a merge conflict label Jun 6, 2023
@github-actions github-actions bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jun 10, 2023
@DisturbHerb DisturbHerb changed the title Reverts perspective map editor icons Adds a perspective map editor walls define to _build.dm Jun 10, 2023
@DisturbHerb
Copy link
Contributor Author

i ate @Laboredih123's idea yummy yummy

@github-actions github-actions bot removed the S-Merge-Conflict Applied and removed when a PR has or no longer has a merge conflict label Jun 10, 2023
@Laboredih123
Copy link
Contributor

darn

@DisturbHerb DisturbHerb requested a review from Sord213 June 13, 2023 06:51
@TobleroneSwordfish TobleroneSwordfish merged commit 0ee20dc into goonstation:master Jun 25, 2023
21 checks passed
@DisturbHerb DisturbHerb deleted the revert-perspective-icons branch June 25, 2023 23:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Mapping A mapping change C-Sprites Automatically applied on any .dmi or icons folder change E-Input-Wanted Input and feedback are wanted. Also posts a discussion thread on the forums. S-Controversial This requires a heightened standard of review before it can be merged 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.

None yet

7 participants