openstack
/
horizon
Code Issues Proposed changes

UX: Add IPSec Policy Modal Hint List is hard to read 

project/vpn: Add IPSec Policy: HInt test has a bulleted list that
looks odd and is hard to read, see here: https://i.imgur.com/1jlPBBj.png

Took the opportunity to also clean up the IKE Policy and VPN views as
well.

Closes-bug: #1570526

Change-Id: Idfff6c48a237ce703756b3ab8e0dd37705a35b22
This commit is contained in:
Diana Whitten 2016-07-29 11:32:49 -07:00
parent 7a3005abc3
commit 79a86b372f
6 changed files with 59 additions and 57 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
openstack_dashboard/dashboards/project/vpn/templates/vpn/_add_ike_policy_help.html Normal file
View File

@ -0,0 +1,20 @@
{% load i18n %}
<p>{% trans "Create IKE Policy for current project." %}</p>
<p>{% trans "An IKE policy is an association of the following attributes:" %}</p>
<dl class="dl-readable">
<dt>{% trans 'Authorization algorithm' %}</dt>
<dd>{% trans 'Auth algorithm limited to SHA1 only.' %}</dd>
<dt>{% trans 'Encryption algorithm' %}</dt>
<dd>{% trans 'The type of algorithm (3des, aes-128, aes-192, aes-256) used in the IKE Policy.' %}</dd>
<dt>{% trans 'IKE version' %}</dt>
<dd>{% trans 'The type of version (v1/v2) that needs to be filtered.' %}</dd>
<dt>{% trans 'Lifetime' %}</dt>
<dd>{% trans "Life time consists of units and value. Units in 'seconds' and the default value is 3600." %}</dd>
<dt>{% trans 'Perfect Forward Secrecy' %}</dt>
<dd>{% trans 'PFS limited to using Diffie-Hellman groups 2, 5 (default) and 14.' %}</dd>
<dt>{% trans 'IKE Phase 1 negotiation mode' %}</dt>
<dd>{% trans "Limited to 'main' mode only." %}</dd>
</dl>
<p>{% trans "All fields are optional." %}</p>

19
openstack_dashboard/dashboards/project/vpn/templates/vpn/_add_ipsec_policy_help.html Normal file
View File

@ -0,0 +1,19 @@
{% load i18n %}
<p>{% trans 'Create IPSec Policy for current project.' %}</p>
<p>{% trans 'An IPSec policy is an association of the following attributes' %}</p>
<dl class="dl-readable">
<dt>{% trans 'Authorization algorithm' %}</dt>
<dd>{% trans 'Auth algorithm limited to SHA1 only.' %}</dd>
<dt>{% trans 'Encapsulation mode' %}</dt>
<dd>{% trans 'The type of IPsec tunnel (tunnel/transport) to be used.' %}</dd>
<dt>{% trans 'Encryption algorithm' %}</dt>
<dd>{% trans 'The type of algorithm (3des, aes-128, aes-192, aes-256) used in the IPSec Policy.' %}</dd>
<dt>{% trans 'Lifetime' %}</dt>
<dd>{% trans "Life time consists of units and value. Units in 'seconds' and the default value is 3600." %}</dd>
<dt>{% trans 'Perfect Forward Secrecy' %}</dt>
<dd>{% trans 'PFS limited to using Diffie-Hellman groups 2, 5 (default) and 14.' %}</dd>
<dt>{% trans 'Transform Protocol' %}</dt>
<dd>{% trans 'The type of protocol (esp, ah, ah-esp) used in IPSec Policy.' %}</dd>
</dl>
<p>{% trans 'All fields are optional.' %}</p>

7
openstack_dashboard/dashboards/project/vpn/templates/vpn/_add_vpn_service_help.html Normal file
View File

@ -0,0 +1,7 @@
{% load i18n %}
<p>{% trans "Create VPN service for current project." %}</p>
<p>{% trans "The VPN service is attached to a router and references to a single subnet to push to a remote site." %}</p>
<p>{% trans "Specify a name, description, router, and subnet for the VPN Service." %}</p>
<p>{% trans "Admin State is UP (True) by default." %}</p>
<p>{% trans "The router, subnet and admin state fields are required. All others are optional." %} </p>

