Moodle

admin/mnet/index.php differnt hardcoded table width

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.8.3
  • Fix Version/s: 1.9.8
  • Component/s: MNet
  • Labels:
    None
  • Affected Branches:
    MOODLE_18_STABLE
  • Fixed Branches:
    MOODLE_19_STABLE

Description

On admin/mnet/index.php two different table width are set:

_ top one no width
_ bottom one 635.

By the way, the mnet admin pages lack "normal" CSS hooks and look strange in advanced themes. Please check and optimize the mnet admin pages.

Thank you very much.

  1. index.patch
    11/Dec/09 12:02 PM
    0.9 kB
    Andrew Davis
  2. styles-layout.patch
    11/Dec/09 12:02 PM
    0.5 kB
    Andrew Davis

Activity

Hide
Urs Hunkler added a comment -

Screenshot showing the different block width :: http://screencast.com/t/yYHPVExwW1T

Show
Urs Hunkler added a comment - Screenshot showing the different block width :: http://screencast.com/t/yYHPVExwW1T
Hide
Andrew Davis added a comment -

The tables that were different widths are actually within those with hard coded widths. The problem was that the two subtables had no widths at all specified on them meaning they defaulted to however wide they needed to be. The top table has skinny content in it so it doesn't take up all of the available space.

I added an entry to /these/standard/styles_layout.css that makes these subtables take up 100% of the space available to them within the outer table.

.generalboxcontent table{
width:100%;
}

Show
Andrew Davis added a comment - The tables that were different widths are actually within those with hard coded widths. The problem was that the two subtables had no widths at all specified on them meaning they defaulted to however wide they needed to be. The top table has skinny content in it so it doesn't take up all of the available space. I added an entry to /these/standard/styles_layout.css that makes these subtables take up 100% of the space available to them within the outer table. .generalboxcontent table{ width:100%; }
Hide
Dakota Duff added a comment -

This style affects table sizes on a number of other pages. Can the style please be modified to prevent this?

Show
Dakota Duff added a comment - This style affects table sizes on a number of other pages. Can the style please be modified to prevent this?
Hide
Andrew Davis added a comment -

It can if necessary. Is the change causing a problem? Can you attach a screenshot?

Show
Andrew Davis added a comment - It can if necessary. Is the change causing a problem? Can you attach a screenshot?
Hide
Mauno Korpelainen added a comment -

I can conform at least one new bug caused by this bug fix:

it breaks (makes too wide) editor toolbar width on course pages.

http://moodle.org/mod/forum/discuss.php?d=139526

Show
Mauno Korpelainen added a comment - I can conform at least one new bug caused by this bug fix: it breaks (makes too wide) editor toolbar width on course pages. http://moodle.org/mod/forum/discuss.php?d=139526
Hide
Mauno Korpelainen added a comment -

Andrew,

you should have more specific css than

.generalboxcontent table

Otherwise any tables inside generalbox content get 100% width if there is no other css that defines table width.

Show
Mauno Korpelainen added a comment - Andrew, you should have more specific css than .generalboxcontent table Otherwise any tables inside generalbox content get 100% width if there is no other css that defines table width.
Hide
Andrew Davis added a comment -

reopening this issue.

Show
Andrew Davis added a comment - reopening this issue.
Hide
Andrew Davis added a comment -

Patches attached to limit it to the mnet admin page. Further comments welcome

Show
Andrew Davis added a comment - Patches attached to limit it to the mnet admin page. Further comments welcome
Hide
Martin Dougiamas added a comment -

Use a #admin-mnet-index selector Andrew

Show
Martin Dougiamas added a comment - Use a #admin-mnet-index selector Andrew
Hide
Andrew Davis added a comment -

Committed using the selector as per Martin's suggestion

Show
Andrew Davis added a comment - Committed using the selector as per Martin's suggestion

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: