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 Noise icon #79722

Closed
wants to merge 1 commit into from
Closed

Conversation

MewPurPur
Copy link
Contributor

@MewPurPur MewPurPur commented Jul 20, 2023

image

What do you think? IMO it looks quiet noise.

@KoBeWi
Copy link
Member

KoBeWi commented Aug 1, 2023

I'd see this icon more fitting for a generic waveform than a noise.

@MewPurPur
Copy link
Contributor Author

image

^ My idea was a reference to noise functions.

But on a second thought, I'm looking at Gradient, GradientTexture1D, and GradientTexture2D... and I'm starting to think the winning move here might be similar - letting Noise use the same pattern as NoiseTexture2D, but letting it sprawl on a bit more.

@MewPurPur MewPurPur closed this Aug 8, 2023
@MewPurPur
Copy link
Contributor Author

My PR with a NoiseTexture3D icon was merged, so I'll brainstorm a design for consistent icons in a new PR.

@MewPurPur MewPurPur deleted the add-noise-icon branch August 8, 2023 10:08
@YuriSizov YuriSizov removed this from the 4.x milestone Aug 8, 2023
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