1341258 - whitespace-only text node shows up as "o"

Status: RESOLVED FIXED

(DevTools :: Inspector, enhancement, P3)

Description

[Luke]

User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:52.0) Gecko/20100101 Firefox/52.0
Build ID: 20170220231536

Steps to reproduce:

Opened inspector on some html with enter between elements.


Actual results:

Whitespace-only text node shows up as "o" in the inspector... why? 


Expected results:

shouldn't show this bogus "o", I thought there was typo in output code.

Comment 1

[Loic]

Any testcase, please?

Comment 2

[YF (Yang)]

Attachment: Screenshot

Comment 3

[YF (Yang)]

Bug 1304685.


It should have some descriptive, such as tooltip.

Comment 4

[Loic]

Attachment: whitespace.jpg

(In reply to YF (Yang) from comment #3)
> Bug 1304685.
> 
> 
> It should have some descriptive, such as tooltip.

It's already the case in Nightly, I don't think there is something to fix here. :)

Comment 5

[YF (Yang)]

(In reply to Loic from comment #4)
> Created attachment 8839565 [details]
> whitespace.jpg
> 
> (In reply to YF (Yang) from comment #3)
> > Bug 1304685.
> > 
> > 
> > It should have some descriptive, such as tooltip.
> 
> It's already the case in Nightly, I don't think there is something to fix
> here. :)

Looks good, but can be better if the tooltip hot area (trigger) is bigger.

Comment 6

[Julian Descottes [:jdescottes]]

We display whitespace text nodes in the inspector if they have an impact on the layout. Such text nodes often create hard to understand layout issues (gaps between nodes etc...), and most inspectors don't let developers inspect them.

Now the representation can probably be improved to avoid the confusion with the letter "o". We discussed several proposals for the initial implementation in Bug 1304685. My initial idea was https://bug1304685.bmoattachments.org/attachment.cgi?id=8794103
I still like it, because it's hard to mistake it for another element (maybe the fading was too much).

Comment 7

[Julian Descottes [:jdescottes]]

Patrick: what do you think about updating the representation here?

Comment 8

[Patrick Brosset <:pbro>]

(In reply to Julian Descottes [:jdescottes] from comment #7)
> Patrick: what do you think about updating the representation here?
Sure, I'm not against it, especially if the character we chose gets confused with the 'o' letter.
Attachment 8794103 [details] looks good to me especially because it looks exactly like the [...] icon we have in between an opening and closing tag when the element has child nodes. So I see the value of consistency here.

Comment 9

[Julian Descottes [:jdescottes]]

Attachment: Bug 1341258 - update display of whitespace nodes in markup view;r=gl

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/6bf5cd5c076b
update display of whitespace nodes in markup view;r=gl

Comment 11

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/6bf5cd5c076b

Comment 12

[Mohammad Maruf Rahman]

I have reproduced this bug with Nightly 54.0a1 (2017-02-21) (64-bit) on Manjaro Linux 17!

This bug's fix is verified with latest Beta!

Build ID 	20170511200247
User Agent 	Mozilla/5.0 (X11; Linux x86_64; rv:54.0) Gecko/20100101 Firefox/54.0

[testday-20170512]

Comment 13

[Dustin Driver]

Docs update here: https://developer.mozilla.org/en-US/docs/Tools/Page_Inspector/How_to/Examine_and_edit_HTML