Hello, I have some pieces of advice for little issues. These issues supposed to be opened in various issue categories. But now, I just sum it up because it needs further discussion.
Change the variable name of
private ConnectionArrowType _arrow; to _arrowType;.
And also
public ConnectionArrowType Arrow { ... } to ArrowType { ... } in Alphabet.cs. These are to prevent the confusion.
Change the declaration of this method:
public static void DrawDics() to be: ... DrawDiscs or ... DrawDisc.
It is better to loads the textures in a single class, like Style.cs. For example we have "delete.png, edit.png, etc". Another benefit, we can implement tab-button states like this. But problems may occur. For example,
private static Texture2D _miniButtonLeftNormalTexture;
_pointsInterfaceStyle = new GUIStyle(EditorStyles.miniButtonLeft);
The GUIStyle() can only be called inside OnGUI(). Image below is an example.
wazho
commented
7 years ago
Hello, I have some pieces of advice for little issues. These issues supposed to be opened in various issue categories. But now, I just sum it up because it needs further discussion.
Change the variable name of
private ConnectionArrowType _arrow;to_arrowType;. And alsopublic ConnectionArrowType Arrow { ... }toArrowType { ... }inAlphabet.cs. These are to prevent the confusion.Change the declaration of this method:
public static void DrawDics()to be:... DrawDiscsor... DrawDisc.It is better to loads the textures in a single class, like
Style.cs. For example we have "delete.png, edit.png, etc". Another benefit, we can implement tab-button states like this. But problems may occur. For example,The
GUIStyle()can only be called insideOnGUI(). Image below is an example.*Many more issues coming ;-)