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

Custom Waveform #26

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Custom Waveform #26

wants to merge 1 commit into from

Conversation

yokemura
Copy link
Owner

(Thanks to SHACHO (directly received the implementation source code))

resolves #25

Implementation of Custom Waveform

  • You can select time division and value division.
  • 7 Preset waveforms are available.

スクリーンショット 2021-08-15 20 51 49

To be discussed

  • Currently the window expands to the right, but I'm not sure if it is a good idea or not.
  • UI implementation(especially window resizing) is not in a manner of other features.
  • If this feature is necessary or not

@Archonic944
Copy link

I think that this is a really good feature. However, in order to maintain some level of normalcy within the plugin, it should be included in 'advanced options'. This way, the UI won't be cluttered unless you turn on 'advanced options'. The custom waveform thing being positioned the right also allows for more stuff to be put in the bottom right corner (which would be empty).

@yokemura
Copy link
Owner Author

I didn't forget about this and sometimes think what to do.

Recently I wonder what if providing this as a separated app like "Magical Wave Memory Plug"

@Archonic944
Copy link

Hmm, that would be interesting. It does kind of seem like a whole new thing since while the previous system attempted to emulate the feel of NES chiptune, this could reach into the realm of Game Boy chiptune.

However, I think putting this inside of a separate window with maybe an advanced menu checkbox that says "Enable Custom Waveform" would be good. Although, I'm not sure if a plugin with multiple windows is even technically possible, and if it is, it might be clunky.

@acheronfail
Copy link

This is a feature I was looking for, and am excited to see some current progress.
I'll provide some of my opinions if it helps:

Currently the window expands to the right, but I'm not sure if it is a good idea or not.

I don't too much about the window expanding to the right. If Waveform isn't selected, then the window will remain normal, and it would only expand to the right if Waveform was selected.

I'm happy with it there, or alternatively it could be placed at the bottom of the window (and thus expand the window down).

UI implementation(especially window resizing) is not in a manner of other features.

I personally don't think we need window resizing for this feature, in my opinion it's okay to leave as is.

If this feature is necessary or not

I would love it!
I think I'd prefer it to be included in Magical8bitPlug2 rather than a separate app, just to make installation easier, but really I don't mind too much.

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.

[FEATURE] Custom Waveform
3 participants