Steffen Wilberg Assignee
	Description • 22 years ago

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.4b) Gecko/20030526 Mozilla Firebird/0.6
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.4b) Gecko/20030526 Mozilla Firebird/0.6

URLs in the js console are not formatted like a link.

Reproducible: Always

Steps to Reproduce:
1. Open Javascript console.
2. Look for a link in it. If you can't find one, browse to
http://java.sun.com/getjava/

Actual Results:  
The url in the js console is not formatted like a link, e.g. blue and underlined.


Expected Results:  
The url should be formatted like a link so that you know where to click on (not
on Source File, not on Line #).

Mozilla does this right.

	Erik
	Comment 1 • 22 years ago

Confirmed using Mozilla/5.0 (Windows; U; WinNT4.0; en-US; rv:1.4b)
Gecko/20030529 Mozilla Firebird/0.6

=> change status to "New" since I didn't find a related, already existing bug

Note: I don't get a JS error when visiting java.cun.com - perhaps one has to use
a different page...

	Karl Guertin
	Comment 2 • 21 years ago

Problem still exists in Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.5b)
Gecko/20030726 Mozilla Firebird/0.6.1

Use http://www.javascriptkit.com/javatutors/errortest.htm for testing, which
should remain permanently erroneous.

	Scot Hale
	Comment 3 • 21 years ago

I am experencing this bug with FireFox 0.8 

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.6) Gecko/20040206 Firefox/0.8

	Chris Neale
	Comment 4 • 21 years ago

chrome://global/content/console.xul needs to reference
chrome://commmunicator/skin/formatting.css - then all will be well : )

	Steffen Wilberg Assignee
	Comment 5 • 21 years ago

Thanks for the tip. But we're not using
chrome://commmunicator/skin/formatting.css anywhere in Firefox, so I'd suggest
to add the rules we need to console/skin/console.css.

	Steffen Wilberg Assignee
	Comment 6 • 21 years ago

I've left out some rules that don't work:
.text-link:focus { color:red }
.text-link:hover:active { color: red }
.text-link[visited="true"] { color: purple }

	Pierre Chanial
	Comment 7 • 21 years ago

marking fixed.
I just saw that there was a patch in here. Some time wasted.
I put the rules in the toolkit formatting.css file. class text-link can be useful. 

	Steffen Wilberg Assignee
	Comment 8 • 21 years ago

Comment on attachment 144317 [details] [diff] [review]
patch: add rules to console/skin/console.css

Fixed by pch.

	José Jeria
	Comment 9 • 21 years ago

Did this patch cause the link in the page info blue and the context menu to it
underlined?

	Steffen Wilberg Assignee
	Comment 10 • 21 years ago

(In reply to comment #9)
> Did this patch cause the link in the page info blue and the context menu to it
> underlined?
Yes and no. It wasn't my patch, but pch's checkin for this bug:
http://bonsai.mozilla.org/cvsquery.cgi?date=explicit&mindate=03%2F20%2F2004+00%3A01&maxdate=03%2F20%2F2004+00%3A01
See bug 238186.

	Steffen Wilberg Assignee
	Comment 11 • 21 years ago

Broken again with the new Winstripe theme. -> Reopening, at least for Windows.

	Steffen Wilberg Assignee
	Comment 12 • 21 years ago

pch said in comment 7:
"I put the rules in the toolkit formatting.css file. class text-link can be
useful."
This patch puts them back there. It has no effect to page info (bug 238186).

Another option is to put these in toolkit/themes/winstripe/global/console.css,
but I agree with pch that these rules can be useful.

	Steffen Wilberg Assignee
	Comment 13 • 21 years ago

Comment on attachment 150378 [details] [diff] [review]
fix it again

Mike, this is a regression fix for 0.9.

	Steffen Wilberg Assignee
	Comment 14 • 21 years ago

patch ready.

	Mike Connor [:mconnor]
	Comment 15 • 21 years ago

Comment on attachment 150378 [details] [diff] [review]
fix it again

blech, this old chestnut? :)

	Steffen Wilberg Assignee
	Comment 16 • 21 years ago

-> js console.

	Steffen Wilberg Assignee
	Comment 17 • 21 years ago

Missed 0.9.

	Blake Ross
	Comment 18 • 21 years ago

Not blocking 1.0, but feel free to check this in.

	Mike Connor [:mconnor]
	Comment 19 • 21 years ago

fixed branch and trunk

	Steffen Wilberg Assignee
	Comment 20 • 21 years ago

Note:
mozilla/toolkit/themes/pinstripe/global/formatting.css and
mozilla/toolkit/themes/qute/global/formatting.css already contain these css
rules on branch and trunk.