From f077b68309285e18fbbc3f0c037e5eb717de1966 Mon Sep 17 00:00:00 2001 From: wangchunsheng Date: Wed, 2 Mar 2011 15:53:28 +0000 Subject: [PATCH] * split the css and js. * rewrite the ui code. --- module/testtask/control.php | 1 + module/testtask/css/cases.css | 0 module/testtask/css/results.css | 1 + module/testtask/css/runcase.css | 1 + module/testtask/js/cases.js | 22 +++++ module/testtask/js/linkcase.js | 7 ++ module/testtask/view/browse.html.php | 84 +++++++++--------- module/testtask/view/cases.html.php | 80 +++++++---------- module/testtask/view/create.html.php | 87 +++++++++---------- module/testtask/view/edit.html.php | 82 +++++++++--------- module/testtask/view/linkcase.html.php | 113 +++++++++++-------------- module/testtask/view/results.html.php | 48 +++++------ module/testtask/view/runcase.html.php | 8 +- module/testtask/view/view.html.php | 98 +++++++++++---------- 14 files changed, 307 insertions(+), 325 deletions(-) create mode 100644 module/testtask/css/cases.css create mode 100644 module/testtask/css/results.css create mode 100644 module/testtask/css/runcase.css create mode 100644 module/testtask/js/cases.js create mode 100644 module/testtask/js/linkcase.js diff --git a/module/testtask/control.php b/module/testtask/control.php index ae70310dd1..cfebe43668 100644 --- a/module/testtask/control.php +++ b/module/testtask/control.php @@ -181,6 +181,7 @@ class testtask extends control $this->view->browseType = $browseType; $this->view->taskID = $taskID; $this->view->moduleID = $moduleID; + $this->view->treeClass = $browseType == 'bymodule' ? '' : 'hidden'; $this->display(); } diff --git a/module/testtask/css/cases.css b/module/testtask/css/cases.css new file mode 100644 index 0000000000..e69de29bb2 diff --git a/module/testtask/css/results.css b/module/testtask/css/results.css new file mode 100644 index 0000000000..620aa31b05 --- /dev/null +++ b/module/testtask/css/results.css @@ -0,0 +1 @@ +body{background:white} diff --git a/module/testtask/css/runcase.css b/module/testtask/css/runcase.css new file mode 100644 index 0000000000..620aa31b05 --- /dev/null +++ b/module/testtask/css/runcase.css @@ -0,0 +1 @@ +body{background:white} diff --git a/module/testtask/js/cases.js b/module/testtask/js/cases.js new file mode 100644 index 0000000000..cdb93ad52e --- /dev/null +++ b/module/testtask/js/cases.js @@ -0,0 +1,22 @@ +function checkall(checker) +{ + $('input').each(function() + { + $(this).attr("checked", checker.checked) + }); +} + +function browseByModule(active) +{ + $('.side').removeClass('hidden'); + $('.divider').removeClass('hidden'); + $('#bymoduleTab').addClass('active'); + $('#' + active + 'Tab').removeClass('active'); +} + +$(document).ready(function() +{ + $("a.iframe").colorbox({width:900, height:600, iframe:true, transition:'none'}); + $('#' + browseType + 'Tab').addClass('active'); + $('#module' + moduleID).addClass('active'); +}); diff --git a/module/testtask/js/linkcase.js b/module/testtask/js/linkcase.js new file mode 100644 index 0000000000..f83e62a9a5 --- /dev/null +++ b/module/testtask/js/linkcase.js @@ -0,0 +1,7 @@ +function checkall(checker) +{ + $('input').each(function() + { + $(this).attr("checked", checker.checked) + }); +} diff --git a/module/testtask/view/browse.html.php b/module/testtask/view/browse.html.php index 745b5756c6..b8d0cd8924 100644 --- a/module/testtask/view/browse.html.php +++ b/module/testtask/view/browse.html.php @@ -12,47 +12,45 @@ ?> -
- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
-
testtask->browse;?>
-
testtask->create);?>
-
idAB;?>testtask->name;?>testtask->project;?>testtask->build;?>testtask->owner;?>testtask->begin;?>testtask->end;?>statusAB;?>actions;?>
id"), sprintf('%03d', $task->id));?>id"), $task->name);?>projectName?>build == 'trunk' ? print('Trunk') : print(html::a($this->createLink('build', 'view', "buildID=$task->build"), $task->buildName));?>owner];?>begin?>end?>testtask->statusList[$task->status];?> - id", $lang->testtask->cases); - common::printLink('testtask', 'linkcase', "taskID=$task->id", $lang->testtask->linkCaseAB); - common::printLink('testtask', 'edit', "taskID=$task->id", $lang->edit); - common::printLink('testtask', 'delete', "taskID=$task->id", $lang->delete, 'hiddenwin'); - ?> -
-
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
+
testtask->browse;?>
+
testtask->create);?>
+
idAB;?>testtask->name;?>testtask->project;?>testtask->build;?>testtask->owner;?>testtask->begin;?>testtask->end;?>statusAB;?>actions;?>
id"), sprintf('%03d', $task->id));?>id"), $task->name);?>projectName?>build == 'trunk' ? print('Trunk') : print(html::a($this->createLink('build', 'view', "buildID=$task->build"), $task->buildName));?>owner];?>begin?>end?>testtask->statusList[$task->status];?> + id", $lang->testtask->cases); + common::printLink('testtask', 'linkcase', "taskID=$task->id", $lang->testtask->linkCaseAB); + common::printLink('testtask', 'edit', "taskID=$task->id", $lang->edit); + common::printLink('testtask', 'delete', "taskID=$task->id", $lang->delete, 'hiddenwin'); + ?> +
diff --git a/module/testtask/view/cases.html.php b/module/testtask/view/cases.html.php index 83b23194a2..ee9593e1f9 100644 --- a/module/testtask/view/cases.html.php +++ b/module/testtask/view/cases.html.php @@ -14,50 +14,36 @@ - -
-
-
- " . $lang->testtask->byModule . " "; - echo "" . html::a($this->inlink('cases', "taskID=$taskID&browseType=all¶m=0"), $lang->testtask->allCases) . ""; - echo "" . html::a($this->inlink('cases', "taskID=$taskID&browseType=assignedtome¶m=0"), $lang->testtask->assignedToMe) . ""; - ?> -
-
- exportCSV, $lang->setFileName); - common::printLink('testtask', 'linkcase', "taskID=$task->id", $lang->testtask->linkCase); - echo html::a($this->session->testtaskList, $lang->goback); - ?> -
+ +
+
+ " . $lang->testtask->byModule . " "; + echo "" . html::a($this->inlink('cases', "taskID=$taskID&browseType=all¶m=0"), $lang->testtask->allCases) . ""; + echo "" . html::a($this->inlink('cases', "taskID=$taskID&browseType=assignedtome¶m=0"), $lang->testtask->assignedToMe) . ""; + ?> +
+
+ exportCSV, $lang->setFileName); + common::printLink('testtask', 'linkcase', "taskID=$task->id", $lang->testtask->linkCase); + echo html::a($this->session->testtaskList, $lang->goback); + ?>
-
' id='mainbox'> -
-
+ + + + + + +
+
+
+
id");?>' target='hiddenwin'> @@ -109,15 +95,7 @@ function browseByModule(active)
- - -
' id='treebox'> -
-
-
- - +
diff --git a/module/testtask/view/create.html.php b/module/testtask/view/create.html.php index edd5082eba..13c7531881 100644 --- a/module/testtask/view/create.html.php +++ b/module/testtask/view/create.html.php @@ -13,51 +13,44 @@ - -
-
- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
testtask->create;?>
testtask->project;?>
testtask->build;?>
testtask->owner;?>
testtask->begin;?> -
testtask->end;?> -
testtask->status;?>testtask->statusList, '', "class='select-3'");?> -
testtask->name;?> -
testtask->desc;?> -
-
-
+
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
testtask->create;?>
testtask->project;?>
testtask->build;?>
testtask->owner;?>
testtask->begin;?> +
testtask->end;?> +
testtask->status;?>testtask->statusList, '', "class='select-3'");?> +
testtask->name;?> +
testtask->desc;?> +
+
diff --git a/module/testtask/view/edit.html.php b/module/testtask/view/edit.html.php index 4252782d78..41aee9f0b6 100644 --- a/module/testtask/view/edit.html.php +++ b/module/testtask/view/edit.html.php @@ -13,46 +13,44 @@ -
-
- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
testtask->edit;?>
testtask->project;?>project, 'class=select-3');?>
testtask->build;?>build, 'class=select-3');?>
testtask->owner;?>owner, 'class=select-3');?>
testtask->begin;?>begin, "class='text-3 date'");?> -
testtask->end;?>end, "class='text-3 date'");?> -
testtask->status;?>testtask->statusList, $task->status, "class='select-3'");?> -
testtask->name;?>name, "class='text-1'");?> -
testtask->desc;?>desc), "rows=10 class='area-1'");?> -
-
-
+
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
testtask->edit;?>
testtask->project;?>project, 'class=select-3');?>
testtask->build;?>build, 'class=select-3');?>
testtask->owner;?>owner, 'class=select-3');?>
testtask->begin;?>begin, "class='text-3 date'");?> +
testtask->end;?>end, "class='text-3 date'");?> +
testtask->status;?>testtask->statusList, $task->status, "class='select-3'");?> +
testtask->name;?>name, "class='text-1'");?> +
testtask->desc;?>desc), "rows=10 class='area-1'");?> +
+
diff --git a/module/testtask/view/linkcase.html.php b/module/testtask/view/linkcase.html.php index 077ed46c42..32c7d409c7 100644 --- a/module/testtask/view/linkcase.html.php +++ b/module/testtask/view/linkcase.html.php @@ -12,66 +12,55 @@ ?> - -
- -
- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
-
testtask->unlinkedCases;?>
-
session->testtaskList, $lang->goback);?>
-
idAB;?>priAB;?>testcase->title;?>testcase->type;?>openedByAB;?>statusAB;?>testtask->linkVersion;?>
createLink('testcase', 'view', "testcaseID=$case->id"), sprintf('%03d', $case->id));?>pri?> - title . ' ( '; - for($i = $case->version; $i >= 1; $i --) - { - echo html::a($this->createLink('testcase', 'view', "caseID=$case->id&version=$i"), "#$i", '_blank'); - } - echo ')'; - ?> - testcase->typeList[$case->type];?>openedBy];?>testcase->statusList[$case->status];?> - - id]", array_combine(range($case->version, 1), range($case->version, 1)), '', 'style=width:50px');?> -
selectAll;?>
-
-
+ +
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
+
testtask->unlinkedCases;?>
+
session->testtaskList, $lang->goback);?>
+
idAB;?>priAB;?>testcase->title;?>testcase->type;?>openedByAB;?>statusAB;?>testtask->linkVersion;?>
createLink('testcase', 'view', "testcaseID=$case->id"), sprintf('%03d', $case->id));?>pri?> + title . ' ( '; + for($i = $case->version; $i >= 1; $i --) + { + echo html::a($this->createLink('testcase', 'view', "caseID=$case->id&version=$i"), "#$i", '_blank'); + } + echo ')'; + ?> + testcase->typeList[$case->type];?>openedBy];?>testcase->statusList[$case->status];?> + + id]", array_combine(range($case->version, 1), range($case->version, 1)), '', 'style=width:50px');?> +
selectAll;?>
+
diff --git a/module/testtask/view/results.html.php b/module/testtask/view/results.html.php index 7d53123b1f..e96810bed2 100644 --- a/module/testtask/view/results.html.php +++ b/module/testtask/view/results.html.php @@ -11,30 +11,28 @@ */ ?> - -
-

CASE#case->id. $lang->colon . $run->case->title;?>

- - - - - - - - - - - case->steps as $key => $step):?> - stepResults[$step->id];?> - - - - - - - - result == 'blocked' or $stepResult->result == 'fail') break;?> - -
RESULT#id . ' ' . $result->date . " " . $lang->testcase->resultList[$result->caseResult] . '';?>
testcase->stepID;?>testcase->stepDesc;?>testcase->stepExpect;?>testcase->result;?>testcase->real;?>
desc);?>expect);?>testcase->resultList[$stepResult->result];?>real;?>
+

CASE#case->id. $lang->colon . $run->case->title;?>

+ + + + + + + + + + + case->steps as $key => $step):?> + stepResults[$step->id];?> + + + + + + + + result == 'blocked' or $stepResult->result == 'fail') break;?> +
RESULT#id . ' ' . $result->date . " " . $lang->testcase->resultList[$result->caseResult] . '';?>
testcase->stepID;?>testcase->stepDesc;?>testcase->stepExpect;?>testcase->result;?>testcase->real;?>
desc);?>expect);?>testcase->resultList[$stepResult->result];?>real;?>
+
diff --git a/module/testtask/view/runcase.html.php b/module/testtask/view/runcase.html.php index 38b64b2272..3c2cf2cd3c 100644 --- a/module/testtask/view/runcase.html.php +++ b/module/testtask/view/runcase.html.php @@ -11,9 +11,7 @@ */ ?> - -
-
+ @@ -45,5 +43,5 @@
CASE#case->id. $lang->colon . $run->case->title;?>
-
-
+ + diff --git a/module/testtask/view/view.html.php b/module/testtask/view/view.html.php index 8ba8f7a24d..f5fec182c4 100644 --- a/module/testtask/view/view.html.php +++ b/module/testtask/view/view.html.php @@ -12,55 +12,53 @@ ?> -
- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
testtask->view;?>
testtask->name;?>'>name;?> -
testtask->project;?>projectName;?>
testtask->build;?>buildName ? print($task->buildName) : print($task->build);?>
testtask->owner;?>owner];?>
testtask->begin;?>begin;?>
testtask->end;?>end;?>
testtask->status;?>testtask->statusList[$task->status];?>
testtask->desc;?>desc;?>
-
- session->testtaskList ? $this->session->testtaskList : $this->createLink('testtask', 'browse', "productID=$task->product"); - if(!$task->deleted) - { - common::printLink('testtask', 'cases', "taskID=$task->id", $lang->testtask->cases); - common::printLink('testtask', 'linkcase', "taskID=$task->id", $lang->testtask->linkCaseAB); - common::printLink('testtask', 'edit', "taskID=$task->id", $lang->edit); - common::printLink('testtask', 'delete', "taskID=$task->id", $lang->delete, 'hiddenwin'); - } - echo html::a($browseLink, $lang->goback); - ?> -
- + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
testtask->view;?>
testtask->name;?>'>name;?> +
testtask->project;?>projectName;?>
testtask->build;?>buildName ? print($task->buildName) : print($task->build);?>
testtask->owner;?>owner];?>
testtask->begin;?>begin;?>
testtask->end;?>end;?>
testtask->status;?>testtask->statusList[$task->status];?>
testtask->desc;?>desc;?>
+
+ session->testtaskList ? $this->session->testtaskList : $this->createLink('testtask', 'browse', "productID=$task->product"); + if(!$task->deleted) + { + common::printLink('testtask', 'cases', "taskID=$task->id", $lang->testtask->cases); + common::printLink('testtask', 'linkcase', "taskID=$task->id", $lang->testtask->linkCaseAB); + common::printLink('testtask', 'edit', "taskID=$task->id", $lang->edit); + common::printLink('testtask', 'delete', "taskID=$task->id", $lang->delete, 'hiddenwin'); + } + echo html::a($browseLink, $lang->goback); + ?>
+