webform_managed_file buttons do not display correctly

I ran into the same issue you are seeing. In my case, I am using the webform module. In the webform supplied CSS there is a rule:

.webform-container-inline div, .webform-container-inline div.form-item { display: inline }

This is overriding bootstrap's rule on the .input-group wrapper of:

.input-group {display: table; }

As a quick fix, I specified the id 'edit-submitted-' css with a display: table !important; which did the trick.

Here's a patch which renders a Webform file component exactly like a regular managed file element, with the input field and Upload button as an input group.

@joelstein, I don't think this issue is about webform specifically, that should probably be a separate issue (and simply returning the output from bootstrap_file_managed_file, not duplicating the code).

Ah, okay. The code is not exactly the same, because Webform file fields can't have the element ID. But otherwise, it's close. I'll open a new issue and we can tackle it there.

I'll open a new issue and we can tackle it there.

Was this ever done, i.e. is this still an issue?

Mark, no, I'm sorry, I never created a follow-up issue. But fwiw, I've been using this patch for some time now on 100+ websites, and it works sweetly.

Idk why I said what I did in #4, after looking at the code in #3 again, this actually is specific to webform. No new issue needs to be created.

The code is not exactly the same, because Webform file fields can't have the element ID.

It may not be "exactly" the same, but we can add some custom variable before passing it to bootstrap_file_managed_file() so it knows not to add the ID.

Also, this is technically a feature request since it's supporting a 3rd party contrib module that creates custom elements (instead of using core's).

But fwiw, I've been using this patch for some time now on 100+ websites, and it works sweetly.

Maybe so, but I'd like to get this committed and not leave an open issue. If you could find some time to fix the above patch so what I mentioned above, I'll gladly commit it.

Here you go. :)

+++ b/templates/webform/webform-managed-file.func.php
@@ -0,0 +1,27 @@
+  require_once DRUPAL_ROOT . '/' . drupal_get_path('theme', 'bootstrap') . '/templates/file/file-managed-file.func.php';

Please use bootstrap_include() here.

Here you go! :)

Thanks, friend!

Cheers :D