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

Remove shader src parameters from GUINode #341

Merged
merged 3 commits into from
Sep 22, 2021

Conversation

RedImp1470
Copy link
Member

@RedImp1470 RedImp1470 commented Sep 21, 2021

Removed vs and ps parameters from the TextureNode and TextNode constructors.
There is only one correct set of shaders per constructor. Furthermore this prevents spelling mistakes like "tex" where "text" is needed.

@RedImp1470 RedImp1470 changed the base branch from master to develop September 21, 2021 12:54
@RedImp1470 RedImp1470 added the prio-should Should be implemted label Sep 21, 2021
@RedImp1470 RedImp1470 changed the title Feature/remove shader src param from GUINode Remove shader src parameters from GUINode Sep 21, 2021
@RedImp1470 RedImp1470 merged commit bdbf82e into develop Sep 22, 2021
@RedImp1470 RedImp1470 deleted the feature/RemoveShaderSrcParamFromGUINode branch September 24, 2021 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
prio-should Should be implemted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants