Here is a quick patch to the js to support input type="file".

CommentFileSizeAuthor
#6 garland_FF_before.PNG18.3 KBelvis2
#6 garland_FF_after.PNG16.75 KBelvis2
#6 garland_Safari-Chrome_before.PNG11.62 KBelvis2
#6 garland_Safari-Chrome_after.PNG10.61 KBelvis2
#3 compact_forms_file_safari_after.PNG4.16 KBelvis2
#2 compact_forms_file_safari_before.PNG4 KBelvis2
#2 compact_forms_file_safari_after.PNG17.14 KBelvis2
#2 compact_forms_patch2.patch1.26 KBelvis2
#2 compact_forms_after.PNG3.62 KBelvis2
#2 compact_forms_before.PNG3.9 KBelvis2
compact_forms_patch.patch821 byteselvis2
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Title: Patch for Form Files or type="file" » Support input fields of type 'file'
Category: support » feature
Status: Needs review » Needs work

The patch is reversed. But also, can you post a screenshot of how this looks, please?

elvis2’s picture

elvis2 CreditAttribution: elvis2 commented
Status: Needs work » Needs review
FileSize
compact_forms_before.PNG3.9 KB
compact_forms_after.PNG3.62 KB
compact_forms_patch2.patch1.26 KB
compact_forms_file_safari_after.PNG17.14 KB
compact_forms_file_safari_before.PNG4 KB

Thanks for pointing out the reversed patch. Attached are 4 screenshots. Two are only related to the file filed itself before and after on FF. The other two are to show the safari/chrome issues (before and after the fix).

elvis2’s picture

elvis2 CreditAttribution: elvis2 commented
FileSize
compact_forms_file_safari_after.PNG4.16 KB

Sorry, compact_forms_file_safari_after.PNG was messed up. Here is the better one.

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Status: Needs review » Needs work

Thanks for re-rolling an providing those screenshots.

Regarding the screenshots, I'm not sure what I'm looking at. I've highly prefer to see a screenshot from one of Drupal core's stock/default themes; e.g., Garland. With this patch, I guess you should be able to compact the node form (with Upload module enabled) and have the file attachments field compacted.

+++ compact_forms.js	2010-09-09 20:05:51.000000000 -0400
@@ -16,21 +16,26 @@
-
+      ¶

(and elsewhere in this patch) Trailing white-space introduced.

+++ compact_forms.js	2010-09-09 20:05:51.000000000 -0400
@@ -16,21 +16,26 @@
+      // If Safari or Chrome and are dealing with file field, we need to hide the label
+      if ($field.is('input:file') && $.browser.safari) {
+         $label.css('display', 'none');
+      }

1) "dealing" does not explain why we are implementing this special, browser-specific code and behavior.

2) The comment talks about Chrome and Safari, but the code is conditionally executed for Safari only.

Powered by Dreditor.

elvis2’s picture

elvis2 CreditAttribution: elvis2 commented

Thanks for the feedback. First you should know this is a webform file field the patch is dealing with. The screenshots are only referring the file field.

compact_forms_before.PNG AND compact_forms_after.PNG show the "file upload" label is not in the textfield form element.
http://drupal.org/files/issues/compact_forms_before.PNG
http://drupal.org/files/issues/compact_forms_after.PNG

On Safari and Chrome, the filefield is overridden by the browser, which is fine except the label field shows (on top) of the "Choose File" button the browser renders. See
http://drupal.org/files/issues/compact_forms_file_safari_before.PNG AND
http://drupal.org/files/issues/compact_forms_file_safari_after_0.PNG

Regarding the code for Safari and not Chrome condition, on my PC JS does not recognize $.browser.chrome when using chrome. But, it does recognize $.browser.safari when using both chrome and safari. Maybe it is different on a Mac.

elvis2’s picture

elvis2 CreditAttribution: elvis2 commented
Status: Needs work » Needs review
FileSize
garland_Safari-Chrome_after.PNG10.61 KB
garland_Safari-Chrome_before.PNG11.62 KB
garland_FF_after.PNG16.75 KB
garland_FF_before.PNG18.3 KB

Here are the screenshots with the garland theme...

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Status: Needs review » Needs work

Sorry, but http://drupal.org/files/issues/garland_Safari-Chrome_after.PNG won't fly -- the label is missing entirely, so the user has no idea what to upload.