Fixing wide table with too many columns when used with user_mailman_register

anoopjohn - December 30, 2008 - 19:20
Project:Mailman Manager
Version:6.x-1.x-dev
Component:User interface
Category:bug report
Priority:normal
Assigned:anoopjohn
Status:closed
Description

When this module is used with user mailman register, the increased number of columns in the table causes the table to grow very wide. Another problem with this is that user_mailman_register introduces the description which is a very long text field which when presented in a narrow column causes the row height to become very large, making the whole table look ugly. This can be fixed by explicitly selecting the required fields - lid, name and command.

#1

anoopjohn - December 30, 2008 - 19:26
Title:Fixing wide table with too many columns when used with user_mailma_register» Fixing wide table with too many columns when used with user_mailman_register
Status:active» needs review

I am attaching a patch for this issue, please review and use

AttachmentSize
mailman_manager_fix_wide_columns_352598_2.patch.gz 699 bytes

#2

anoopjohn - December 30, 2008 - 20:07

The table also lacked the empty header declarations and I missed to add it.
Please add
at line 397

                   array('data' => NULL),
                   array('data' => NULL),

so that the header declaration in function _mailman_manager_admin_display() becomes
  $headers = array(array('data' => t('ID'), 'field' => 'lid', 'sort' => 'asc'),
                   array('data' => t('Name'), 'field' => 'name'),
                   array('data' => t('Request Address'), 'field' => 'command'),
                   array('data' => NULL),
                   array('data' => NULL),
                   );

If the empty headers are not present and when there are more than one row in the table and when you scroll the page floating headers will not get correctly aligned to the columns.

#3

samuelet - January 15, 2009 - 17:20
Version:6.x-2.1» 6.x-1.x-dev
Status:needs review» fixed

In late, but finally commited ;-)
Thanks

#4

System Message - January 29, 2009 - 17:30
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.