Skip to content
Snippets Groups Projects
Commit d6e6a6b0 authored by Tom Robinson's avatar Tom Robinson
Browse files

FieldValuesWidget: improve empty state logic and text

parent f364f1bd
No related branches found
No related tags found
No related merge requests found
......@@ -2,7 +2,7 @@
import React, { Component } from "react";
import { connect } from "react-redux";
import { t } from "c-3po";
import { t, jt } from "c-3po";
import TokenField from "metabase/components/TokenField";
import RemappedValue from "metabase/containers/RemappedValue";
......@@ -257,17 +257,21 @@ export class FieldValuesWidget extends Component {
autoLoad={false}
/>
)}
layoutRenderer={({ valuesList, optionsList, focused, onClose }) => (
layoutRenderer={({ valuesList, optionsList, isFiltered }) => (
<div>
{valuesList}
{this.props.alwaysShowOptions || this.state.focused
? optionsList ||
(this.hasList() ? (
(this.hasList() && !isFiltered ? (
<OptionsMessage
message={t`Including every option in your filter probably won’t do much…`}
/>
) : this.isSearchable() && loadingState === "LOADED" ? (
<OptionsMessage message={t`No matching results found`} />
<OptionsMessage
message={jt`No matching ${(
<strong>&nbsp;{searchField.display_name}&nbsp;</strong>
)} found.`}
/>
) : null)
: null}
</div>
......
......@@ -469,6 +469,7 @@ export default class TokenField extends Component {
} = this.props;
let {
inputValue,
searchValue,
filteredOptions,
focused,
selectedOptionValue,
......@@ -604,6 +605,7 @@ export default class TokenField extends Component {
valuesList,
optionsList,
focused,
isFiltered: !!searchValue,
onClose: this.onClose,
});
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment