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

Presets wiki #42

Open
Colbix opened this issue Feb 25, 2023 · 0 comments
Open

Presets wiki #42

Colbix opened this issue Feb 25, 2023 · 0 comments

Comments

@Colbix
Copy link

Colbix commented Feb 25, 2023

I've started using Bitfocus Companion v3.0 beta and wanted to develop a module as an existing one either wasn't compatible or didn't meet my use case. The wiki for presets1 can be improved that would clarify aspects with module development. I thought to provide some feedback after my first go through with using presets where there were some knowledge gaps.

  1. combineRgb reference in color and bgcolor reference. If combineRgb isn't imported it will cause an error, especially if trying to separate the presets into their own file. Recommend either include const { combineRgb } = require('@companion-module/base') at the beginning of the example code or make a note that it needs to be imported into the file where it's referenced.
  2. this.setPresetDefinitions(presets). If used with v3.0 creates a TypeError. Recommend changing the example to self.setPresetDefinitions(presets) as it corrects the error.
  3. options. A minor thing with options, but helps to clarify the use as it relates to actions, should mention they should be stacked if multiple options are located in the referenced actions. ie:
options: {
// options values to use
brightness_level_1: 100,
brightness_level_2: 75,
},
  1. a broken link. the link at the end of the sentence "You can see the full list of values that can be set and their valid values in the style object here" is broken. Broken Style. Recommend using the following link instead: Style

I haven't quite figured out how to make use of the icons, maybe it's similar in use as the combineRgb where I'm missing an import? For now I've used a similar module that contained a base64 image of what I needed.

Overall I enjoyed the experience and am looking forward to v3.0 coming out of beta. Thank you for your time.

Footnotes

  1. Presets

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

No branches or pull requests

1 participant