fix(admin-ui): Make copy on manage locations modal better when no locations (#3666)

This commit is contained in:
Rares Stefan
2023-03-31 14:38:01 +02:00
committed by GitHub
parent bd12a95083
commit ca3b32d53c
2 changed files with 33 additions and 13 deletions
+5
View File
@@ -0,0 +1,5 @@
---
"@medusajs/admin-ui": patch
---
fix(admin-ui): Make copy and UI on manage locations modal better when no locations
@@ -1,7 +1,7 @@
import { Controller, useFieldArray, UseFormReturn } from "react-hook-form" import { Controller, useFieldArray, UseFormReturn } from "react-hook-form"
import { nestedForm } from "../../../../../utils/nested-form" import { nestedForm } from "../../../../../utils/nested-form"
import React, { useMemo, useState } from "react" import React, { useMemo, useState } from "react"
import clsx from "clsx"
import Accordion from "../../../../organisms/accordion" import Accordion from "../../../../organisms/accordion"
import BuildingsIcon from "../../../../fundamentals/icons/buildings-icon" import BuildingsIcon from "../../../../fundamentals/icons/buildings-icon"
import Button from "../../../../fundamentals/button" import Button from "../../../../fundamentals/button"
@@ -342,16 +342,30 @@ const ManageLocationsForm = ({
setSelectedLocations(locationOptions.map((l) => l.id)) setSelectedLocations(locationOptions.map((l) => l.id))
} }
const locationsAvailable = !!locationOptions.length
return ( return (
<div className="h-full w-full"> <div className="h-full w-full">
<form onSubmit={handleSubmit}> <form onSubmit={handleSubmit}>
<Modal.Content> <Modal.Content>
<div> <div>
<div className="border-grey-20 pb-base text-grey-50 flex w-full items-center justify-between border-b"> <div
<div className=""> className={clsx(
<p>Select locations that stock the selected variant</p> "border-grey-20 text-grey-50 flex w-full items-center justify-between",
{ "pb-base border-b": locationsAvailable }
)}
>
<div>
<p>
{locationsAvailable
? "Select locations that stock the selected variant"
: "You don't have any locations"}
</p>
{locationsAvailable && (
<p>{`(${selectedLocations.length} of ${locationOptions.length} selected)`}</p> <p>{`(${selectedLocations.length} of ${locationOptions.length} selected)`}</p>
)}
</div> </div>
{locationsAvailable && (
<Button <Button
size="small" size="small"
variant="ghost" variant="ghost"
@@ -360,6 +374,7 @@ const ManageLocationsForm = ({
> >
Select all Select all
</Button> </Button>
)}
</div> </div>
{locationOptions.map((loc) => { {locationOptions.map((loc) => {
const existingLevel = selectedLocations.find((l) => l === loc.id) const existingLevel = selectedLocations.find((l) => l === loc.id)