Closed
Bug 1231419
Opened 9 years ago
Closed 9 years ago
In Web Console, "1 more..." appears for jQuery selections with only 1 result
Categories
(DevTools :: Console, defect)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 1301999
People
(Reporter: rick3162, Unassigned)
References
Details
Attachments
(1 file)
(deleted),
image/jpeg
|
Details |
I just noticed it.
To reproduce (fresh profile Nightly 45.0a1 x64 (2015-12-09) in win10 x64):
- open https://www.instagram.com/instagram/
- open Developer Toolbar(shift+F2) and type 'inject jQuery' (you'll get 'jQuery loaded')
- open Web Console (ctrl+shift+K) and type:
$("a:contains('Load more')")
You'll get
> 'Object { length: 1, prevObject: Object, context: HTMLDocument → instagram, selector: "a:contains('Load more')", 1 more… }'
Clicking '1 more...' shows: https://i.imgur.com/yrrSZsQ.jpg (i.e. see attachment)
There shouldn't be '1 more...' when there's only 1 result.
PS. In Firefox 42 the issue doesn't occur.
https://hg.mozilla.org/integration/fx-team/pushloghtml?fromchange=ca30d9070e53f75b90b9710d2533bb5bf0a74240&tochange=74d91f50bad6807a8bfc7d9f89ee724e2502ec35
Tom Tromey — Bug 1035545 - show wrapped primitives specially in console and variable view; r=vporof,fitzgen
Tom, your opinion?
Flags: needinfo?(ttromey)
OS: Unspecified → All
Hardware: Unspecified → All
Version: unspecified → 45 Branch
Comment 3•9 years ago
|
||
I tried this. I see "1 more...".
Clicking on that, the variable view opens.
The variable view shows that the object in question has a property "0".
So, my inclination is to say there isn't a bug here.
Do you see the 0 property as well?
Flags: needinfo?(rick3162)
Yes, as it's in the screenshot.
So, based on your comment, the '1 more..' means that (apart from the 4 properties currently shown in the line: length, prevObject, context, selector)
there is 1 more property in variable view, the property 0.
Maybe that's because this 5th property doesn't fit to the line?
But, here is another use case -not with jQuery- (found in bug 1035545):
> var x = new String('abc');
> x;
This gives:
> String { "abc", 4 more… }
i.e. https://i.imgur.com/L0Hl4U9.jpg
In this case, if I'm correct, the 4 properties aren't supposed to be displayed in the line, i.e not showing something like:
> String { "abc", 0: "a", 1: "b", 2: "c", length: 3 }
but only inside the variable view, via clicking the "4 more".
So, shouldn't in this case, instead of "4 more.." to show "4 properties"?
In other words, the wording "more" to appear only when some properties don't fit to the current line?
Or, in which cases is the wording "more" supposed to be displayed?
Flags: needinfo?(rick3162)
Comment 6•9 years ago
|
||
(In reply to Kostas from comment #5)
Sorry, forgot about this for a bit.
Feel free to NI me.
> Maybe that's because this 5th property doesn't fit to the line?
TBH I don't know what heuristic is used here.
> So, shouldn't in this case, instead of "4 more.." to show "4 properties"?
> In other words, the wording "more" to appear only when some properties don't
> fit to the current line?
> Or, in which cases is the wording "more" supposed to be displayed?
I see what you mean. It is perhaps a bit confusing to say "4 more" when there
aren't any obvious properties to begin with.
I tend to think that the current approach is ok, on the basis that it isn't
really more than mildly confusing; and that it's better to be a bit terse here.
But I will ask someone more versed in the console than I to look at it.
Comment 7•9 years ago
|
||
I agree with Tom, I think the current approach is the best way to go on this.
Status: UNCONFIRMED → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
Comment 8•8 years ago
|
||
This seems the same problem I fixed in bug 1301999.
Resolution: WONTFIX → DUPLICATE
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•