Skip to content

Commit

Permalink
Remove measure big num changes
Browse files Browse the repository at this point in the history
  • Loading branch information
djbarnwal committed Sep 9, 2023
1 parent ef63719 commit 9a1798c
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 13 deletions.
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
<script lang="ts">
import { WithTween } from "@rilldata/web-common/components/data-graphic/functional-components";
import { metricsExplorerStore } from "@rilldata/web-common/features/dashboards/dashboard-stores";
import PercentageChange from "@rilldata/web-common/components/data-types/PercentageChange.svelte";
import CrossIcon from "@rilldata/web-common/components/icons/CrossIcon.svelte";
import Tooltip from "@rilldata/web-common/components/tooltip/Tooltip.svelte";
Expand All @@ -18,8 +17,6 @@
humanizeDataTypeExpanded,
} from "../humanize-numbers";
export let metricViewName: string;
export let measureName: string;
export let value: number;
export let comparisonOption: TimeComparisonOption = undefined;
export let comparisonValue: number = undefined;
Expand All @@ -43,15 +40,10 @@
/** when the measure is a percentage, we don't show a percentage change. */
$: measureIsPercentage = formatPresetEnum === FormatPreset.PERCENTAGE;
function expandMeasure() {
metricsExplorerStore.setExpandedMeasureName(metricViewName, measureName);
}
</script>

<button
on:click={() => expandMeasure()}
class="flex flex-col px-2 py-1 text-left hover:bg-gray-100 {withTimeseries
class="flex flex-col px-2 py-1 text-left {withTimeseries
? 'my-2'
: 'justify-between'}"
>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -194,8 +194,6 @@
class="inline-grid"
>
<MeasureBigNumber
{metricViewName}
measureName={measure.name}
value={bigNum}
description={measure?.description ||
measure?.label ||
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -363,8 +363,6 @@
FormatPreset[measure?.format] || FormatPreset.HUMANIZE}
<!-- FIXME: I can't select a time series by measure id. -->
<MeasureBigNumber
{metricViewName}
measureName={measure.name}
value={bigNum}
{showComparison}
comparisonOption={$timeControlsStore?.selectedComparisonTimeRange?.name}
Expand Down

0 comments on commit 9a1798c

Please sign in to comment.