Infrastructure/ghost
0
Fork 0
mirror of https://github.com/TryGhost/Ghost.git synced 2025-03-11 02:12:21 -05:00
Code Issues Activity

Correctly used the max length variable for the bio text box in modal

Browse source 
refs https://github.com/TryGhost/Team/issues/1716
This commit is contained in:
James Morris 2022-08-04 13:28:52 +01:00 • committed by Simon Backx
parent 0a1b1eb70a
commit 2027c88f58
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
apps/comments-ui/src/components/modals/AddDetailsDialog.js
View file

@ -11,8 +11,8 @@ const AddNameDialog = (props) => {
const [name, setName] = useState(member.name ?? '');
const [bio, setBio] = useState(member.bio ?? '');
const maxBioCharsLeft = 50;
const [bioCharsLeft, setBioCharsLeft] = useState(maxBioCharsLeft);
const maxBioChars = 50;
const [bioCharsLeft, setBioCharsLeft] = useState(maxBioChars);
const [error, setError] = useState({name: '', bio: ''});
@ -116,7 +116,7 @@ const AddNameDialog = (props) => {
placeholder="Head of Marketing at Acme, Inc"
onChange={(e) => {
let bioText = e.target.value;
setBioCharsLeft(maxBioCharsLeft - bioText.length);
setBioCharsLeft(maxBioChars - bioText.length);
setBio(bioText);
}}
onKeyDown={(e) => {
@ -125,7 +125,7 @@ const AddNameDialog = (props) => {
submit();
}
}}
maxLength="50"
maxLength={maxBioChars}
/>
<button
className="transition-opacity duration-200 ease-linear w-full h-[44px] mt-8 px-8 flex items-center justify-center rounded-md text-white font-sans font-semibold text-[15px] bg-[#3204F5] opacity-100 hover:opacity-90"