Closed
Bug 232985
Opened 22 years ago
Closed 20 years ago
Script highlighting in source view doesn't understand regular expressions
Categories
(Other Applications Graveyard :: Venkman JS Debugger, enhancement)
Other Applications Graveyard
Venkman JS Debugger
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: timeless, Assigned: bugzilla-mozilla-20000923)
References
(
URL
)
Details
Attachments
(1 file)
|
3.87 KB,
patch
|
rginda
:
review+
|
Details | Diff | Splinter Review |
bug 127751 seems to be the bug of record for this
build: cvs from 02/02/04 midafternoon.
steps:
load:
data:text/html,<html><script>function quote1(s) {return s.replace(/'/g,"\\'")}
function quote2(s) {return s.replace(/"/g,"\\'")}</script>
load venkman
select loaded scripts
select "script>"
double click to view source
the single line will look like this:
plain: <html><script>
bold: function
plain: quote1(s) {
bold: return
plain: s.replace(/'
green: /g,"\\
plain: '"
green: )} function quote2(s) {return s.replace(/
plain: "/g,"
green: \\'
plain: ")}</script>
expected results:
plain: <html><script>
bold: function
plain: quote1(s) {
bold: return
plain: s.replace(/
something: '
plain: /g,"
green: \\'
plain: ")}
bold: function
plain: quote2(s) {
bold: return
plain: s.replace(/
something: "
plain: /g,"
green: \\'
plain: ")}</script>
|
||
Updated•21 years ago
|
Product: Core → Other Applications
|
Assignee | |
Comment 1•20 years ago
|
||
Where's the "and accept bug" button, BugZilla! :P
Assignee: rginda → silver
Status: UNCONFIRMED → NEW
Ever confirmed: true
|
|
Assignee | |
Updated•20 years ago
|
Status: NEW → ASSIGNED
OS: Windows XP → All
Hardware: PC → All
Version: Other Branch → Trunk
|
|
Assignee | |
Comment 2•20 years ago
|
||
This marks all the way from the opening / to the end / in <r>, and colours it purple. It ignores the letter modifiers, as they don't otherwise hurt the parser.
Attachment #223225 -
Flags: review?(rginda)
|
||
Comment 3•20 years ago
|
||
Comment on attachment 223225 [details] [diff] [review]
Add REGEXP type to source formatter
r=rginda
Attachment #223225 -
Flags: review?(rginda) → review+
|
|
Assignee | |
Comment 4•20 years ago
|
||
Checked in --> FIXED.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
|
||
Updated•7 years ago
|
Product: Other Applications → Other Applications Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•