Skip to content
This repository was archived by the owner on Sep 8, 2020. It is now read-only.

Commit 2d43748

Browse files
committed
Merge branch 'v0.14.x-dev'
2 parents a462388 + e3e0548 commit 2d43748

14 files changed

+1597
-390
lines changed

Diff for: .gitignore

+1-1
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,4 @@ node_modules/
33
coverage/
44
junit/
55
dist/
6-
out/
6+
out/

Diff for: README.md

+4-9
Original file line numberDiff line numberDiff line change
@@ -51,9 +51,9 @@ Apply the directive to your form elements:
5151
**Developing Notes:**
5252

5353
* `ng-model` is required, so that the directive knows which model to update.
54-
* `ui-sortable` element should only contain one `ng-repeat` and not any other elements (above or below).
55-
Otherwise the index matching of the generated DOM elements and the `ng-model`'s items will break.
56-
**In other words: The items of `ng-model` must match the indexes of the generated DOM elements.**
54+
* `ui-sortable` element should contain only one `ng-repeat`, but other non-repeater elements above or below may still exist.
55+
Otherwise the index matching of the `ng-model`'s items and the DOM elements generated by the `ng-repeat` will break.
56+
**In other words: The items of `ng-model` must match the indexes of the DOM elements generated by the `ng-repeat`.**
5757
* [`Filters`](https://docs.angularjs.org/guide/filter) that manipulate the model (like [filter](https://docs.angularjs.org/api/ng/filter/filter), [orderBy](https://docs.angularjs.org/api/ng/filter/orderBy), [limitTo](https://docs.angularjs.org/api/ng/filter/limitTo),...) should be applied in the `controller` instead of the `ng-repeat` (refer to [the provided examples](#examples)).
5858
This is the preferred way since it:
5959
- is performance wise better
@@ -90,12 +90,6 @@ The suggested pattern is to use callbacks for emmiting events and altering the s
9090

9191
#### Floating
9292

93-
**Update**: Issue [~~7498~~](bugs.jqueryui.com/ticket/7498) was resolved in jquery-ui v1.11.4.
94-
Calling `angular.element('ui-sortable').sortable('refresh')` (use a more appropriate selector in your use case)
95-
should make jquery-ui-sortable recognize the position and orientation of the existing and any new items.
96-
As a result, since ui-sortable makes a call to `sortable('refresh')` after the sortable items are created by the repeater, it is not any more necessary to use the `ui-floating` property if the orientation of your list is not changing dynamically.
97-
**TL;DR:** If you are using jquery-ui v1.11.4+ and you are not changing the orientation of your list dynamically, then you probably don't need to use `ui-floating` property.
98-
9993
To have a smooth horizontal-list reordering, jquery.ui.sortable needs to detect the orientation of the list.
10094
This detection takes place during the initialization of the plugin (and some of the checks include: whether the first item is floating left/right or if 'axis' parameter is 'x', etc).
10195
There is also a [known issue](bugs.jqueryui.com/ticket/7498) about initially empty horizontal lists.
@@ -210,6 +204,7 @@ For more details about the events check the [jQueryUI API documentation](http://
210204
- [Draggable Handle](http://codepen.io/thgreasi/pen/ihAyr)
211205
- [Drop Zone](http://codepen.io/thgreasi/pen/LVXWeO)
212206
- [Draggable-Sortable like interaction](http://codepen.io/thgreasi/pen/LVVJgK)
207+
- [Static HTML Sorting](http://codepen.io/thgreasi/pen/qdmjwy)
213208

214209
## Integrations
215210
- [ui.bootstrap.accordion](http://plnkr.co/edit/TGIeeEbbvJwpJ3WRqo2z?p=preview)

Diff for: gruntFile.js

+3-2
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,8 @@ module.exports = function(grunt) {
77
require('load-grunt-tasks')(grunt);
88

99
// Default task.
10-
grunt.registerTask('default', ['jshint', 'karma:unit']);
10+
grunt.registerTask('default', ['test']);
11+
grunt.registerTask('test', ['jshint', 'karma:unit']);
1112
grunt.registerTask('serve', ['karma:continuous', 'dist', 'build:gh-pages', 'connect:continuous', 'watch']);
1213
grunt.registerTask('dist', ['ngmin', 'surround:main', 'uglify', 'surround:banner' ]);
1314
grunt.registerTask('coverage', ['jshint', 'karma:coverage']);
@@ -40,7 +41,7 @@ module.exports = function(grunt) {
4041
var testConfig = function(configFile, customOptions) {
4142
var options = { configFile: configFile, singleRun: true };
4243
var travisOptions = process.env.TRAVIS && {
43-
browsers: ['Firefox', 'PhantomJS'],
44+
browsers: ['Chrome', 'Firefox'],
4445
reporters: ['dots', 'coverage', 'coveralls'],
4546
preprocessors: { 'src/*.js': ['coverage'] },
4647
coverageReporter: {

Diff for: package.json

+11-10
Original file line numberDiff line numberDiff line change
@@ -15,21 +15,22 @@
1515
"grunt-contrib-uglify": "0.2.x",
1616
"grunt-contrib-watch": "0.5.x",
1717
"grunt-conventional-changelog": "~1.0.0",
18-
"grunt-karma": "0.6.x",
18+
"grunt-karma": "^0.12.1",
1919
"grunt-ngmin": "0.0.x",
2020
"grunt-surround": "0.1.x",
21-
"karma": "0.10.x",
21+
"jasmine-core": "^2.4.1",
22+
"karma": "^0.13.22",
2223
"karma-chrome-launcher": "0.1.x",
23-
"karma-coffee-preprocessor": "0.1.x",
24-
"karma-coverage": "~0.2.0",
24+
"karma-coffee-preprocessor": "^0.3.0",
25+
"karma-coverage": "^0.5.5",
2526
"karma-coveralls": "~0.1.4",
2627
"karma-firefox-launcher": "0.1.x",
27-
"karma-html2js-preprocessor": "0.1.x",
28-
"karma-jasmine": "0.1.x",
29-
"karma-junit-reporter": "0.2.x",
30-
"karma-phantomjs-launcher": "0.1.x",
31-
"karma-requirejs": "0.2.x",
32-
"karma-script-launcher": "0.1.x",
28+
"karma-html2js-preprocessor": "^0.1.0",
29+
"karma-jasmine": "^0.3.7",
30+
"karma-junit-reporter": "^0.4.0",
31+
"karma-phantomjs-launcher": "^1.0.0",
32+
"karma-requirejs": "^0.2.5",
33+
"karma-script-launcher": "^0.2.0",
3334
"load-grunt-tasks": "0.2.x",
3435
"requirejs": "2.1.x",
3536
"wiredep": "1.8.x"

0 commit comments

Comments
 (0)