Fixed group / layer height is buggy and giving "infinite loop in redraw"

[callmeteus]

I'm trying to set a fixed group/layer height.

My current options are

showMajorLabels: false,
editable: true,
height: "calc(45vh - 20px)",
groupHeightMode: "fixed",
showCurrentTime: false,
verticalScroll: true,
horizontalScroll: true,
groupEditable: true

This is the actual working timeline (not fully, it's a wip haha):
http://prntscr.com/p9lh0m

Did some CSS work to make the groups appear 100% height:

/* This class (group) is applied to all groups */
.vis-inner .group {
    height: 100%;
}

.vis-inner {
    width: 100%;
    height: 100%;
    padding: 0 !important;
    overflow: hidden;
}

.vis-labelset > .vis-label {
    display: flex;
    justify-content: center;
    align-items: center;
}

.vis-labelset {
    height: 100%;
}

.vis-content {
    top: 0 !important;
}

.vis-panel {
    box-sizing: content-box;
}

When I zoom the timeline, sometimes the groups get invalid heights and collapse themselves (http://prntscr.com/p9lgtx) and sometimes they just flicker and constantly try to update the height every zoom.

Also, I'm receiving WARNING: infinite loop in redraw? in the console when I use the zoom, and when the timeline gets drawed in the first time, maybe it can be related to #17?

I tried to remove the custom heights from .vis-labelset and .vis-inner, but they also give me the same errors.

Anyways, the only thing that I'm updating is the current time position that I have using timeline.setCustomTime, I'm not even touching the redraw functions :(

[callmeteus]

Also, observed that when I try to drag the groups, they have the same behavior, they just contract to buggy heights.

[callmeteus]

Did a workaround for this because it was really annoying me.

I don't really know if the "fixed" group height mode was supposed to set groups to a fixed height, but this is was I did:

/**
 * Calculate group height
 * @return {Number}
 */
Group.prototype._calculateHeight 	= function(margin) {
	// Divide the groups container height by
	// the number of groups to get the desired group height
	// minus the axis margin
	const height 					= (this.dom.label.parentElement.clientHeight / this.dom.label.parentElement.children.length) - margin.axis;

	// Return the fixed height
	return height;
};

This also stopped the warnings in the console.

[yotamberk]

I've submitted a PR solving an issue that is very similar to this one. The PR should also solve this issue.
#174
Please try again with the new version once it's in.

[yotamberk]

Closing due to inactivity. Please reopen if the issue is still relevant.