UI: Improved Add tag UX and added services search-bar back Also fixed services loader issue (#1043)

* UI: Improved Add tag UX and added services search-bar back Also fixed services loader issue

* Addressing comments
This commit is contained in:
darth-coder00 2021-11-03 19:02:28 +05:30 committed by GitHub
parent 4cc922c885
commit ea4e3b4319
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 9 additions and 9 deletions

View File

@ -516,12 +516,8 @@ const EntityTable = ({
/> />
</button> </button>
) : ( ) : (
<span className="tw-opacity-0 group-hover:tw-opacity-100"> <span className="tw-opacity-60 group-hover:tw-opacity-100 tw-text-grey-muted group-hover:tw-text-primary">
<Tags <Tags tag="+ Add tag" type="outlined" />
className="tw-border-main"
tag="+ Add tag"
type="outlined"
/>
</span> </span>
) )
) : null} ) : null}
@ -567,7 +563,7 @@ const EntityTable = ({
</div> </div>
{checkIfJoinsAvailable(row.original.name) && ( {checkIfJoinsAvailable(row.original.name) && (
<div className="tw-mt-3"> <div className="tw-mt-3">
<span className="tw-text-gray-400 tw-mr-1"> <span className="tw-text-grey-muted tw-mr-1">
Frequently joined columns: Frequently joined columns:
</span> </span>
<span> <span>

View File

@ -394,7 +394,11 @@ const ServicesPage = () => {
value: service.collection.name, value: service.collection.name,
}; };
}); });
setIsLoading(false); // Removed "setIsLoading(false)" from here,
// If there are service categories available
// Loader should wait for each category to fetch list of services
// Then only it should stop Loading and show available data or No data template
// This is handled in "updateServiceList" method
} else { } else {
setIsLoading(false); setIsLoading(false);
} }
@ -423,7 +427,7 @@ const ServicesPage = () => {
</div> </div>
<div className="tw-flex"> <div className="tw-flex">
<div className="tw-w-4/12"> <div className="tw-w-4/12">
{searchText || serviceList.length > 10 ? ( {searchText || serviceList.length > 0 ? (
<Searchbar <Searchbar
placeholder={`Search for ${servicesDisplayName[serviceName]}...`} placeholder={`Search for ${servicesDisplayName[serviceName]}...`}
searchValue={searchText} searchValue={searchText}