From 8dc532eb09c685d31ba56317959b85a9fdc49875 Mon Sep 17 00:00:00 2001 From: Surma Date: Thu, 27 May 2021 13:53:32 +0100 Subject: [PATCH] Fix power-of-2 range slider --- src/features/encoders/basis/client/index.tsx | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/features/encoders/basis/client/index.tsx b/src/features/encoders/basis/client/index.tsx index 6fc5ac75..c570e5a4 100644 --- a/src/features/encoders/basis/client/index.tsx +++ b/src/features/encoders/basis/client/index.tsx @@ -51,11 +51,14 @@ export class Options extends Component { mipmap: inputFieldChecked(form.mipmap, options.mipmap), srgb_mipmap: inputFieldChecked(form.srgb_mipmap, options.srgb_mipmap), mipmap_filter: form.mipmap_filter?.value ?? defaultOptions.mipmap_filter, + // FIXME: We really should support range remapping + // in the range-slider component. For now I’ll + // shoe-horn it into the state management. mipmap_min_dimension: 2 ** inputFieldValueAsNumber( form.mipmap_min_dimension, - options.mipmap_min_dimension, + Math.floor(Math.log2(options.mipmap_min_dimension)), ), quality: inputFieldValueAsNumber(form.quality, options.quality), compression: inputFieldValueAsNumber(