Skip to content

feat: add a Select template with editable options - #999

Open
SmrutiSawant wants to merge 13 commits into
frappe:developfrom
SmrutiSawant:feat-form-select
Open

SmrutiSawant wants to merge 13 commits into
frappe:developfrom
SmrutiSawant:feat-form-select

Conversation

@SmrutiSawant

@SmrutiSawant SmrutiSawant commented Oct 26, 2026 •

Copy link
Copy Markdown
Collaborator

Depends on #997 . This PR includes its commits until that one merges.

What

Adds a Select (dropdown) to Form parts. It comes with a label and a few options to start with.

When you select the dropdown, you can set:

  • Name: what the answer is saved as
  • Required: whether an option must be picked before the form can be sent
  • Options: the choices in the list.
  • Default: the option shown before the visitor picks one

Demo

select.mp4

Input, textarea and select fields now show Name and Required in Input
Options, so you no longer have to set them in HTML Attributes. These
settings read customAttributes first, where the form templates keep
them, and keep both copies in sync when edited, since published pages
let customAttributes win. Type is hidden on textareas and selects, and
Placeholder on selects, where they do nothing.

The Label template's input now has the id its label points at.
@greptile-apps

greptile-apps Bot commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] The PR appears safe to merge.

Reviews (4) · Last reviewed commit: "fix: only rename options of a select ins..." · Reviewed by Greptile

Comment thread frontend/src/block.ts Resolved
Comment thread frontend/src/block.ts Resolved
@codecov

codecov Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.10%. Comparing base (10faafe) to head (f969147).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #999      +/-   ##
===========================================
+ Coverage    76.07%   76.10%   +0.03%     
===========================================
  Files          131      131              
  Lines        13405    13423      +18     
===========================================
+ Hits         10198    10216      +18     
  Misses        3207     3207              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mergify

mergify Bot commented Nov 10, 2026

Copy link
Copy Markdown
Contributor

Tick the box to add this pull request to the merge queue (same as @mergifyio queue). If it does not match the queue conditions yet, it enters the queue once it does.

  • Queue this pull request

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant