[PATCH 4 of 8] admin: e-mail: display settings as read-only input fields

[Mads Kiilerich]

On 07/22/2015 09:50 PM, Thomas De Schampheleire wrote:
> # HG changeset patch
> # User Thomas De Schampheleire <thomas.de.schampheleire at gmail.com>
> # Date 1437509310 -7200
> #      Tue Jul 21 22:08:30 2015 +0200
> # Node ID c8e5401db62afbfe417a35b10189bf85aff97726
> # Parent  07fc479b91cef2b8f6240d1b64baabdf3caebdc0
> admin: e-mail: display settings as read-only input fields
>
> Using a definition list to display the e-mail settings has following
> disadvantages:
> - different look&feel than the other admin settings, including the equally
>    read-only hooks page
> - there are only two elements in a definition list (the item and its value)
>    and it is therefore difficult to add more clarification about these items
>
> Instead, follow the same strategy as on the hooks page, showing each setting
> in a read-only text input field.
>
> diff --git a/kallithea/templates/admin/settings/settings_email.html b/kallithea/templates/admin/settings/settings_email.html
> --- a/kallithea/templates/admin/settings/settings_email.html
> +++ b/kallithea/templates/admin/settings/settings_email.html
> @@ -1,4 +1,3 @@
> -<dl class="dl-horizontal">
>   <%
>    elems = [
>       (_('Email prefix'), 'email_prefix'),
> @@ -15,15 +14,25 @@
>       (_('SMTP auth'), 'smtp_auth'),
>    ]
>   %>
> -%for dt, dd in elems:
> -  <dt style="width:150px; text-align: left">${dt}:</dt>
> -  <dd style="margin-left: 160px">${c.ini.get(dd)}</dd>
> -%endfor
> -</dl>
>   
>   ${h.form(url('admin_settings_email'), method='post')}
>       <div class="form">
>   
> +        <h4>${_('Settings (read-only)')}</h4>

Also, how about giving a hint of the reason instead of the consequence: 
that this is showing values from the .ini file.

/Mads