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

increase maximum waveform zoom out level #1165

Merged
merged 2 commits into from
Apr 1, 2017

Conversation

Be-ing
Copy link
Contributor

@Be-ing Be-ing commented Feb 1, 2017

As requested by a user. I have found myself staring at overview waveforms to know when a transition is coming, which is not what the overviews are for. Zooming the waveforms out further should help with that.

It would also be nice if the current play position relative to the widget's length could be adjusted. It is more important to see what is coming than what has already played, so it would be good to set this 1/4 or 1/3 from the left. There is a Launchpad ticket for that.

@Be-ing
Copy link
Contributor Author

Be-ing commented Feb 1, 2017

I tested setting the max level even higher and I think that beyond 10% the user may as well be looking at the overview waveform. As the waveform is zoomed out, the beat marker lines stay the same width and get closer together, so they become distracting.

@SaturnusDJ
Copy link

Thanks.
Is there a compile available with this? I can't build currently.

@Be-ing
Copy link
Contributor Author

Be-ing commented Feb 6, 2017

If you use Windows, you can download AppVeyor's build. I'm not sure if Travis keeps the builds for Mac and GNU/Linux. When this is merged, the Mixxx build server will build it for all platforms and put it online.

@Be-ing
Copy link
Contributor Author

Be-ing commented Feb 18, 2017

Can someone test this?

@SaturnusDJ
Copy link

Sure I would love to. Though, it would save me a lot of time if anyone has a build ready somewhere.

@daschuer
Copy link
Member

There is a small issue here

m_pWaveformZoom->setStepCount(WaveformWidgetRenderer::s_waveformMaxZoom -

    m_pWaveformZoom->setStepCount(WaveformWidgetRenderer::s_waveformMaxZoom -
WaveformWidgetRenderer::s_waveformMinZoom);

from 1 ... 10 we have 10 steps. This formula results in 9.
But there seams to be an additional issue. Because the Deere Zoom buttons have 7 steps, and I cannot reach the minimum.

@Be-ing
Copy link
Contributor Author

Be-ing commented Mar 30, 2017

Because the Deere Zoom buttons have 7 steps, and I cannot reach the minimum.

You can use the mouse scroll wheel to zoom or set the default zoom level in the preferences. I will fix this in #940 to avoid merge conflicts.

@daschuer
Copy link
Member

Is the issue related to the skin? It looks like the Deere button are directly connected to COs.

@Be-ing
Copy link
Contributor Author

Be-ing commented Mar 30, 2017

Hm, maybe it is a bug in ControlPotmeter or ControlPotmeterBehavior?

@Be-ing
Copy link
Contributor Author

Be-ing commented Mar 30, 2017

I think the problem is that the waveform_zoom CO is a ControlPotmeter, not that something is wrong with ControlPotmeter. I'll change it to a plain ControlObject and manually create the waveform_zoom_up/down COs.

@daschuer
Copy link
Member

daschuer commented Apr 1, 2017

I did not understand why potmeter not works, but who cares. Now it is working and the interface is not changed. If one has the time to have a deeper look, we can fix it. I will file a bug at Launchpad.

@daschuer
Copy link
Member

daschuer commented Apr 1, 2017

Here it is:
https://bugs.launchpad.net/mixxx/+bug/1678441
The rest LGTM.
Thank you for the work.

@daschuer daschuer merged commit afd018e into mixxxdj:master Apr 1, 2017
@Be-ing Be-ing deleted the waveform_zoom_out_farther branch April 1, 2017 12:44
@Be-ing
Copy link
Contributor Author

Be-ing commented Apr 1, 2017

I don't entirely understand either. It had something to do with scaling between [set/get][value/parameter].

@SaturnusDJ
Copy link

Cross-post from https://bugs.launchpad.net/mixxx/+bug/1656209

I've just seen the new zoom levels in v2.1. Am I doing something wrong? It seems that there is barely more too see. I can see ahead for only 30 beats. I've put the preferences at max zoom out (10%). As originally said, requesting to be able to see ahead for about 200 beats.

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.

3 participants