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

Enable rendering of non-editable marker lines #126

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ziggythehamster
Copy link
Contributor

Today, in Peaks, if a marker is not editable, only the waveform highlight gets rendered. If you want the lines on either side of the marker (or if it's a point, just the line) to be rendered, that cannot be controlled. This PR adds the ability to control if we draw marker lines or not (we do not draw the handle) if the marker is not editable.

This defaults to the existing behavior; users must opt-in and choose which waveforms they want the marker lines to render on if they want this feature.

@ziggythehamster
Copy link
Contributor Author

@oncletom Any chance of getting this reviewed? :)

@thom4parisot
Copy link
Contributor

Hi @ziggythehamster, I am unsure to understand the actual behaviour - my head is not fresh yet ;-)

Any chance any of you to share a screenshot of the current behaviour, and a screenshot with the code of this PR?

My concern is some complexity is added in the code – it seems to do the job – but still, I wonder if it has any value to make it configurable.

@ziggythehamster
Copy link
Contributor Author

Today, if you have editable: false on a segment, only the waveform is highlighted - there are no bars on both sides as when editable: true. Points don't display at all when editable: false.

This PR merely makes it possible to display the bars on both sides of a segment or the single bar of a point when editable: false. The handles on the bars are not rendered.

The default behavior is exactly what Peaks has been doing already.

Also, you can configure the waveforms differently from one another. Maybe you hide the bars on the overview but show them on the zoom, or vice versa.

I can try to get you some screenshots if you'd like, though it's pretty much what you'd expect: with this option off, it looks like it does today, with this option on, it looks like editable mode just without the handles. :)

@chrisn chrisn force-pushed the master branch 2 times, most recently from 0e6b861 to b8c6557 Compare August 10, 2017 17:55
@chrisn chrisn force-pushed the master branch 2 times, most recently from 38e7da9 to bb8dbce Compare August 22, 2017 21:01
@chrisn chrisn force-pushed the master branch 3 times, most recently from d73e5c8 to c85041a Compare June 26, 2021 21:40
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