I feel it wouuld be better if in taxonomy_vtn.pages.inc you set the table widths as below

   $output .= theme('table', array(), $row, array('class' => 'taxonomy_vtn_nodes', 'id' => 'taxonomy_vtn_nodes','width'=>'100%'));

   $output .= theme('table', array(), $row, array('class' => 'taxonomy_vtn_vocabularies', 'id' => 'taxonomy_vtn_vocabularies','width'=>'100%'));

        $output .= theme('table', array(), $row, array('class' => 'taxonomy_vtn_terms', 'id' => 'taxonomy_vtn_terms','width'=>'100%'));

this is because you otherwise rely on the theme to set the table width to 100% in CSS
If css sets the table width to 100% then the end user can not enter a table to a width of his choice because the css overrides this

In some themes the the table is not set at all in css

brillant module by the way

Comments

tomaszx’s picture

tomaszx commented
Assigned: Unassigned » tomaszx
Category: bug » feature
Status: Active » Fixed

Hello,

If i set this width in style="width:100%" then this is master to class and you not be able to overwrite this width never.

i tested with marvin and pushbutton themes and this work correctly. My decision is:

in taxonomy_vtn.css i add:

table.taxonomy_vtn_vocabularies , table.taxonomy_vtn_terms, table.taxonomy_vtn_nodes {
  width: 100%;
}

and you can overwrite this in exmaple: "themes/chameleon/marvin/style.css" using:

table.taxonomy_vtn_vocabularies , table.taxonomy_vtn_terms, table.taxonomy_vtn_nodes {
  width: 70%;
}

I think that this is probably the best decision for your issue.

Listen: Your issue is not a bug report (maybe feature request)

By the way, set width to table in css not should be to all tables but for example: "table.class_name" - only for table with class name "class_name".

sykic’s picture

sykic commented

Your solution is better than mine

I have tested it and it works well thank you

Will you put it in the next release ?

Did not mean to rasie this as a bug sorry !

tomaszx’s picture

tomaszx commented

yes i put this in next release.

Thank you for activity :)

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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