godotengine / godot

Godot Engine – Multi-platform 2D and 3D game engine
https://godotengine.org
MIT License
89.43k stars 20.25k forks source link

Some odd-looking symbols in script editor #11499

Closed Zylann closed 5 years ago

Zylann commented 7 years ago

Godot 3.0 master

Minor in general, noticed a few characters look a bit odd with the new Hack font / icons:

New: image Old: image

(other than that the font looks fine so far)

Edit: Github shows no images anywhere on my end, I hope you can see them...?

Edit 2: At the moment I'll most likely try another font myself and stick with it. I'm trying to test a TTF but I see no way of setting it up easily in Godot...

bruvzg commented 7 years ago

Indentation marks are icons. https://github.com/godotengine/godot/blob/master/scene/gui/text_edit.cpp#L1078

toger5 commented 6 years ago

I also noted folling issues with them beeing icons:

Isnt there a really good font icon (in terms of a character...) that could be used?

toger5 commented 6 years ago

@Zylann i think the two minor ones are very personal. I like the braces. But since there is still the option for a coustom font im fine with any default. (but i think it is not worth to start another discussion about default code font. There was one recently,if im not mistaking, so the overall opinion seems to what this as the default.

djrm commented 6 years ago

@toger5 , using a character might be a really good idea, i think it should be this one , it never cossed my mind, also yeah its clear that the icon wont scale and thats a bad thing.

pwab commented 6 years ago

I really like these | indentation marks used in Notepad++ or VSCode.

indentation_mark_vscode

Problem here is that using a 'line'-character could lead to little spaces in between:

Zylann commented 6 years ago

@toger5 I managed to get my favourite font setup so I don't really mind anymore about the two minor points :)

So yeah... I think we could find more discrete and regular indentation marks, like the old Godot ones, or the verticals from Sublime (but in ST verticals are procedural, they aren't icons or even characters)

toger5 commented 6 years ago

I like both proposals: The character @djrm posted (i will tri it fith the current code font in godot) And the line (@pwab) Maybe it even would be worth to implement the line drawing? Not sure how to do it... So i cant really tell how much work it is.

Zylann commented 6 years ago

Not a fan of the ⇥, because it stands out too much for something that is shown all the time (unless the font happens to have a discrete indicator by pure luck... which isn't the case for all my fonts). I know it really is supposed to represent a tab, but I never see it used as indentation guide, but more likely shown to stand out in diff tools or "show non-printed characters" modes.

In ST it is shown only on selection (with spaces too (and the extra lines are still not characters here because they span longer than a column)): image

toger5 commented 6 years ago

just another idea. Could we adda custom character to the current default font so that it looks like it does right not but it is not an icon?

toger5 commented 6 years ago

Best solution by far is the vertical line (imo) but that is also most work.

bruvzg commented 6 years ago

Vertical lines for indentation and arrows for all other tabs (to highlight excessive tabs at the end of line). tabs

Zylann commented 6 years ago

What's wrong about the icon though? You could also draw it with a scaling rectangle based on line height, or draw an actual line instead (or have a line in the icon heheh), which isn't much work, just different drawing.

toger5 commented 6 years ago

Or just draw an appropritaly scaled line instead of the icon... Maybe the line approach is easier to inpmement than i thought.

bojidar-bg commented 6 years ago

There is: ⎸U+23B8 LEFT VERTICAL BOX LINE │U+2502 BOX DRAWINGS LIGHT VERTICAL

Zylann commented 6 years ago

@bojidar-bg character-based lines leave spaces in between rows

akien-mga commented 6 years ago

We have now entered release freeze for Godot 3.0 and want to focus only on release critical issues for that milestone. Therefore, we're moving this issue to the 3.1 milestone, though a fix may be made available for a 3.0.x maintenance release after it has been tested in the master branch during 3.1 development. If you consider that this issue is critical enough to warrant blocking the 3.0 release until fixed, please comment so that we can assess it more in-depth.

akien-mga commented 5 years ago

I think by now, most users are either fine with the new font, or using a custom one, so I think we can close this. The only thing that is likely to be fixed is added more subtle indentation guides when #20725 is revisited for 3.2.