Skip to content

Commit 905d4e1

Browse files
committed
Fix - drawing color palette
Single bin cannot exceed provided range
1 parent 990f9fe commit 905d4e1

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

modules/hist/TPavePainter.mjs

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -911,18 +911,19 @@ class TPavePainter extends ObjectPainter {
911911
for (let i = 0; i < levels.length-1; ++i) {
912912
let z0 = Math.round(this.z_handle.gr(levels[i])),
913913
z1 = Math.round(this.z_handle.gr(levels[i+1])),
914-
lvl = (levels[i]+levels[i+1])/2, d;
914+
lvl = (levels[i] + levels[i+1])*0.5, d;
915915

916916
if (this._palette_vertical) {
917917
if ((z1 >= s_height) || (z0 < 0)) continue;
918918
z0 += 1; // ensure correct gap filling between colors
919919

920920
if (z0 > s_height) {
921921
z0 = s_height;
922-
lvl = levels[i]*0.001+levels[i+1]*0.999;
922+
lvl = levels[i]*0.001 + levels[i+1]*0.999;
923+
if (z1 < 0) z1 = 0;
923924
} else if (z1 < 0) {
924925
z1 = 0;
925-
lvl = levels[i]*0.999+levels[i+1]*0.001;
926+
lvl = levels[i]*0.999 + levels[i+1]*0.001;
926927
}
927928
d = `M0,${z1}H${s_width}V${z0}H0Z`;
928929
} else {
@@ -931,10 +932,11 @@ class TPavePainter extends ObjectPainter {
931932

932933
if (z1 > s_width) {
933934
z1 = s_width;
934-
lvl = levels[i]*0.999+levels[i+1]*0.001;
935+
lvl = levels[i]*0.999 + levels[i+1]*0.001;
936+
if (z0 < 0) z0 = 0;
935937
} else if (z0 < 0) {
936938
z0 = 0;
937-
lvl = levels[i]*0.001+levels[i+1]*0.999;
939+
lvl = levels[i]*0.001 + levels[i+1]*0.999;
938940
}
939941
d = `M${z0},0V${s_height}H${z1}V0Z`;
940942
}

0 commit comments

Comments
 (0)