Steps to reproduce on vanilla site

  • define a new content type TEST
  • add entity reference field, autocomplete on bundle basic pages, # of values unlimited (this created the table)
  • create several basic pages with similar name i.e test1, test2, test3 ...
  • create new content of type TEST

Inspect the table with the entity reference field, notice the classes

field-multiple-table 
table 
table-hover 
table-striped 
table 
table-hover 
table-striped 
sticky-enabled 
tabledrag-processed 
tableheader-processed 
sticky-table

Comments

mxlav created an issue. See original summary.

sylus’s picture

Status: Active » Fixed

This was a known problem in bootstrap before the 3.4 release. This was fixed as of WxT 4.2.

mxlav’s picture

Status: Fixed » Active

just tested this on a simpletest.me site of wetkit 4.2, still has double set of classes

mxlav’s picture

Version: 7.x-4.0 » 7.x-4.2
sylus’s picture

Okay usually when I see double classes it was related to this issue in bootstrap but that regression was fixed and applied to pretty much every class on the site.

This looks like it is only affecting a subset of classes as not ever class is duplicated. It looks to me like both bootstrap + wetkit_bootstrap are adding the same classes though the sub-theme should be overriding.

I'd bet if you went to wetkit_bootstrap_preprocess_table() and commented out the following code you would only have a single set of classes.

  // Add the necessary classes to the table.
  //_wetkit_bootstrap_table_add_classes($variables['attributes']['class'], $variables);

This would then let only bootstrap's table.func.php be called.

sylus’s picture

Status: Active » Needs work
mxlav’s picture

commenting out that line did indeed remove the double classes.

sylus’s picture

Status: Needs work » Fixed

Okay I am going to remove that for now until we have the actual need to override bootstrap's default settings which are already aligned with:

http://wet-boew.github.io/wet-boew-styleguide/v4/design/tables-en.html

Thanks a bunch! Appreciated!

  • sylus committed 552e8bc on 7.x-4.x
    Fixed WetKit Bootstrap for Issue #2666192: double set of table classes
    

Status: Fixed » Closed (fixed)

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