I'm sure it was like this in a previous release...but it'd be great if we weren't restricted to the tiny box for Executing PHP snippets! I often have Firebug pulled quite high or in a seperate window, and although I can pull the textfield so it's larger...I can't actually see it (currently using DFF 1.4 on D6) :(

Awesomely useful piece of kit though!

CommentFileSizeAuthor
#1 drupalforfirebug-fixphpwindow-1105058.patch662 byteschapabu

Comments

chapabu’s picture

chapabu commented
Version: 7.x-1.x-dev » 7.x-1.2
StatusFileSize
new drupalforfirebug-fixphpwindow-1105058.patch662 bytes

I've attached a patch to fix this against 7.x-1.x.

The same issue as #1289330: Invalid CSS syntax in drualforfirebug_get_php_exec_area() but for 7.x rather than 6.x - would be good to get commited to both :-)

chapabu’s picture

chapabu commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
Component: Firefox Extension » Code
Category: feature » bug
Status: Active » Needs review

Whoops - wrong version...sorry

chapabu’s picture

chapabu commented
Version: 7.x-1.2 » 7.x-1.x-dev
Status: Needs review » Reviewed & tested by the community

I've used this on patch a few sites now, and it works fine. Marking as RTbC - would be good to get this patch into dev if it's not already.

Can't comment on the D6 version in #1289330: Invalid CSS syntax in drualforfirebug_get_php_exec_area() but it's the same, so I'd assume it's fine.

frob’s picture

frob
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Patch applied.

In the future, you shouldn't mark your own patches as RTBC.

chapabu’s picture

chapabu commented

D'oh...Apologies!

I've since read the guidelines...I meant to come back and change this but I completely forgot!

Thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.