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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chapabu’s picture

chapabu CreditAttribution: chapabu commented
Version: 7.x-1.x-dev » 7.x-1.2
FileSize
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 CreditAttribution: 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 CreditAttribution: 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
CreditAttribution: frob 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 CreditAttribution: 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.