Moodle in English: Warning messages with Topcoll and latest Moodle 2.4.1+

General plugins

Warning messages with Topcoll and latest Moodle 2.4.1+

This discussion has been locked because a year has elapsed since the last post. Please start a new discussion topic.

I was just getting ready to upgrade my production Moodle site. After upgrading my experimental site, I am getting the following warning messages (see graphic below). This seems to be related to the Topcoll plugin that I am using (Gareth). If I go to a course that is not using Topcoll, I do not get this message. Of course, I can not show this message by turning debug messages off, but this is why I leave it on.

Also, this is the first time that I am getting these error messages, they did not occur in Moodle 2.4+, only in Moodle 2.4.1+.

I might post this as a "bug" in tracker too, just in case it really has something to do with Moodle's code, and not Topcoll.

Warning Messages.

Average of ratings: -

Permalink | Reply

Dear Rick,

Seems to be happening with the Grid format too - https://moodle.org/mod/forum/discuss.php?d=218505#p958565 - a possible core change.

Cheers,

Gareth

Average of ratings: -

Permalink | Show parent | Reply

Dear Rick,

It looks like MDL-36095 made changes to the core '/course/format/renderer.php' (diff file attached) which means that course formats that implement their own custom header (Collapsed Topics and Grid) will have to be updated - i.e. me - oh joy! This is also why I stipulate in the readme.txt the required version of Moodle - even minor ones.

Cheers,

Gareth

renderer_php_diff.diff

Average of ratings: -

Permalink | Show parent | Reply

Well Gareth, yep, a little more work cut out for you. Maybe it was appropriate for me to post this in the Moodle Tracker because maybe someone will find a work around. Or they might simply tell you to fix your plugin.

I could not detect any harm in Moodle's operation (yet), so maybe I can just turn off debug messages (although I hate to do this because sometimes students let me know about these messages.)

Average of ratings: -

Permalink | Show parent | Reply

Dear Rick,

No problem. The fix is trivial, but I suspect would have lead to functionality loss. My mind is now fuzzy having done the fix in three course formats each with two branches.

There might be a developer's brefing / news list somewhere, perhaps I just have not found it. The snag is, half the time I only realise that something has happened when people report issues.

Cheers,

Gareth

Average of ratings: -

Permalink | Show parent | Reply

Gareth, I just loaded and tried your newest CT. It did not produce the Warnings messages, so it appears that you fixed these problems. However, I do notice that the Section headings are larger, maybe too large and crowding the arrows. Wow, I thought that I would like larger headings too, but now I see that I prefer the old. I think that I see that the new headings pick up the "h3" heading style. I wonder if there is a way to give the user more control over this, like another setting just as you let the user left or center the heading. Here's the differences:

Average of ratings: -

Permalink | Show parent | Reply

I got a note from Gareth saying "

Average of ratings: -

Permalink | Show parent | Reply

I got a note from Gareth say "That particular piece of css despite using font size terminology sets the margin of the text within the toggle. This selector is specific to Collapsed Topics, however the html when created shares css selectors with the theme to facilitate the font size setting in the theme to be applied as well. Therefore to change the size adjust the css in your theme. I hope this make sense"

Well, I am not sure exactly what I need to do. I know where to put in custom css into my formal white theme, but I don't know the form of the code. It might be something like ul.ctopics.topics.... Maybe Mary, Gareth, or someone else can help.

Average of ratings: -

Permalink | Show parent | Reply

Dear Rick,

Please give me time and I'll FireBug the correct generic css selector to apply to the h3 tag or specific selector for CT toggles only. The text should not be bumping into the arrows and I did put some padding in my styles.css to sort this.

Cheers,

Gareth

Average of ratings: -

Permalink | Show parent | Reply

Dear Rick,

To target Collapsed Topics only toggle headings, please use the '.course-content ul.ctopics li.section .content .toggle a h3.sectionname' css selector, e.g:

.course-content ul.ctopics li.section .content .toggle a h3.sectionname {

font-size: 18pt;

}

To target all 'h3' section name headings use the 'h3.sectionname' css selector, e.g:

h3.sectionname {

font-size:16pt;

}

Note that there is no space between the 'h3' and '.sectionname' to make the class 'sectionname' be attributed to the 'h3' tag that has it as an attribute rather than with a space an element that is a child of the 'h3' tag.

Cheers,

Gareth

Average of ratings: -

Permalink | Show parent | Reply

Gareth, this is exactly what I was looking for. Once I knew the general form of this CSS, I was able to modify it even more to my own liking. Wow, I think that we are there! I have had success on my experimental site, so maybe this weekend or so, I will upgrade my production site with the newest Moodle, Collapsed Topics, and Checklist (just began using this a couple days ago).

Here is the CSS that I added to my Formal White:

.course-content ul.ctopics li.section .content .toggle a h3.sectionname {
font-size: 10pt;
padding-left: 8px;
line-height: 120%;
}

Here is what it produces (ignore the red/green experimentation).

CT Improvements

Average of ratings: Useful (1)

Permalink | Show parent | Reply