-
-
Notifications
You must be signed in to change notification settings - Fork 59
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
$mol_audio refactor, added sample and scheduled nodes #654
Conversation
audio/context/context.node.ts
Outdated
} | ||
}) | ||
|
||
$.$mol_audio_context = $mol_audio_context_node |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
$node['web-audio-api']
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Возможно даже так можно:
static context() {
return $mol_dom_context.AudioContext || $node['web-audio-api']
}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
да все-равно надо билдер опять фиксить, пока можно стаб оставить, я так и не смог найти
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Не надо тут ничего фиксить, нодовая зависимость должна динамически загружаться.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Это ж универсальная реализация. Отдельный слайс для ноды не нужен.
audio/context/context.node.ts
Outdated
} | ||
}) | ||
|
||
$.$mol_audio_context = $mol_audio_context_node |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Это ж универсальная реализация. Отдельный слайс для ноды не нужен.
audio/context/context.node.ts
Outdated
static override create_context(): AudioContext & AudioContextNode { | ||
const AudioContext = this.$.$node['web-audio-api'].AudioContext | ||
return new AudioContext() | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Context() {
return ( this.$.$mol_dom_context || this.$.$node['web-audio-api'] ).AudioContext
}
No description provided.