Skip to content

Commit

Permalink
Merge pull request #2731 from Rithvik-padma/master
Browse files Browse the repository at this point in the history
fix: disable the checkbox for budget control during payments
  • Loading branch information
rolznz authored Sep 15, 2023
2 parents 6fbb7f7 + 4d9c397 commit 8da1278
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 1 deletion.
5 changes: 4 additions & 1 deletion src/app/components/BudgetControl/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ type Props = {
budget: string;
onBudgetChange: ChangeEventHandler<HTMLInputElement>;
fiatAmount: string;
disabled?: boolean;
};

function BudgetControl({
Expand All @@ -18,6 +19,7 @@ function BudgetControl({
budget,
onBudgetChange,
fiatAmount,
disabled = false,
}: Props) {
const { t } = useTranslation("components", {
keyPrefix: "budget_control",
Expand All @@ -27,12 +29,13 @@ function BudgetControl({

return (
<div className="mb-4">
<div className="flex items-center">
<div className={`flex items-center`}>
<Checkbox
id="remember_me"
name="remember_me"
checked={remember}
onChange={onRememberChange}
disabled={disabled}
/>
<label
htmlFor="remember_me"
Expand Down
2 changes: 2 additions & 0 deletions src/app/components/form/Checkbox.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ function Checkbox({
name,
checked,
onChange,
disabled,
}: React.InputHTMLAttributes<HTMLInputElement>) {
return (
<input
Expand All @@ -11,6 +12,7 @@ function Checkbox({
type="checkbox"
checked={checked}
onChange={onChange}
disabled={disabled}
className="h-4 w-4 text-primary focus:ring-primary border-gray-300 rounded cursor-pointer"
/>
);
Expand Down
1 change: 1 addition & 0 deletions src/app/screens/ConfirmKeysend/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -160,6 +160,7 @@ function ConfirmKeysend() {
}}
budget={budget}
onBudgetChange={(event) => setBudget(event.target.value)}
disabled={loading}
/>
<ConfirmOrCancel
disabled={loading}
Expand Down
1 change: 1 addition & 0 deletions src/app/screens/ConfirmPayment/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -173,6 +173,7 @@ function ConfirmPayment() {
}}
budget={budget}
onBudgetChange={(event) => setBudget(event.target.value)}
disabled={loading}
/>
)}
<ConfirmOrCancel
Expand Down

0 comments on commit 8da1278

Please sign in to comment.