From e636397f90106d78f4e354e52a40237171a5fa87 Mon Sep 17 00:00:00 2001 From: Alexander Rose <alexander.rose@weirdbyte.de> Date: Wed, 15 Mar 2023 20:56:08 -0700 Subject: [PATCH] ensure marking edges are at least one pixel wide --- CHANGELOG.md | 1 + src/mol-canvas3d/passes/marking.ts | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 959388fa2..a502f069d 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -11,6 +11,7 @@ Note that since we don't clearly distinguish between a public and private interf - Reduce over-blurring occlusion at larger view distances - Fix occlusion artefact with non-canvas viewport and pixel-ratio > 1 - Update nodejs-shims conditionals to handle polyfilled document object in NodeJS environment. +- Ensure marking edges are at least one pixel wide ## [v3.31.4] - 2023-02-24 diff --git a/src/mol-canvas3d/passes/marking.ts b/src/mol-canvas3d/passes/marking.ts index 8c17af2b7..bab5b8fda 100644 --- a/src/mol-canvas3d/passes/marking.ts +++ b/src/mol-canvas3d/passes/marking.ts @@ -106,7 +106,7 @@ export class MarkingPass { const { highlightEdgeColor, selectEdgeColor, edgeScale, innerEdgeFactor, ghostEdgeStrength, highlightEdgeStrength, selectEdgeStrength } = props; const { values: edgeValues } = this.edge; - const _edgeScale = Math.round(edgeScale * this.webgl.pixelRatio); + const _edgeScale = Math.max(1, Math.round(edgeScale * this.webgl.pixelRatio)); if (edgeValues.dEdgeScale.ref.value !== _edgeScale) { ValueCell.update(edgeValues.dEdgeScale, _edgeScale); this.edge.update(); -- GitLab