Download & Extend
Issues

theme_admin_block_display colspan adjustment

Posted by Jacine on April 14, 2008 at 2:48pm
Project:Drupal core
Version:5.7
Component:block.module
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

Hi,

First patch here, so I hope I did it right.

In the theme_admin_block_display function, the colspan defaults to 7 columns for sites with throttle enabled, and 6 for sites without throttle, when really there are only 6 columns with throttle and 5 without. The attached patch just changes the defaults to 6 and 5.

With Throttle

  1. Block
  2. Region
  3. Weight
  4. Throttle
  5. Operations (configure)
  6. Operations (delete)

Without Throttle

  1. Block
  2. Region
  3. Weight
  4. Operations (configure)
  5. Operations (delete)
AttachmentSizeStatusTest resultOperations
theme_admin_block_colspan.diff1.14 KBIgnored: Check issue status.NoneNone
Login or register to post comments

Comments

#1

Posted by drumm on May 13, 2008 at 4:51am
Status:needs review» fixed

Committed to 5.x.

#2

Posted by Anonymous (not verified) on May 27, 2008 at 4:53am
Status:fixed» closed (fixed)

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