62
openstack_dashboard/dashboards/project/vpn/workflows.py
View File

@ -70,17 +70,7 @@ class AddVPNServiceAction(workflows.Action):
class Meta(object):
name = _("Add New VPN Service")
permissions = ('openstack.services.network',)
help_text = _("Create VPN Service for current project.\n\n"
"The VPN service is attached to a router "
"and references to a single subnet "
"to push to a remote site.\n"
"Specify a name, description, router, and subnet "
"for the VPN Service. "
"Admin State is UP (True) by default.\n\n"
"The router, subnet and admin state "
"fields are required, "
"all others are optional."
)
help_text_template = "project/vpn/_add_vpn_service_help.html"
class AddVPNServiceStep(workflows.Step):
@ -175,27 +165,7 @@ class AddIKEPolicyAction(workflows.Action):
class Meta(object):
name = _("Add New IKE Policy")
permissions = ('openstack.services.network',)
help_text = _("Create IKE Policy for current project.\n\n"
"An IKE policy is an association "
"of the following attributes:\n\n"
"<li>Authorization algorithm: "
"Auth algorithm limited to SHA1 only.</li>"
"<li>Encryption algorithm: "
"The type of algorithm "
"(3des, aes-128, aes-192, aes-256) "
"used in the IKE Policy.</li>"
"<li>IKE version: The type of version (v1/v2) "
"that needs to be filtered.</li>"
"<li>Lifetime: Life time consists of units and value. "
"Units in 'seconds' "
"and the default value is 3600.</li>"
"<li>Perfect Forward Secrecy: "
"PFS limited to using Diffie-Hellman "
"groups 2, 5(default) and 14.</li>"
"<li>IKE Phase 1 negotiation mode: "
"Limited to 'main' mode only.</li>\n"
"All fields are optional."
)
help_text_template = "project/vpn/_add_ike_policy_help.html"
class AddIKEPolicyStep(workflows.Step):
@ -296,29 +266,7 @@ class AddIPSecPolicyAction(workflows.Action):
class Meta(object):
name = _("Add New IPSec Policy")
permissions = ('openstack.services.network',)
help_text = _("Create IPSec Policy for current project.\n\n"
"An IPSec policy is an association "
"of the following attributes:\n\n"
"<li>Authorization algorithm: "
"Auth_algorithm limited to SHA1 only.</li>"
"<li>Encapsulation mode: "
"The type of IPsec tunnel (tunnel/transport) "
"to be used.</li>"
"<li>Encryption algorithm: "
"The type of algorithm "
"(3des, aes-128, aes-192, aes-256) "
"used in the IPSec Policy.</li>"
"<li>Lifetime: Life time consists of units and value. "
"Units in 'seconds' "
"and the default value is 3600.</li>"
"<li>Perfect Forward Secrecy: "
"PFS limited to using Diffie-Hellman "
"groups 2, 5(default) and 14.</li>"
"<li>Transform Protocol: "
"The type of protocol "
"(esp, ah, ah-esp) used in IPSec Policy.</li>\n"
"All fields are optional."
)
help_text_template = 'project/vpn/_add_ipsec_policy_help.html'
class AddIPSecPolicyStep(workflows.Step):
@ -441,8 +389,8 @@ class AddIPSecSiteConnectionAction(workflows.Action):
class Meta(object):
name = _("Add New IPSec Site Connection")
permissions = ('openstack.services.network',)
help_text = _("Create IPSec Site Connection for current project.\n\n"
"Assign a name and description for the "
help_text = _("Create IPSec Site Connection for current "
"project. Assign a name and description for the "
"IPSec Site Connection. "
"All fields in this tab are required."
)

7
openstack_dashboard/static/dashboard/scss/components/_dl_lists.scss Normal file
View File

@ -0,0 +1,7 @@
.dl-readable {
dd {
padding-left: $padding-base-vertical;
padding-bottom: $padding-base-horizontal;
}
}

1
openstack_dashboard/static/dashboard/scss/horizon.scss
View File

@ -20,6 +20,7 @@
@import "components/charts";
@import "components/checkboxes";
@import "components/datepicker";
@import "components/dl_lists";
@import "components/dropdowns";
@import "components/forms";
@import "components/help_panel";