Closed (fixed)
Project:
Bootstrap
Version:
7.x-3.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
26 Dec 2013 at 03:37 UTC
Updated:
30 Jul 2014 at 05:58 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
bdanin commentedI am also having this problem. Additionally, the vertical tabs in Display suite aren't showing up properly. When I convert my admin theme to seven, the problems go away. Also, I did not have this problem with Bootstrap-2.x so it's specific to this release.
Comment #2
bdanin commentedI haven't been able to figure out what the problem is, but using display suite, I cannot change any of the field orders, because ajax or jquery stall. Ultimately, I cannot move fields from hidden to other zones unless I turn off javascript and manually make those changes without .js
Anyway, here is a hacked-together solution that allows me to keep using Bootstrap 3.x, and then fail-back to seven for when I need to work with display suite (I don't seem to have problems elsewhere, so I'm not too concerned, but this does need to be fixed):
1) enable theme seven
2) install and enable page_theme (https://drupal.org/project/page_theme)
3) add seven, and tell it to display on this path: admin/structure/types/manage/*/display*
One note, I'm using bootstrap-sass (see: https://drupal.org/node/2116345), so I haven't yet had a chance to test this using LESS to see if that's part of the problem or not.
Comment #3
mkhamash commentedThe problem is that bootstrap_button theme function is turning all input submit to button, in the case of draggable table there is an input id=#edit-refresh that fire the ajax update.
for more information on look at AJAXRefreshRows in Drupal field_ui.js file.
Comment #4
mkhamash commentedSorry, there was an extra whitespace in the patch "1 line adds whitespace errors", re-uploaded.
Comment #5
markhalliwellActually the issues lies with the JavaScript. This project should not make exceptions for the "button" element. Normally the offending module should be patched (see related issue about Panels), but this is for core and considering it's fixed for 8.x (can't seem to find that issue ATM, but it is), I doubt it will get backported to 7.x.
Committed 4fede42 to 7.x-3.x:
Comment #6
bdanin commentedpatch in #4 seems to fix this issue for me
Comment #7
markhalliwellThe patch in #4 will not be applied. I have already committed the fix in #5.
Comment #8
bdanin commentedwhen I tried to pull down latest dev branch using drush, it erred:
File bootstrap-7.x-3.x-dev.tar.gz_date=1392651805 is corrupt (wrong md5 checksum). [error]Also, note that the patch in #4 causes problems in places, specifically in views, the "apply" button disappears
Comment #9
markhalliwellIf you click on the commit hash link in #5, it will take you to drupalcode.org where you can then download the patch against the repo
Comment #12
ryan.armstrong commentedMoving this to the 8.x-3.x branch as it has been committed to that branch. Making sure the fix gets tested in the Drupal 8 release.
Comment #14
markhalliwellI'm just moving this back to 7.x. If this needs re-evaluation in 8.x, create a new issue.