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

settings panel disappears instantly after opening it in Tango & Deere skin #9461

Closed
mixxxbot opened this issue Aug 23, 2022 · 13 comments
Closed
Labels
Milestone

Comments

@mixxxbot
Copy link
Collaborator

Reported by: iamcodemaker
Date: 2018-10-02T14:09:34Z
Status: Fix Released
Importance: Low
Launchpad Issue: lp1795663


Sometimes when opening the settings panel on the Tango skin in the Mixxx 2.2 beta the panel closes when the mouse is moved. I can work around this by clicking and dragging the settings button before releasing the mouse button.

@mixxxbot mixxxbot added the bug label Aug 23, 2022
@mixxxbot
Copy link
Collaborator Author

Commented by: rryan
Date: 2018-10-05T19:41:52Z


Thanks for the report -- are you on Windows, macOS or Linux? What kind of mouse are you using (touchpad, standalone mouse, touchscreen)

@mixxxbot
Copy link
Collaborator Author

Commented by: iamcodemaker
Date: 2018-10-06T13:50:47Z


On macos, using the built in track pad.

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2018-10-10T06:51:01Z


Does this also happen with Deere?

In both Tango and Deere the skin settings toggle in the tool bar is in the exact same position as the close button in the skin settings menu is.

Could it be the menu opens so quickly that the trackpad release event is sent to the close button? Or that a false release event is sent as soon as the trackpad cursor is moved?

@mixxxbot
Copy link
Collaborator Author

Commented by: iamcodemaker
Date: 2018-10-10T22:17:06Z


Tested. This happens on the Deere skin too. And also, you may be right about the button up event, as whatever control my mouse ends up on gets activated.

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2018-10-21T14:23:15Z


btw, do the FX unit expand toggles work for you in Tango or LateNight?
They are also swapped underneath the pointer when clicked, but the actual widget switching might be slower than opening the sin menu..

@mixxxbot
Copy link
Collaborator Author

Commented by: iamcodemaker
Date: 2018-10-21T14:42:28Z


FX units expand fine under Tango

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2018-10-23T16:00:12Z


Does it also happen the other way around? (panel is open, click on Close button)

Are you keen on testing <EmitOnPressAndRelease>false</EmitOnPressAndRelease> for the skinsettings toggle?

In skin_settings.xml and topbar.xml replace
<Template src="skin:../Tango/button_2state.xml">
  <SetVariable name="TooltipId">skin_settings</SetVariable>
  <SetVariable name="ObjectName">SkinSettingsClose</SetVariable>
  <SetVariable name="Size">20f,24f</SetVariable>
  <SetVariable name="ConfigKey">[Master],skin_settings</SetVariable>
</Template>

with
<PushButton>
  <TooltipId>skin_settings</TooltipId>
  <ObjectName>SkinSettingsClose</ObjectName>
  <Size>20f,24f</Size>
  <EmitOnPressAndRelease>false</EmitOnPressAndRelease>
  <NumberStates>2</NumberStates>
  <State>
    <Number>0</Number>
  </State>
  <State>
    <Number>1</Number>
  </State>
  <Connection>
    <ConfigKey>[Master],skin_settings</ConfigKey>
    <ButtonState>LeftButton</ButtonState>
  </Connection>
</PushButton>

@mixxxbot
Copy link
Collaborator Author

Commented by: iamcodemaker
Date: 2018-10-23T16:22:33Z


It doesn't happen when closing the panel. I'll test with those changes this evening.

@mixxxbot
Copy link
Collaborator Author

Commented by: iamcodemaker
Date: 2018-10-24T02:01:11Z


Making those changes in the skin file fixes the issue.

I should also note, this issue does not occur when I build the latest git master from source. Not sure if that is relevant.

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2018-10-24T13:17:45Z


Fine, thanks!
Here's a quick fix for 2.2 then #1866

@mixxxbot
Copy link
Collaborator Author

Commented by: Be-ing
Date: 2018-10-24T14:16:03Z


Does this also affect the skin settings button in LateNight?

I should also note, this issue does not occur when I build the latest git master from source. Not sure if that is relevant.

I don't think there have been any substantial changes to the skins or skin system in master that aren't in the 2.2 branch. However, our build server is building Mixxxx with the 10.13 macOS SDK. If you're using the 10.14 macOS SDK or running macOS 10.14 that could potentially explain this.

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2018-12-22T18:09:13Z


this bug popped up again in the 2.2 release.
#1954

@mixxxbot
Copy link
Collaborator Author

Issue closed with status Fix Released.

@mixxxbot mixxxbot transferred this issue from another repository Aug 24, 2022
@mixxxbot mixxxbot added this to the 2.2.0 milestone Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant