Merge ~ya-bo-ng/maas:improve-vanilla-help-text into maas:master

Proposed by Anthony Dillon
Status: Merged
Approved by: Anthony Dillon
Approved revision: c385a3c0558a7698f50020dd88cf3f0331d92089
Merge reported by: MAAS Lander
Merged at revision: not available
Proposed branch: ~ya-bo-ng/maas:improve-vanilla-help-text
Merge into: maas:master
Diff against target: 39 lines (+4/-4)
3 files modified
src/maasserver/static/js/angular/directives/tests/test_power_parameters.js (+2/-2)
src/maasserver/static/scss/_patterns_card.scss (+1/-1)
src/maasserver/templates/maasserver/form_field.html (+1/-1)
Reviewer Review Type Date Requested Status
Lilyana Videnova (community) Approve
MAAS Lander Approve
Newell Jensen (community) Approve
Review via email: mp+363385@code.launchpad.net

Commit message

Improve the Vanilla help text in settings

Description of the change

Add Vanilla help classes to the help text in settings forms.

To post a comment you must log in.
Revision history for this message
MAAS Lander (maas-lander) wrote :

UNIT TESTS
-b improve-vanilla-help-text lp:~ya-bo-ng/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: FAILED
LOG: http://maas-ci-jenkins.internal:8080/job/maas/job/branch-tester/5113/console
COMMIT: 3450fd6bc1f5c0fc81a64a10b4b3d1581a73664a

review: Needs Fixing
1b43e37... by Anthony Dillon

White space to kick off CI

b44d8d2... by Anthony Dillon

Update test

c385a3c... by Anthony Dillon

Remove fit

Revision history for this message
Newell Jensen (newell-jensen) wrote :

LGTM

review: Approve
Revision history for this message
MAAS Lander (maas-lander) wrote :

UNIT TESTS
-b improve-vanilla-help-text lp:~ya-bo-ng/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: SUCCESS
COMMIT: c385a3c0558a7698f50020dd88cf3f0331d92089

review: Approve
Revision history for this message
Lilyana Videnova (lilyanavidenova) wrote :

lgtm!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/src/maasserver/static/js/angular/directives/tests/test_power_parameters.js b/src/maasserver/static/js/angular/directives/tests/test_power_parameters.js
2index 1836c90..d760a12 100644
3--- a/src/maasserver/static/js/angular/directives/tests/test_power_parameters.js
4+++ b/src/maasserver/static/js/angular/directives/tests/test_power_parameters.js
5@@ -195,8 +195,8 @@ describe("maasPowerParameters", function() {
6 var directive = compileDirective("powerTypes", "value");
7 var select = directive.find("select");
8 expect(select.attr("data-ng-model")).toBe("ngModel.type");
9- expect(select.attr("data-ng-options")).toBe(
10- "type as type.description for type " +
11+ expect(select.attr("data-ng-options")).toBe("type as " +
12+ "type.description for type " +
13 "in maasPowerParameters track by type.name");
14 expect(select.attr("data-ng-disabled")).toBe(
15 "ngDisabled || ngModel.in_pod");
16diff --git a/src/maasserver/static/scss/_patterns_card.scss b/src/maasserver/static/scss/_patterns_card.scss
17index 813f5cc..c977eba 100644
18--- a/src/maasserver/static/scss/_patterns_card.scss
19+++ b/src/maasserver/static/scss/_patterns_card.scss
20@@ -20,4 +20,4 @@
21 margin-top: - $sp-unit * 2;
22 }
23 }
24-}
25\ No newline at end of file
26+}
27diff --git a/src/maasserver/templates/maasserver/form_field.html b/src/maasserver/templates/maasserver/form_field.html
28index a50a5af..7497203 100644
29--- a/src/maasserver/templates/maasserver/form_field.html
30+++ b/src/maasserver/templates/maasserver/form_field.html
31@@ -22,6 +22,6 @@
32 <strong>Error:</strong> {{ field.errors }}
33 </span>
34 {% endif %}
35- <p>{% if field.help_text %}{{ field.help_text }}{% endif %}</p>
36+ <p class="p-form-help-text">{% if field.help_text %}{{ field.help_text }}{% endif %}</p>
37 </div>
38 </li>
39\ No newline at end of file

Subscribers

People subscribed via source and target branches