From 1b3702b241f02ab16c3b5366acb47cefbf089521 Mon Sep 17 00:00:00 2001 From: Vidhu Kant Sharma Date: Sun, 16 Oct 2022 18:36:17 +0530 Subject: minor bug fixes in ItemPicker input --- src/components/editors/invoice-headers-editor.js | 40 ++++++++++++++++++++++++ src/components/pickers/item-picker.js | 25 +++++++++++---- src/components/pickers/scss/item-picker.scss | 4 +++ src/components/tables/invoice-item-table.js | 2 +- src/components/tables/invoice-summary.js | 7 +++-- src/components/tables/item-table.js | 2 +- src/components/tables/scss/summary.scss | 25 +++++++++++++++ src/components/tables/scss/table.scss | 2 +- 8 files changed, 95 insertions(+), 12 deletions(-) create mode 100644 src/components/editors/invoice-headers-editor.js create mode 100644 src/components/tables/scss/summary.scss (limited to 'src/components') diff --git a/src/components/editors/invoice-headers-editor.js b/src/components/editors/invoice-headers-editor.js new file mode 100644 index 0000000..f71b319 --- /dev/null +++ b/src/components/editors/invoice-headers-editor.js @@ -0,0 +1,40 @@ +/* OpenBills-web - Web based libre billing software + * Copyright (C) 2022 Vidhu Kant Sharma + + * This program is free software: you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation, either version 3 of the License, or + * (at your option) any later version. + + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + + * You should have received a copy of the GNU General Public License + * along with this program. If not, see . + */ + +import './scss/item-editor.scss'; + +import { useState, useEffect } from 'react'; + +const InvoiceHeadersEditor = ({roundOff, setRoundOff}) => { + + return ( + <> +

Invoice Options:

+
e.preventDefault()}> + +
+ + ) +} + +export default InvoiceHeadersEditor; diff --git a/src/components/pickers/item-picker.js b/src/components/pickers/item-picker.js index c486660..3029fb5 100644 --- a/src/components/pickers/item-picker.js +++ b/src/components/pickers/item-picker.js @@ -44,15 +44,18 @@ const ItemPicker = ({invoiceItems, addInvoiceItem}) => { } const handleInput = e => { - const { name, value, type } = e.target; - const val = type === "number" ? parseFloat(value) : value + const { name, value } = e.target; setItem(prevItem => ({ ...prevItem, - [name]: type === "number" ? (isNaN(val) ? prevItem[name] : val) : val + [name]: `${value}` })); } const validate = () => { + if (!isNumeric(item.Quantity) || !isNumeric(item.DiscountPercentage) + || !isNumeric(item.GSTPercentage) || !isNumeric(item.UnitPrice)) { + return false; + } if (item.Id === null) { return false; } @@ -68,7 +71,8 @@ const ItemPicker = ({invoiceItems, addInvoiceItem}) => { if (item.MaxQuantity > 0 && item.Quantity > item.MaxQuantity) { return false; } - if (item.DiscountPercentage > 100 || item.GSTPercentage > 100) { + if (item.DiscountPercentage > 100 || item.DiscountPercentage < 0 + || item.GSTPercentage > 100 || item.GSTPercentage < 0) { return false; } return true; @@ -83,6 +87,8 @@ const ItemPicker = ({invoiceItems, addInvoiceItem}) => { } } + const isNumeric = (i) => !isNaN(i) && !isNaN(parseFloat(i)); + // input elements are sorted on the basis of // how likely they are going to be edited return ( @@ -106,10 +112,10 @@ const ItemPicker = ({invoiceItems, addInvoiceItem}) => { Quantity: 0 && item.Quantity > item.MaxQuantity) || (item.Quantity < item.MinQuantity))) ? "warning" : ""} step="0.01" value={item.Quantity} name="Quantity" - min={item.MinQuantity > 0 ? item.MinQuantity : 1} max={item.MaxQuantity > 0 ? item.MaxQuantity : null} onChange={handleInput} /> @@ -117,6 +123,7 @@ const ItemPicker = ({invoiceItems, addInvoiceItem}) => { Price: { Discount %: 100) ? "warning" : ""} + max="100" + min="0" + step="0.1" value={item.DiscountPercentage} name="DiscountPercentage" onChange={handleInput} /> @@ -152,6 +162,9 @@ const ItemPicker = ({invoiceItems, addInvoiceItem}) => { 100) ? "warning" : ""} + max="100" + min="0" value={item.GSTPercentage} name="GSTPercentage" onChange={handleInput} /> diff --git a/src/components/pickers/scss/item-picker.scss b/src/components/pickers/scss/item-picker.scss index b3e797a..63ddbcf 100644 --- a/src/components/pickers/scss/item-picker.scss +++ b/src/components/pickers/scss/item-picker.scss @@ -92,4 +92,8 @@ color: $fgColorAlt; background-color: $warningColor; } + + input.warning { + border-color: $warningColor; + } } diff --git a/src/components/tables/invoice-item-table.js b/src/components/tables/invoice-item-table.js index d008ebc..865b326 100644 --- a/src/components/tables/invoice-item-table.js +++ b/src/components/tables/invoice-item-table.js @@ -38,7 +38,7 @@ const ItemTable = ({items, setItems, isInterstate, sum}) => { */ return ( <> - +
diff --git a/src/components/tables/invoice-summary.js b/src/components/tables/invoice-summary.js index 7018d50..a5c594f 100644 --- a/src/components/tables/invoice-summary.js +++ b/src/components/tables/invoice-summary.js @@ -15,16 +15,17 @@ * along with this program. If not, see . */ +import './scss/summary.scss'; import { currency } from '../../classes/item'; -const InvoiceSummary = ({sum}) => { +const InvoiceSummary = ({sum, roundOff}) => { const totalRoundedOff = currency(sum.Amount !== undefined ? Math.round(sum.Amount.value) : 0.00); - const roundedOffDiff = sum.Amount !== undefined ? sum.Amount.subtract(totalRoundedOff) : currency(0.00); + const roundedOffDiff = sum.Amount !== undefined && roundOff ? sum.Amount.subtract(totalRoundedOff) : currency(0.00); return ( <>

Summary:

-
S. No
+
{sum.UnitPrice !== undefined && diff --git a/src/components/tables/item-table.js b/src/components/tables/item-table.js index 5a405a5..2fb7210 100644 --- a/src/components/tables/item-table.js +++ b/src/components/tables/item-table.js @@ -39,7 +39,7 @@ const ItemTable = (props) => { } return ( -
+
diff --git a/src/components/tables/scss/summary.scss b/src/components/tables/scss/summary.scss new file mode 100644 index 0000000..d5714f9 --- /dev/null +++ b/src/components/tables/scss/summary.scss @@ -0,0 +1,25 @@ +/* OpenBills-web - Web based libre billing software + * Copyright (C) 2022 Vidhu Kant Sharma + + * This program is free software: you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation, either version 3 of the License, or + * (at your option) any later version. + + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + + * You should have received a copy of the GNU General Public License + * along with this program. If not, see . + */ + +@import "colors"; + +table.summary-table { + min-width: 25rem; + tr:hover { + background-color: rgba($thColor, 0.4); + } +} diff --git a/src/components/tables/scss/table.scss b/src/components/tables/scss/table.scss index 0bb5a48..d201a49 100644 --- a/src/components/tables/scss/table.scss +++ b/src/components/tables/scss/table.scss @@ -17,7 +17,7 @@ @import "colors"; -table { +table.item-table { width: 90%; margin: auto; -- cgit v1.2.3
S. No