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 help label about creating multiple/big tiles #79904

Conversation

KoBeWi
Copy link
Member

@KoBeWi KoBeWi commented Jul 25, 2023

image

@kleonc
Copy link
Member

kleonc commented Jul 26, 2023

Have you tried/considered aligning the hints vertically? Like in the 2D editor:
BT9WSCueaF
Seems easier to skim through them.

@KoBeWi
Copy link
Member Author

KoBeWi commented Jul 26, 2023

idk, doesn't it take more space?
image
The editor is more horizontal than vertical.

@kleonc
Copy link
Member

kleonc commented Jul 26, 2023

idk, doesn't it take more space?

Seems like it does, but spacing between the lines is way bigger compared to the 2D editor though. Also now looking at it I wonder if dimming these hints in a similar manner would make sense. OTOH I do remember pointing some user at the hints in 2D editor when answering some how-to, and the user mentioned they've never noticed these hints. So not really sure if dimming is good. 🙃
Horizontally they look strange to me, that's all, wanted to point it out (I don't know GUI design principles or something like though).

@YuriSizov YuriSizov merged commit cd074f9 into godotengine:master Jul 27, 2023
13 checks passed
@YuriSizov
Copy link
Contributor

Thanks! Let's go with what we have for now. Can always move things around later. I agree with kleonc that it does look a bit weird, just floating in there. But it's not a big deal for now. If we get more hints in the future, then having them vertically would probably be better.

@KoBeWi KoBeWi deleted the how_to_create_huge_tiles_using_a_simple_trick branch July 27, 2023 13:39
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.

4 participants