-
Notifications
You must be signed in to change notification settings - Fork 86
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.
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
Add a POST form to the landing page #144
Conversation
Add configuration to the landing page generator to allow a POST form. * Reduce header font size. Signed-off-by: SuperQ <[email protected]>
👍 Good enough for this purpose :) If you wanted to avoid the input length business, you could also just put the label and input on separate lines (like it's done in most forms nowadays), but this way is also fine. |
Yea, I was just trying to keep the inline format. |
<div> | ||
<form action="{{ .Form.Action}}"> | ||
{{ range .Form.Inputs }} | ||
<label>{{ .Label }}:</label> <input type="{{ .Type }}" name="{{ .Name }}" placeholder=" .Placeholder }}" value="{{ .Value }}"><br> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
{{ is missing for placeholder
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, oops, will followup.
My comment was still pending.. |
* [FEATURE] Client TLS: Add option to require a specific Subject Alternate Names #126 * [FEATURE] Add a POST form to the landing page #144 * [FEATURE] Add generic customization to landing page #146 * [ENHANCEMENT] Add a Content-Type header to the landing page #142 * [BUGFIX] Fix Nil pointer references for WebSystemdSocket #127 Signed-off-by: SuperQ <[email protected]>
Add configuration to the landing page generator to allow a POST form.