sync_headers in dhtmlxgrid_splt.js has problem.

Suite and Optimus Suite 5
#1

in file dhtmlxgrid_splt.js: about line 725,

this.sync_headers=this._fake.sync_headers=function(){ ... if (window._KHTMLrv) { var lindex = 0; while (this._fake._hrrar[lindex]) lindex++; this._fake.hdr.rows[i].childNodes[lindex].style.height=this.hdr.rows[i].childNodes[td].style.height=Math.max(ha,hb)+"px"; //this line is not right. } } ... }

this line this._fake.hdr.rows[i].childNodes[lindex].style.height=this.hdr.rows[i].childNodes[td].style.height=Math.max(ha,hb)+“px”; is not right,
this._fake.hdr.rows[i].childNodes[lindex].style.height should be Math.max(ha,hb) - paddingtop - paddingbottom, because height not include padding.

and window._KHTMLrv is not set right too, for my Chrome/60.0.3112.101 browser, the userAgent is “User-Agent:Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/60.0.3112.101 Safari/537.36”
window._KHTMLrv should be set false, but dhtmlx set window._KHTMLrv = 537.36,

below code in file dhtmlxcommon.js: about line 1403:

if ((navigator.userAgent.indexOf('Safari') != -1)||(navigator.userAgent.indexOf('Konqueror') != -1)){
	 _KHTMLrv = parseFloat(navigator.userAgent.substr(navigator.userAgent.indexOf('Safari')+7, 5));

	if (_KHTMLrv > 525){ //mimic FF behavior for Safari 3.1+
		_isFF=true;
		 _FFrv = 1.9;
		//? need set _KHTMLrv = false. ?
	} else
		_isKHTML=true;
} else if (navigator.userAgent.indexOf('Opera') != -1){
	_isOpera=true;
	_OperaRv=parseFloat(navigator.userAgent.substr(navigator.userAgent.indexOf('Opera')+6, 3));
}

above code maybe has problem, when _KHTMLrv > 525 maybe need set _KHTMLrv = false.

thanks!