Skip to content

Conversation

@AlexParadisProg
Copy link
Collaborator

New Interactive HTML v1.0.

All triggers not connected to their functions right now, only the trig.

@vercel
Copy link

vercel bot commented Apr 8, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/rafkhan/lowstepper/JEAyvi1Z7XDzXAisqj4TYnEhwBbX
✅ Preview: https://lowstepper-git-oka-46-interactive-html-2-rafkhan.vercel.app

"react": "17.0.0",
"react-dom": "17.0.0",
"react-p5-wrapper": "2.3.0",
"react-rotary-knob": "^1.1.8",
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this thing is sick

}

#root .white_hole{
position:absolute;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you legend lol

I would suggest doing it with flexbox... but I can't imagine that's very easy. This seems to work fine.


return (

<div className="App">
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We will have to eventually move this to it's own component

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(And not be at the top of App)

<div className="trigWrapper">
<div>
<div className="buttonWrapper red">
<button className="button" onMouseDown={(e) => {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wrap setState({ ...state, "gate": true }) in some function

you can use a react component as well for the repetition of onMouseUp / onMouseOut, etc

if (distance > maxDistance) {
return;
} else {
this.setState({ value: val });
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add onUpdate prop here, call with value

<div className="bg_potentiometer"></div>
<h2 className="parameterName"> {currentKnob.name}</h2>
<div className="bgKnob knobB">
<LimitedKnob
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add onUpdate to this component, then update corresponding props value per knob

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

validate by seeing rate / morph / chunks change

}

.container {
width: 440px;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for some reason this looks enormous on my screen currently... not sure why, maybe 2x monitor?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants