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

feat: audio input gain control #800

Merged
merged 10 commits into from
Jan 3, 2023
Merged

feat: audio input gain control #800

merged 10 commits into from
Jan 3, 2023

Conversation

TimPietrusky
Copy link
Member

See #727

@TimPietrusky TimPietrusky requested a review from 2xAA December 13, 2022 22:06
@TimPietrusky TimPietrusky marked this pull request as draft December 13, 2022 22:12
@TimPietrusky
Copy link
Member Author

  • Set the gain via "Input Device Config"
  • Use the gainNode in meyda

@2xAA 2xAA marked this pull request as ready for review December 14, 2022 23:03
switchingAudio: false,
switchingVideo: false
switchingVideo: false,
gainRangeValue: 1
Copy link
Member

Choose a reason for hiding this comment

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

Using a value on data so we can keep track of what the value is doing.
Vue wasn't aware of the value before this.

We also update the value when the component is created below in the created function.

@2xAA
Copy link
Member

2xAA commented Dec 14, 2022

@TimPietrusky If you're happy with this PR, we can merge!

@2xAA 2xAA changed the title feat/727/gain feat: audio input gain control Dec 14, 2022
@2xAA 2xAA linked an issue Dec 14, 2022 that may be closed by this pull request
@TimPietrusky
Copy link
Member Author

TimPietrusky commented Dec 17, 2022

I will not merge this yet, as it's "hard" to get back to 1 or any specific value, as we can't use a right click on the slider as with the other slider we have. I will use the branch for my gig, but we should refactor this so that it's also possible to set the value via input field.

Also the tooltip of the range-slider and the value shown on the right are not the same in some cases, which is misleading.

@2xAA
Copy link
Member

2xAA commented Dec 21, 2022

@TimPietrusky Ready to review again. This should have addressed your comments

@TimPietrusky TimPietrusky merged commit 07ac0a0 into next Jan 3, 2023
@TimPietrusky TimPietrusky deleted the feat/727/gain branch January 3, 2023 21:53
github-actions bot pushed a commit that referenced this pull request Mar 1, 2023
# [3.26.0](3.25.5...3.26.0) (2023-03-01)

### Bug Fixes

* **checkbox:** add boolean emission prop ([#792](#792)) ([e62f47a](e62f47a))
* **color-picker:** dismiss color picker by clicking outside color picker ([#793](#793)) ([015b84a](015b84a)), closes [#461](#461)
* **plugins:** emit boolean from checkbox ([#795](#795)) ([c13f54a](c13f54a))
* **styling:** fix unwanted top range control alignment in module inspector ([#806](#806)) ([0082450](0082450))
* **styling:** update tab styles to match design. fix tab dropdown stacking ([#815](#815)) ([a280099](a280099))
* **swap:** properly set default swap structure when finished swapping ([#766](#766)) ([144453c](144453c)), closes [#765](#765)

### Features

* add animated loader, remove when loaded to prevent redraws ([#771](#771)) ([57f3658](57f3658))
* add automation of browserslist update ([#787](#787)) ([16f33a6](16f33a6)), closes [#691](#691)
* audio input gain control ([#800](#800)) ([07ac0a0](07ac0a0))
* **gallery-item:** implement better gallery item scaling ([#791](#791)) ([9146c1e](9146c1e))
@github-actions
Copy link

github-actions bot commented Mar 1, 2023

🎉 This PR is included in version 3.26.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Increase audio input gain directly in meyda
2 participants