Moodle

Anomaly theme inconsistency

Details

  • Type: Sub-task Sub-task
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 2.0.1
  • Fix Version/s: 2.2
  • Component/s: Themes
  • Environment:
    Firefox 3.6.13 for Ubuntu 10.10.
    Chrome 8.0.552.237 for Ubuntu 10.10 (but without rounded corners, which I thought was odd).
  • Testing Instructions:
    Hide

    1. Select Anomaly theme and test-drive in modern browsers only.
    2. Test that the left-hand column, in any forum post, is a solid light Gray colour.
    3. Test that the top-right corner of the page header is not broken in IE8, IE9 and Opera
    4. Test docking blocks.
    5. With editing enabled test to see if "Add Block" menu bar sits neatly within the block.

    Show
    1. Select Anomaly theme and test-drive in modern browsers only. 2. Test that the left-hand column, in any forum post, is a solid light Gray colour. 3. Test that the top-right corner of the page header is not broken in IE8, IE9 and Opera 4. Test docking blocks. 5. With editing enabled test to see if "Add Block" menu bar sits neatly within the block.
  • URL:
  • Affected Branches:
    MOODLE_20_STABLE
  • Fixed Branches:
    MOODLE_22_STABLE
  • Pull from Repository:
  • Pull Master Branch:
  • Pull Master Diff URL:

Description

The default Anomaly theme as installed with 2.0.1 has some visual inconsistencies when used with forum posts. Please see attached images:

anomaly.png: a screenshot-snippet of a forum post with the default Anomaly theme and Moodle 2.0.1.
anomaly-suggested.png: Due to low contrast of colours in first image, I have highlighted the 'block' of colour and squiggled where the colour is missing.
anomaly-mockup.png: What I believe it should look like (mocked up in GIMP, not patched).

  1. anomaly_mockup.png
    15 kB
    02/Feb/11 11:16 PM
  2. anomaly_suggested.png
    15 kB
    02/Feb/11 11:16 PM
  3. anomaly.png
    14 kB
    02/Feb/11 11:16 PM
  4. MDL-26240-Anomaly.jpg
    46 kB
    27/Nov/11 9:29 AM

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Helen Foster added a comment -

Setting a fix version of STABLE backlog as part of our triaging process.

Show
Helen Foster added a comment - Setting a fix version of STABLE backlog as part of our triaging process.
Hide
Permalink
Mary Evans added a comment -

I've set up MDL-29561 with an aim: To FIX all outstanding issues associated with Anomaly.
Related Issues:
MDL-25961 - Anomaly theme should display the course summary below the header.
MDL-26240 - Anomaly theme inconsistency.
MDL-26694 - Anomaly theme tabs are not well drawn.

Show
Mary Evans added a comment - I've set up MDL-29561 with an aim: To FIX all outstanding issues associated with Anomaly. Related Issues: MDL-25961 - Anomaly theme should display the course summary below the header. MDL-26240 - Anomaly theme inconsistency. MDL-26694 - Anomaly theme tabs are not well drawn.
Hide
Permalink
Sam Hemelryk added a comment -

Hi Mary,

I've sending this back presently.
By adding overflow:hidden on #page you break the theme's ability to overflow horizontally which is important for the report layouts.
It pays to be very careful with using overflow:hidden on the main content area or any of its parents as you run into these sort of troubles.

Cheers
Sam

Show
Sam Hemelryk added a comment - Hi Mary, I've sending this back presently. By adding overflow:hidden on #page you break the theme's ability to overflow horizontally which is important for the report layouts. It pays to be very careful with using overflow:hidden on the main content area or any of its parents as you run into these sort of troubles. Cheers Sam
Hide
Permalink
Mary Evans added a comment - - edited

Hi Sam,
Just read your message!
Am I too late to reverse this?
I've just committed the change in general.css in MDL-26240 just this minute!
Please say it can go thru with the change?

I have so much more to add to this theme but wanted this cleaning up first!!!

Cheers
Mary

