From df6c453c862045ddbb253dda738b2865d9a51830 Mon Sep 17 00:00:00 2001 From: mabashian Date: Mon, 12 Nov 2018 08:48:32 -0500 Subject: [PATCH] Fix long name tooltip. Fixed bug adding new node before finishing adding new link. Fixed template list column layout. Ensure that we're getting 200 workflow nodes per GET request --- awx/ui/client/src/templates/templates.service.js | 4 ++-- .../workflow-chart/workflow-chart.directive.js | 2 +- .../forms/workflow-node-form.partial.html | 12 ++++++------ .../workflow-maker/workflow-maker.controller.js | 8 ++++++++ .../workflow-results/workflow-results.controller.js | 4 ++-- .../src/workflow-results/workflow-results.route.js | 2 +- 6 files changed, 20 insertions(+), 12 deletions(-) diff --git a/awx/ui/client/src/templates/templates.service.js b/awx/ui/client/src/templates/templates.service.js index 1a63cb6553..bb1edfe661 100644 --- a/awx/ui/client/src/templates/templates.service.js +++ b/awx/ui/client/src/templates/templates.service.js @@ -133,10 +133,10 @@ export default ['Rest', 'GetBasePath', '$q', 'NextPage', function(Rest, GetBaseP getWorkflowJobTemplateNodes: function(id, page) { var url = GetBasePath('workflow_job_templates'); - url = url + id + '/workflow_nodes'; + url = url + id + '/workflow_nodes?page_size=200'; if(page) { - url += '/?page=' + page; + url += '/&page=' + page; } Rest.setUrl(url); diff --git a/awx/ui/client/src/templates/workflows/workflow-chart/workflow-chart.directive.js b/awx/ui/client/src/templates/workflows/workflow-chart/workflow-chart.directive.js index 18b562982d..c2f969cad3 100644 --- a/awx/ui/client/src/templates/workflows/workflow-chart/workflow-chart.directive.js +++ b/awx/ui/client/src/templates/workflows/workflow-chart/workflow-chart.directive.js @@ -832,7 +832,7 @@ export default ['$state','moment', '$timeout', '$window', '$filter', 'Rest', 'Ge .attr("height", tipHeight+20) .attr("class", "WorkflowChart-tooltip") .html(function(){ - return "
" + $filter('sanitize')(resourceName) + "
"; + return "
" + $filter('sanitize')(resourceName) + "
"; }); } diff --git a/awx/ui/client/src/templates/workflows/workflow-maker/forms/workflow-node-form.partial.html b/awx/ui/client/src/templates/workflows/workflow-maker/forms/workflow-node-form.partial.html index ee64faadb9..0732e14819 100644 --- a/awx/ui/client/src/templates/workflows/workflow-maker/forms/workflow-node-form.partial.html +++ b/awx/ui/client/src/templates/workflows/workflow-maker/forms/workflow-node-form.partial.html @@ -7,23 +7,23 @@
-
- +
+
-
+
No records matched your search.
-
PLEASE ADD ITEMS TO THIS LIST
+
PLEASE ADD ITEMS TO THIS LIST
- - diff --git a/awx/ui/client/src/templates/workflows/workflow-maker/workflow-maker.controller.js b/awx/ui/client/src/templates/workflows/workflow-maker/workflow-maker.controller.js index 065c55835c..8b5d40c4e9 100644 --- a/awx/ui/client/src/templates/workflows/workflow-maker/workflow-maker.controller.js +++ b/awx/ui/client/src/templates/workflows/workflow-maker/workflow-maker.controller.js @@ -306,6 +306,10 @@ export default ['$scope', 'TemplatesService', $scope.cancelNodeForm(); } + if ($scope.linkConfig) { + $scope.cancelLinkForm(); + } + $scope.graphState.arrayOfNodesForChart.push({ index: $scope.graphState.arrayOfNodesForChart.length, id: workflowMakerNodeIdCounter, @@ -342,6 +346,10 @@ export default ['$scope', 'TemplatesService', $scope.cancelNodeForm(); } + if ($scope.linkConfig) { + $scope.cancelLinkForm(); + } + $scope.graphState.arrayOfNodesForChart.push({ index: $scope.graphState.arrayOfNodesForChart.length, id: workflowMakerNodeIdCounter, diff --git a/awx/ui/client/src/workflow-results/workflow-results.controller.js b/awx/ui/client/src/workflow-results/workflow-results.controller.js index 00328f4f30..ea1ff2d49b 100644 --- a/awx/ui/client/src/workflow-results/workflow-results.controller.js +++ b/awx/ui/client/src/workflow-results/workflow-results.controller.js @@ -1,9 +1,9 @@ export default ['workflowData', 'workflowResultsService', 'workflowDataOptions', 'jobLabels', 'workflowNodes', '$scope', 'ParseTypeChange', - 'ParseVariableString', 'count', '$state', 'i18n', 'WorkflowChartService', + 'ParseVariableString', 'count', '$state', 'i18n', 'WorkflowChartService', '$filter', 'moment', function(workflowData, workflowResultsService, workflowDataOptions, jobLabels, workflowNodes, $scope, ParseTypeChange, - ParseVariableString, count, $state, i18n, WorkflowChartService, + ParseVariableString, count, $state, i18n, WorkflowChartService, $filter, moment) { var runTimeElapsedTimer = null; let nodeIdToChartNodeIdMapping = {}; diff --git a/awx/ui/client/src/workflow-results/workflow-results.route.js b/awx/ui/client/src/workflow-results/workflow-results.route.js index 84562c4783..06fbb5f814 100644 --- a/awx/ui/client/src/workflow-results/workflow-results.route.js +++ b/awx/ui/client/src/workflow-results/workflow-results.route.js @@ -49,7 +49,7 @@ export default { // flashing as rest data comes in. Provides the list of workflow nodes workflowNodes: ['workflowData', 'Rest', '$q', function(workflowData, Rest, $q) { var defer = $q.defer(); - Rest.setUrl(workflowData.related.workflow_nodes + '?order_by=id'); + Rest.setUrl(workflowData.related.workflow_nodes + '?order_by=id&page_size=200'); Rest.get() .then(({data}) => { if(data.next) {
+ +