Option to display topics in multiple columns uses old Boostrap classes

gjbarnard / moodle-format_topcoll

Collapsed Topics course format for Moodle.

http://docs.moodle.org/en/Collapsed_Topics_course_format

GNU General Public License v3.0

35 stars 60 forks source link

Option to display topics in multiple columns uses old Boostrap classes #33

Closed lexxkoto closed 7 years ago

lexxkoto commented 7 years ago

The option to display topics in multiple columns uses the old Bootstrap 2 column classes (i.e. "row-fluid", "span3", "span4" and "span6". This doesn't work in newer versions of Boostrap (used by the Boost theme).

These should be replaced with the newer Boostrap "col-[xs|sm|md|lg|xl]-*" class names.

gjb2048 commented 7 years ago

You're wrong, not broken in Boost see: https://github.com/gjb2048/moodle-format_topcoll/blob/master/renderer.php#L1126.

gjb2048 commented 7 years ago

And: https://github.com/gjb2048/moodle-format_topcoll/blob/master/renderer.php#L1118