-
Notifications
You must be signed in to change notification settings - Fork 120
Class thumbnails broken #858
Comments
Fine in M3.2. |
Is broken, don't know why, leaving for a while. |
Not an issue in Shoelace. |
Thank you Gareth.
From: Gareth J Barnard [mailto:notifications@github.com]
Sent: Saturday, February 03, 2018 12:26 PM
To: gjb2048/moodle-theme_essential
Cc: WINGsRealityEDU; Author
Subject: Re: [gjb2048/moodle-theme_essential] Class thumbnails broken (#858)
Is broken, don't know why, leaving for a while.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <#858 (comment)> , or mute the thread <https://github.com/notifications/unsubscribe-auth/Aib3_2kUxGf8v7gwg5Yx_Ov6V1_fVXKsks5tRJajgaJpZM4R4OMQ> .Image removed by sender.
|
How can I help you determine what is going on? This is affecting 16 of our courses and some in use right now. Moodle debugging shows no errors. |
Ok, could you provide a course MBZ backup (without user data) that demonstrates the issue please? Then I may have a chance of replicating locally and possibly finding a solution. If the problem is a theme one then I'll try and fix it, if it is you then I'll say it is but not explain why. |
What web browser are you using please? |
Ok, cannot replicate what you have as.... the style guide issue is f62e34a which has fixes that pertain only to the style guide: |
Therefore either a local issue with conflicting styles that is out of my control or is the theme that the course backup would demonstrate. Or provide me with the markup you are using to create thumbnails on a course. |
I just finished a test on a new test server with Moodle 3.4.1+ (Build: 20180118) Version 2017111301.01 PHP 7.0 and there is no issue. On our production server that just had the Moodle 3.4.1 (Build: 20180115) PHP 7.0.0 and Essential 3.4.0.1 (Build: 2017102900) upgrade this is the LMS with the issue. I also have additional themes and the issue continues with them. I am happy to give you a copy of a course MBZ backup is to large to attached here. I can send on chapter of a book that has the issue, |
Getting The selected file is not a valid Moodle backup file and can't be restored. |
Ok, fine now. Why did you put an mbz which is a zip file inside a zip file? |
Been out and just got back in. Yha I get the same result on my test server but still broken on my production server. I am going to uninstall the Adaptable theme 1.6.0.2 (2018011502) tonight after the usage quiets down. This was the other theme that got updated at the same time as the other upgrades. The one thing I do know is all this started after the upgrades. I think it must be what you stated "Therefore either a local issue with conflicting styles that is out of my control or is the theme that the course backup would demonstrate." I will get back after I have done the theme uninstall. Regards, |
Just wanted to report in on what I have found. I removed the Adaptable theme and nothing change so I placed it back in and added the Enlight lite theme to see if a new theme is effected and the issue is on all themes. The issue I am having may be higher up then the theme level. |
.thumbnail class is part of the bootstrap styles core. Adaptable do not include or overwrite this class in any way. |
Ok, not in bootstrapbase.css file (which actually does not exist) as.... I cannot replicate the issue and changing theme does not affect the issue locally. Therefore the problem is with a custom plugin (not theme) or user entered CSS / bad markup. |
Thank you for your help. |
Just to give you and others an update on the cause of the broken thumbnail and thumbnails class. Regards, |
Update, issue resolved. Regards, |
So nothing to do with Essential? |
No the Essential theme is working perfect. It was a Big Blue Button plugin, thumbnail class that was overriding the Essential thumbnail class. I have a temp fix until they release there next version sometime in the next week or so. The help you gave, helped lead me down the road to find the issue. Both your support and the Blindside Networks customer support are outstanding to work with.
Thank you,
Robert
From: Gareth J Barnard [mailto:notifications@github.com]
Sent: Tuesday, February 27, 2018 7:23 PM
To: gjb2048/moodle-theme_essential
Cc: WINGsRealityEDU; Author
Subject: Re: [gjb2048/moodle-theme_essential] Class thumbnails broken (#858)
So nothing to do with Essential?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <#858 (comment)> , or mute the thread <https://github.com/notifications/unsubscribe-auth/Aib3_x6og9R-IHbTTC7dNCaWceE4NhwCks5tZJxMgaJpZM4R4OMQ> . <https://github.com/notifications/beacon/Aib3_2pimc69_oJNacf_gNF5VVlJF_SEks5tZJxMgaJpZM4R4OMQ.gif>
|
Week old upgrade to Moodle 3.4.1 (Build: 20180115) PHP 7.0.0 and Essential 3.4.0.1 (Build: 2017102900).
Now any use of the Class=”thumbnails” and the hole container falls apart. Here are some images one of Essential > Style guide
The other is in our sites library
The text was updated successfully, but these errors were encountered: