FireRainbow breaks Firebugs 'Add Watch'
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 20/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Stale
- Tech stack
- javascript
- Domain
- devtools
Research direction
Start with the supplied HTML reproduction and the Firebug Script Panel's Add Watch entry point. Reproduce the selection being changed to "spa" in the Watch Panel, then trace how FireRainbow's highlighting affects the selected text. Done means the highlighted expression is consistently added to the Watch Panel without a syntax error.
Written by the indexing model from the issue text.
Description
Hi Antonin I have had a problem in Firebug for a while now that if I highlight text
in the Script Panel, right-click and 'Add Watch' the wrong text is added
... I have just discovered that this issue is caused by
FireRainbow. Here is a small test case so that you can see what is happening: ================= <html> <head> <script> function doit() { var s = 'Just some dummy Text'; debugger; } </script> </head> <body onload="doit();"> 1. Activate Firebug and refresh the page to trigger debugging 2. When debugging starts, highlight the 's' in s += 'Just some dummy
Text'; 3. Right-click the highlighted 's' and choose 'Add Watch' 4. If the correct text was added to the Watch Tab then right-click
the text and add watch again I just get a '<' symbol added to the watch panel and a syntax
error ... strangely, it sometimes works but very rarely. </body> </html> ================= What steps will reproduce the problem? 1. Install FireRainbow and restart Firefox 2. Load the test file 3. Activate Firebug and refresh the page to trigger debugging 4. When debugging starts, highlight the 's' in s += 'Just some dummy
Text'; 5. Right-click the highlighted 's' and choose 'Add Watch' 6. If the correct text was added to the Watch Tab then right-click the
text and add watch again What is the expected output? I expect to see the highlighted text added to the Watch Panel What do you see instead? I just get "spa" added to the watch panel and syntax error. Additional info: This only happens when Firebug is installed ================= Thanks for your hard work. Mike Ratcliffe
- Dominant language
- JavaScript
- Stars
- 58
- Forks
- 12
- PR merge metrics
- No merged PRs in 30d
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from binaryage/firerainbow
-
Difficulty 3/5 1-2 days Newbie friendliness 35/100
binaryage/firerainbow#16 · 1 comment ·
-
Difficulty 4/5 3-5 days Newbie friendliness 15/100
binaryage/firerainbow#14 · 1 comment ·
-
Difficulty 4/5 3-5 days Newbie friendliness 25/100
binaryage/firerainbow#13 · 4 comments ·
-
Difficulty 4/5 3-5 days Newbie friendliness 20/100
binaryage/firerainbow#12 ·
-
Difficulty 4/5 3-5 days Newbie friendliness 20/100
binaryage/firerainbow#11 ·
All issues in binaryage/firerainbow
Similar issues
-
area/install-update comp/cli comp/desktop P3 sweeper:risk-compatibility type/bug
Difficulty 2/5 1-3 hours Newbie friendliness 86/100
NousResearch/hermes-agent#122386 · 1 comment ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
-
security
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
IBM/node-sdk-core#373 ·
-
docs web/
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 65/100