Skip to content

Allow using the label as a placeholder #61

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Allow using the label as a placeholder #61

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
5c75787
Allow using inline placeholders instead of labels
tiberiuichim Mar 7, 2022
fdbad7a
Merge remote-tracking branch 'origin/main' into placeholders
tiberiuichim Aug 2, 2022
28513a4
Use state to force the update of disabled flag on captcha validation
tiberiuichim Aug 13, 2022
50a3cea
Also show explicit message in submit button
tiberiuichim Aug 13, 2022
a2ada2a
Merge remote-tracking branch 'origin/fix_captcha_update' into placeho…
tiberiuichim Aug 13, 2022
9ecf092
Merge with main
tiberiuichim Sep 28, 2022
caaba74
Field extenders should be applied in FormView
tiberiuichim Oct 5, 2022
fee8a75
Merge remote-tracking branch 'origin/placeholders' into field_extende…
tiberiuichim Oct 6, 2022
7387a27
Merge remote-tracking branch 'origin/main' into field_extenders_place…
tiberiuichim Mar 27, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 9 additions & 4 deletions src/components/Field.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@ const Field = ({
disabled = false,
formHasErrors = false,
id,
labelsAsPlaceholders,
}) => {
const intl = useIntl();

Expand All @@ -57,21 +58,24 @@ const Field = ({
<TextWidget
id={name}
name={name}
title={label}
title={labelsAsPlaceholders ? '' : label}
placeholder={labelsAsPlaceholders ? label : ''}
description={description}
required={required}
onChange={onChange}
value={value}
isDisabled={disabled}
invalid={isInvalid().toString()}
columns={labelsAsPlaceholders ? 1 : undefined}
{...(isInvalid() ? { className: 'is-invalid' } : {})}
/>
)}
{field_type === 'textarea' && (
<TextareaWidget
id={name}
name={name}
title={label}
title={labelsAsPlaceholders ? '' : label}
placeholder={labelsAsPlaceholders ? label : ''}
description={description}
required={required}
onChange={onChange}
Expand All @@ -86,7 +90,7 @@ const Field = ({
<SelectWidget
id={name}
name={name}
title={label}
title={labelsAsPlaceholders ? '' : label}
description={description}
getVocabulary={() => {}}
getVocabularyTokenTitle={() => {}}
Expand Down Expand Up @@ -185,7 +189,8 @@ const Field = ({
<EmailWidget
id={name}
name={name}
title={label}
title={labelsAsPlaceholders ? '' : label}
placeholder={labelsAsPlaceholders ? label : ''}
description={description}
required={required}
onChange={onChange}
Expand Down
2 changes: 2 additions & 0 deletions src/components/FormView.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -110,6 +110,7 @@ const FormView = ({
disabled
valid
formHasErrors={formErrors?.length > 0}
labelsAsPlaceholders={data.labelsAsPlaceholders}
/>
</Grid.Column>
</Grid.Row>
Expand Down Expand Up @@ -155,6 +156,7 @@ const FormView = ({
}
valid={isValidField(name)}
formHasErrors={formErrors?.length > 0}
labelsAsPlaceholders={data.labelsAsPlaceholders}
/>
</Grid.Column>
</Grid.Row>
Expand Down
5 changes: 5 additions & 0 deletions src/formSchema.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,7 @@ export default () => {
'captcha',
'store',
'send',
'labelsAsPlaceholders',
],
},
],
Expand Down Expand Up @@ -107,6 +108,10 @@ export default () => {
type: 'boolean',
title: intl.formatMessage(messages.send),
},
labelsAsPlaceholders: {
type: 'boolean',
title: 'Use labels as placeholders',
},
},
required: ['default_to', 'default_from', 'default_subject'],
};
Expand Down