On Oct 17, 3:30 pm, Simon Lindholm <simon.lindhol...@gmail.com> wrote:
> This is just a list of minor bugs/feature requests, not really worthy
> of their own issues:
>
> * editor.js misspells selectionStart as seletionStart in several
> places. This breaks Ctrl-Backspace for deleting a whole word, and
> sometimes the up/down keys for going through the command history.

R8058

> * In the same places, the code tests for if(textBox.selectionStart),
> which fails for selectionStart = 0. This is either a bug or unclear
> logic.

R8059. One case is correct I guess, the backspace.

> * AutoCompleter.complete is called with 3, 4, 5 or 6 arguments from
> different places.

The remaining arguments are null or false in these cases.  Is there a
bug?

> * .fbCommandLineCompletions should probably have white-space:pre set
> (or equivalent fix), so that double spaces show up in the
> autocompletion box.

? what kinds of double spaces?

> * I sometimes copy-paste single lines ending in a newline, but Firebug
> detects the newline and opens the large command line, from where I
> have to remove the single newline and click the "smaller command line"
> button before being able to evaluate the expression. Would it be
> possible to automatically strip these newlines when pasting text? Ugly
> patch: [1]
> * In the console panel, hovering over a Window object belonging to a
> different domain throws an security exception. Testcase: [2]

Please open issues for these, thanks.


>
> Related question: The three first issues could probably be caught with
> some sort of JSLint-ish tool - is this viable?

I don't think so. JSLint is a static analysis tool and no static
analysis would be able to help in these cases. Dynamic analysis tools
are needed. I'll work on them if you take over developing editor.js.

jjb

>
> [1]http://simonsoftware.se/other/commandLine.js.newline.patch
> [2]http://simonsoftware.se/other/fb-cross-domain-tooltip-testcase.html

-- 
You received this message because you are subscribed to the Google Groups 
"Firebug" group.
To post to this group, send email to fire...@googlegroups.com.
To unsubscribe from this group, send email to 
firebug+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/firebug?hl=en.

Reply via email to