search

SingularSound / openbbm

BeatBuddy Manager
https://singularsound.com/product/beatbuddy/
34 stars 9 forks source link

Have to double click after drag to show APBox #103

Closed JulissaDantes closed 4 years ago

JulissaDantes commented 4 years ago

When a fill which has AP enabled is copied, the pasted fill has AP enabled, but no settings input box. User needs to disable and enable AP of the fill in order for settings to appear. What should happen is that either the input box is displayed along with the check-mark or neither the check mark nor the input box are displayed.

JulissaDantes commented 4 years ago

This fix will be included:

When drop fill on top of another already set fill, saving and reopening the previously saved setting is removed.

Kiroto commented 4 years ago

Tested, everything working smoothly!

JulissaDantes commented 4 years ago

This will be closed after QC

JulissaDantes commented 4 years ago

@brennansingular This is ready to be tested on master, the release.zip file is already updated.

brennansingular commented 4 years ago

@JulissaDantes ignore my above comments was confused

Copy-pasting is not making the box disappear, so I guess that was the fix here? Or is it only an issue when dragging and dropping?

JulissaDantes commented 4 years ago

@brennansingular the issue here is when dragging and dropping the AP box of the fills goes to a state where the box to put the ap values is hidden but the checkbox is marked. What should happen is when swappin 2 fills of dragging one to an empty space n the same song it should keep its settings.