Skip to content

Commit 6405c86

Browse files
committed
Limit for number of jQuery items rendered
1 parent 7323960 commit 6405c86

File tree

2 files changed

+22
-1
lines changed

2 files changed

+22
-1
lines changed

lib/jquery-renderer.js

Lines changed: 16 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,11 @@ module.metadata = {
66
"stability": "stable"
77
};
88

9+
// Addon SDK
910
const { Cu, Ci } = require("chrome");
11+
const { prefs } = require("sdk/simple-prefs");
12+
13+
// Firebug SDK
1014
const { Trace, TraceError } = require("firebug.sdk/lib/core/trace.js").get(module.id);
1115

1216
// DevTools
@@ -51,7 +55,17 @@ var JQueryRenderer = {
5155
// A helper that renders a comma between items if isFirst == false.
5256
let renderSeparator = () => !isFirst && this._text(", ");
5357

58+
// Keep in mind that only specified maximum number of items
59+
// is rendered in the Console panel.
60+
let maxItemsRendered = prefs.maxItemsRendered || 400;
61+
62+
// Iterate over the array and render all items.
63+
var shown = 0;
5464
for (let item of items) {
65+
if (maxItemsRendered-- < 0) {
66+
break;
67+
}
68+
5569
if (item === null) {
5670
emptySlots++;
5771
}
@@ -73,14 +87,15 @@ var JQueryRenderer = {
7387

7488
this.renderData(item, elem);
7589
}
90+
91+
shown++;
7692
}
7793

7894
if (emptySlots) {
7995
renderSeparator();
8096
this.renderEmptySlots(emptySlots, false);
8197
}
8298

83-
let shown = items.length;
8499
if (shown < preview.length) {
85100
this._text(", ");
86101

package.json

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,12 @@
5959
"description": "",
6060
"type": "integer",
6161
"value": 1000
62+
}, {
63+
"name": "maxItemsRendered",
64+
"title": "maxItemsRendered",
65+
"description": "",
66+
"type": "integer",
67+
"value": 400
6268
}, {
6369
"name": "useLint",
6470
"title": "useLint",

0 commit comments

Comments
 (0)