Show
Mary Evans added a comment - - edited Hi Sam, Just read your message! Am I too late to reverse this? I've just committed the change in general.css in MDL-26240 just this minute! Please say it can go thru with the change? I have so much more to add to this theme but wanted this cleaning up first!!! Cheers Mary
Hide
Permalink
Mary Evans added a comment -

In deleting old merged branches I deleted this in error!
Let's hope I have the original files on my computer!

Show
Mary Evans added a comment - In deleting old merged branches I deleted this in error! Let's hope I have the original files on my computer!
Hide
Permalink
Mary Evans added a comment -

@Sam

I spent 3 hours last night trying to resolve conflicts while re-basing and ended up with a diverging origin/master...which is now in my recycle bin!

I'm now re-building my Moodle-GIT and working on MDL-26240 ready for submitting today.

Cheers
Mary

Show
Mary Evans added a comment - @Sam I spent 3 hours last night trying to resolve conflicts while re-basing and ended up with a diverging origin/master...which is now in my recycle bin! I'm now re-building my Moodle-GIT and working on MDL-26240 ready for submitting today. Cheers Mary
Hide
Permalink
Mary Evans added a comment -

As well as restyling the forumpost left column I also restyled the docking tabs to fit in with the theme colors

Show
Mary Evans added a comment - As well as restyling the forumpost left column I also restyled the docking tabs to fit in with the theme colors
Hide
Permalink
Sam Hemelryk added a comment -

Hi Mary, I've just been clearing my Email's.
This issue is on my list for today

Show
Sam Hemelryk added a comment - Hi Mary, I've just been clearing my Email's. This issue is on my list for today
Hide
Permalink
Sam Hemelryk added a comment -

Hi Mary,
I've just had a look at this again, its VERY close now.
Unfortunately there is a nasty bug with the dock.
A quick glance suggests you should remove the following:

body.has_dock #page {
    width: 97%;
    right: 3%;
}

and instead add:

body.has_dock {
    margin:0;
}

I thought about integrating that as a solution but I think it is best if you have a look and think about it, I may have misinterpreted what you were aiming for.
Either way seems to be easily to reproduce the bug just by docking a block using Chrome.

Cheers
Sam

Show
Sam Hemelryk added a comment - Hi Mary, I've just had a look at this again, its VERY close now. Unfortunately there is a nasty bug with the dock. A quick glance suggests you should remove the following: body.has_dock #page { width: 97%; right: 3%; } and instead add: body.has_dock { margin:0; } I thought about integrating that as a solution but I think it is best if you have a look and think about it, I may have misinterpreted what you were aiming for. Either way seems to be easily to reproduce the bug just by docking a block using Chrome. Cheers Sam
Hide
Permalink
Mary Evans added a comment -

Sam,

Can you fix this when integrating? If so do it. I did not notice it in resolution 1024 x 768 as I had my screen. Now swapping to Chrome and 1400 x 1050 I can certainly see the BUG...thanks for that.

Mary

Show
Mary Evans added a comment - Sam, Can you fix this when integrating? If so do it. I did not notice it in resolution 1024 x 768 as I had my screen. Now swapping to Chrome and 1400 x 1050 I can certainly see the BUG...thanks for that. Mary
Hide
Permalink
Sam Hemelryk added a comment -

Thanks Mary - I made those changes in a commit afterward the integration so all fixed up and in

Show
Sam Hemelryk added a comment - Thanks Mary - I made those changes in a commit afterward the integration so all fixed up and in
Hide
Permalink
Rossiani Wijaya added a comment -

This looks good.

Test passed.

Show
Rossiani Wijaya added a comment - This looks good. Test passed.
Hide
Permalink
Eloy Lafuente (stronk7) added a comment -

The master fixes corresponding to this issue have been sent upstream. Fixes for other branches (19, 20, 21 stable) will be sent in the very-next days.

Thanks for the hard work! Closing, ciao

Show
Eloy Lafuente (stronk7) added a comment - The master fixes corresponding to this issue have been sent upstream. Fixes for other branches (19, 20, 21 stable) will be sent in the very-next days. Thanks for the hard work! Closing, ciao

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: