Version 2.2.1

CHANGELOG.md

@@ -1,6 +1,21 @@
 
 # Changelog
 
+## 2.2.1 (2017-02-07)
+
+  * Fix task editing when no actions are specified
+
+## 2.2.0 (2017-01-27)
+
+  * Add support for Pa11y actions
+  * Update dependencies
+    * pa11y-webservice: ~2.1.2 to ^2.3.0
+
+## 2.1.2 (2016-12-12)
+
+  * Hide the "add" button in readonly mode
+  * Add a contributing guide
+
 ## 2.1.1 (2016-11-20)
 
   * Use arrows instead of plus and minus for collapsibles/expanders

CONTRIBUTING.md

@@ -0,0 +1,21 @@
+
+Contributing Guide
+==================
+
+Thanks for getting involved :tada:
+
+The Pa11y team loves to see new contributors, and we strive to provide a welcoming and inclusive environment. We ask that all contributors read and follow [our code of conduct][code-of-conduct] before joining. If you represent an organisation, then you might find our [guide for companies][companies] helpful.
+
+Our website outlines the many ways that you can contribute to Pa11y:
+
+  - [Help us to talk to our users][communications]
+  - [Help us out with design][designers]
+  - [Help us with our code][developers]
+
+
+
+[code-of-conduct]: http://pa11y.org/contributing/code-of-conduct/
+[communications]: http://pa11y.org/contributing/communications/
+[companies]: http://pa11y.org/contributing/companies/
+[designers]: http://pa11y.org/contributing/designers/
+[developers]: http://pa11y.org/contributing/developers/

README.md

@@ -22,7 +22,7 @@ Pa11y Dashboard is a web interface to the [Pa11y][pa11y] accessibility reporter;
 Setup
 -----
 
-Pa11y Dashboard requires [Node.js][node] 4+ and [PhantomJS][phantom]. See the [Pa11y][pa11y] documentation for detailed instructions on how to install these dependencies on your operating system.
+Pa11y Dashboard requires [Node.js][node] 4+. See the [Pa11y][pa11y] documentation for detailed instructions on how to install this on your operating system.
 
 You'll also need to have [MongoDB][mongo] installed and running. See the [MongoDB install guide][mongo-install] for more information on this.
 
@@ -82,10 +82,12 @@ The boot configurations for Pa11y Dashboard are as follows. Look at the sample J
 This can either be an object containing [Pa11y Webservice configurations][pa11y-webservice-config], or a string which is the base URL of a [Pa11y Webservice][pa11y-webservice] instance you are running separately. If using environment variables, prefix the webservice vars with `WEBSERVICE_`.
 
 
-Development
+Contributing
------------
+------------
 
-To develop Pa11y Dashboard, you'll need to clone the repo and get set up as outlined in the [setup guide](#setup).
+There are many ways to contribute to Pa11y Dashboard, we cover these in the [contributing guide](CONTRIBUTING.md) for this repo.
+
+If you're ready to contribute some code, you'll need to clone the repo and get set up as outlined in the [setup guide](#setup).
 
 You'll need to start the application in test mode with:
 
@@ -126,7 +128,7 @@ We also maintain a [migration guide](MIGRATION.md) to help you migrate.
 | :grey_question: | Major Version | Last Minor Release | Node.js Versions | Support End Date |
 | :-------------- | :------------ | :----------------- | :--------------- | :--------------- |
 | :heart:         | 2             | N/A                | 4+               | N/A              |
-| :hourglass:     | 1             | 1.12               | 0.10–6           | 2016-12-05       |
+| :skull:         | 1             | 1.12               | 0.10–6           | 2016-12-05       |
 
 If you're opening issues related to these, please mention the version that the issue relates to.
 
@@ -134,8 +136,8 @@ If you're opening issues related to these, please mention the version that the i
 License
 -------
 
-Pa11y Dashboard is licensed under the [GNU General Public License 3.0][info-license].  
+Pa11y Dashboard is licensed under the [GNU General Public License 3.0][info-license].<br/>
-Copyright &copy; 2013–2016, Springer Nature
+Copyright &copy; 2013–2017, Team Pa11y
 
 
 
@@ -157,5 +159,5 @@ Copyright &copy; 2013–2016, Springer Nature
 [info-build]: https://travis-ci.org/pa11y/dashboard
 [shield-license]: https://img.shields.io/badge/license-GPL%203.0-blue.svg
 [shield-node]: https://img.shields.io/badge/node.js%20support-4–6-brightgreen.svg
-[shield-version]: https://img.shields.io/badge/version-2.1.1-blue.svg
+[shield-version]: https://img.shields.io/badge/version-2.2.1-blue.svg
 [shield-build]: https://img.shields.io/travis/pa11y/dashboard/master.svg

package.json

@@ -1,52 +1,53 @@
 {
-	"name": "pa11y-dashboard",
+  "name": "pa11y-dashboard",
-	"version": "2.1.1",
+  "version": "2.2.1",
-	"private": true,
+  "private": true,
-
+  "description": "Pa11y Dashboard is a visual web interface to the Pa11y accessibility reporter",
-	"description": "Pa11y Dashboard is a visual web interface to the Pa11y accessibility reporter",
+  "keywords": [
-	"keywords": [ "accessibility", "analysis", "report" ],
+    "accessibility",
-	"contributors": [
+    "analysis",
-		"Perry Harlock (http://www.phwebs.co.uk/)",
+    "report"
-		"Rowan Manning (http://rowanmanning.com/)"
+  ],
-	],
+  "author": "Team Pa11y",
-
+  "contributors": [
-	"repository": {
+    "Perry Harlock (http://www.phwebs.co.uk/)",
-		"type": "git",
+    "Rowan Manning (http://rowanmanning.com/)"
-		"url": "https://github.com/pa11y/dashboard.git"
+  ],
-	},
+  "repository": {
-	"homepage": "https://github.com/pa11y/dashboard",
+    "type": "git",
-	"bugs": "https://github.com/pa11y/dashboard/issues",
+    "url": "https://github.com/pa11y/dashboard.git"
-	"license": "GPL-3.0",
+  },
-
+  "homepage": "https://github.com/pa11y/dashboard",
-	"engines": {
+  "bugs": "https://github.com/pa11y/dashboard/issues",
-		"node": ">=4"
+  "license": "GPL-3.0",
-	},
+  "engines": {
-	"dependencies": {
+    "node": ">=4"
-		"body-parser": "~1.15",
+  },
-		"chalk": "~1.1",
+  "dependencies": {
-		"compression": "~1.6",
+    "body-parser": "~1.15",
-		"express": "~4.14",
+    "chalk": "~1.1",
-		"express-hbs": "~1.0",
+    "compression": "~1.6",
-		"http-headers": "^3.0.1",
+    "express": "~4.14",
-		"moment": "~2.13",
+    "express-hbs": "~1.0",
-		"pa11y-webservice": "^2.1.2",
+    "http-headers": "^3.0.1",
-		"pa11y-webservice-client-node": "^1.2.1",
+    "moment": "~2.13",
-		"underscore": "~1.8"
+    "pa11y-webservice": "^2.3.0",
-	},
+    "pa11y-webservice-client-node": "^1.2.1",
-	"devDependencies": {
+    "underscore": "~1.8"
-		"bower": "~1.7",
+  },
-		"cheerio": "~0.20",
+  "devDependencies": {
-		"jscs": "^2",
+    "bower": "~1.7",
-		"jshint": "^2",
+    "cheerio": "~0.20",
-		"less": "~2.7",
+    "jscs": "^2",
-		"mocha": "^2",
+    "jshint": "^2",
-		"proclaim": "^3",
+    "less": "~2.7",
-		"request": "^2.74",
+    "mocha": "^2",
-		"uglify-js": "~2.6"
+    "proclaim": "^3",
-	},
+    "request": "^2.74",
-
+    "uglify-js": "~2.6"
-	"scripts": {
+  },
-		"start": "node index.js",
+  "scripts": {
-		"test": "make ci"
+    "start": "node index.js",
-	}
+    "test": "make ci"
+  }
 }

route/new.js

@@ -40,11 +40,21 @@ function route(app) {
 
 	app.express.post('/new', (req, res) => {
 
+		let parsedActions;
+		if (req.body.actions) {
+			parsedActions = req.body.actions.split(/[\r\n]+/)
+				.map(action => {
+					return action.trim();
+				})
+				.filter(action => {
+					return Boolean(action);
+				});
+		}
+
 		let parsedHeaders;
 		if (req.body.headers) {
 			parsedHeaders = httpHeaders(req.body.headers, true);
 		}
-		console.log(parsedHeaders);
 
 		const newTask = {
 			name: req.body.name,
@@ -53,6 +63,7 @@ function route(app) {
 			ignore: req.body.ignore || [],
 			timeout: req.body.timeout || undefined,
 			wait: req.body.wait || undefined,
+			actions: parsedActions,
 			username: req.body.username || undefined,
 			password: req.body.password || undefined,
 			headers: parsedHeaders,
@@ -73,6 +84,8 @@ function route(app) {
 					});
 					return standard;
 				});
+				newTask.actions = req.body.actions;
+				newTask.headers = req.body.headers;
 				return res.render('new', {
 					error: err,
 					standards: standards,

route/task/edit.js

@@ -13,7 +13,7 @@
 // You should have received a copy of the GNU General Public License
 // along with Pa11y Dashboard.  If not, see <http://www.gnu.org/licenses/>.
 
-/*jshint maxcomplexity:10*/
+/*jshint maxcomplexity:12*/
 
 'use strict';
 
@@ -43,6 +43,7 @@ function route(app) {
 				});
 				return standard;
 			});
+			task.actions = (task.actions ? task.actions.join('\n') : '');
 			res.render('task/edit', {
 				edited: (typeof req.query.edited !== 'undefined'),
 				standards: standards,
@@ -57,10 +58,23 @@ function route(app) {
 			if (err) {
 				return next();
 			}
+			const originalActions = req.body.actions;
 			const originalHeaders = req.body.headers;
 			req.body.ignore = req.body.ignore || [];
 			req.body.timeout = req.body.timeout || undefined;
 			req.body.wait = req.body.wait || undefined;
+			if (req.body.actions) {
+				req.body.actions = req.body.actions.split(/[\r\n]+/)
+					.map(action => {
+						return action.trim();
+					})
+					.filter(action => {
+						return Boolean(action);
+					});
+			}
+			if (!req.body.actions) {
+				req.body.actions = [];
+			}
 			req.body.username = req.body.username || undefined;
 			req.body.password = req.body.password || undefined;
 			req.body.hideElements = req.body.hideElements || undefined;
@@ -71,6 +85,7 @@ function route(app) {
 					task.ignore = req.body.ignore;
 					task.timeout = req.body.timeout;
 					task.wait = req.body.wait;
+					task.actions = originalActions;
 					task.username = req.body.username;
 					task.password = req.body.password;
 					task.headers = originalHeaders;

test/integration/route/new.js

@@ -70,6 +70,11 @@ describe('GET /new', function() {
 			assert.strictEqual(field.attr('value'), '');
 		});
 
+		it('should have an "actions" field', function() {
+			const field = this.form.find('textarea[name=actions]').eq(0);
+			assert.isDefined(field);
+		});
+
 		it('should have a "username" field', function() {
 			const field = this.form.find('input[name=username]').eq(0);
 			assert.isDefined(field);

test/integration/route/task/edit.js

@@ -71,6 +71,11 @@ describe('GET /<task-id>/edit', function() {
 			assert.strictEqual(field.attr('value'), '0');
 		});
 
+		it('should have an "actions" field', function() {
+			const field = this.form.find('textarea[name=actions]').eq(0);
+			assert.isDefined(field);
+		});
+
 		it('should have a disabled "standard" field', function() {
 			const field = this.form.find('select[name=standard]').eq(0);
 			assert.isDefined(field);

view/new.html

@@ -84,6 +84,19 @@ along with Pa11y Dashboard.  If not, see <http://www.gnu.org/licenses/>.
 		</div>
 	</div>
 
+	<div class="form-group clearfix">
+		<div class="row">
+			<div class="col-md-4 col-sm-4 col-xs-6">
+				<label class="control-label" for="new-task-actions">
+					Task Actions
+					(<a href="https://github.com/pa11y/pa11y#actions">see Pa11y documentation</a>)
+				</label>
+				<textarea class="form-control" id="new-task-actions" name="actions" aria-describedby="action-detail" placeholder="E.g. Click element #login-button">{{task.actions}}</textarea>
+				<em id="action-detail">(one action per line)</em>
+			</div>
+		</div>
+	</div>
+
 	<div class="form-group clearfix">
 		<div class="row">
 			<div class="col-md-4 col-sm-4 col-xs-6">

view/partial/page-footer.html

@@ -18,7 +18,7 @@ along with Pa11y Dashboard.  If not, see <http://www.gnu.org/licenses/>.
 	<div class="footer" role="contentinfo">
 		<div class="container">
 			<div class="col-md-5">
-				<small>&copy; {{year}} Springer Nature.<br/>Pa11y Dashboard is licensed under the GNU General Public License 3.0.<br/>Version {{version}}</small>
+				<small>&copy; 2013–{{year}} Team Pa11y.<br/>Pa11y Dashboard is licensed under the GNU General Public License 3.0.<br/>Version {{version}}</small>
 			</div>
 			<div class="col-md-7 clearfix">
 				<ul class="crunch-bottom floated-list nav">

view/partial/tasks.html

@@ -16,19 +16,15 @@ along with Pa11y Dashboard.  If not, see <http://www.gnu.org/licenses/>.
 }}
 
 <ul class="list-unstyled clearfix crunch-bottom">
-	<li class="col-md-4 col-sm-6 task-card add-task">
+
-		{{#if readonly}}
+	{{#unless readonly}}
-			<span class="well task-card-link crunch-bottom">
+	        <li class="col-md-4 col-sm-6 task-card add-task">
-				<p class="h3 crunch">Add new URL</p>
-				<p class="supersize-me crunch">+</p>
-			</span>
-		{{else}}
 			<a class="well task-card-link crunch-bottom" data-role="add-task" href="/new" data-test="add-task">
 				<p class="h3 crunch">Add new URL</p>
 				<p class="supersize-me crunch">+</p>
 			</a>
-		{{/if}}
+		</li>
-	</li>
+	{{/unless}}
 	{{#each tasks}}
 		<li class="col-md-4 col-sm-6 task-card" data-test="task" data-role="task" data-keywords="{{lowercase name}} {{lowercase standard}} {{simplify-url url}}">
 			<a class="well task-card-link crunch-bottom" title="Details for URL {{simplify-url url}}" href="{{href}}">
@@ -49,14 +45,14 @@ along with Pa11y Dashboard.  If not, see <http://www.gnu.org/licenses/>.
 			</a>
 			{{#unless ../readonly}}
 				<div class="btn-group options-button text-right">
-						<button type="button" class="btn btn-info btn-xs dropdown-toggle" data-toggle="dropdown"><span class="sr-only">Options</span><span class="glyphicon glyphicon-cog"></span></button>
+					<button type="button" class="btn btn-info btn-xs dropdown-toggle" data-toggle="dropdown"><span class="sr-only">Options</span><span class="glyphicon glyphicon-cog"></span></button>
-						<ul class="dropdown-menu pull-right" role="menu">
+					<ul class="dropdown-menu pull-right" role="menu">
-							<li><a href="{{href}}/edit">Edit this task</a></li>
+						<li><a href="{{href}}/edit">Edit this task</a></li>
-							<li><a href="{{href}}/delete">Delete this task</a></li>
+						<li><a href="{{href}}/delete">Delete this task</a></li>
-							<li class="divider"></li>
+						<li class="divider"></li>
-							<li><a href="{{href}}/run" data-test="run-task">Run Pa11y</a></li>
+						<li><a href="{{href}}/run" data-test="run-task">Run Pa11y</a></li>
-						</ul>
+					</ul>
-					</div>
+				</div>
 			{{/unless}}
 		</li>
 	{{/each}}

view/task/edit.html

@@ -96,6 +96,19 @@ along with Pa11y Dashboard.  If not, see <http://www.gnu.org/licenses/>.
 		</div>
 	</div>
 
+	<div class="form-group clearfix">
+		<div class="row">
+			<div class="col-md-4 col-sm-4 col-xs-6">
+				<label class="control-label" for="new-task-actions">
+					Task Actions
+					(<a href="https://github.com/pa11y/pa11y#actions">see Pa11y documentation</a>)
+				</label>
+				<textarea class="form-control" id="new-task-actions" name="actions" aria-describedby="action-detail" placeholder="E.g. Click element #login-button">{{task.actions}}</textarea>
+				<em id="action-detail">(one action per line)</em>
+			</div>
+		</div>
+	</div>
+
 	<div class="form-group clearfix">
 		<div class="row">
 			<div class="col-md-4 col-sm-4 col-xs-6">