search

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

Move Left/Move Right doesn't work till page refresh #136

Closed jwalits closed 1 year ago

jwalits commented 1 year ago

Move Left/Move right doesn't work till page refresh

To Reproduce Steps to reproduce the behaviour:

  1. Go to any course using the Collapsed Topics format
  2. Turn editing mode on
  3. On any activity, click on the 3 dots and click "Move Right" or "Move Left"
  4. The section remains as is. Once you refresh the page, section will be indented.

Expected behaviour When "Move Right" or "Move left" is clicked, the section should be indented without needing page to be manually refreshed

Versions

gjb2048 commented 1 year ago

@jwalits CT currently does not react to this event and the area is difficult to understand. It's on my TODO list.

gjb2048 commented 1 year ago

cm_moveleft etc.

Screenshot 2023-08-01 165243

jwalits commented 1 year ago

Hey @gjb2048 - thanks for the quick feedback and the patch. We will test it out and let you know shortly if everything is working as expected.

Thanks again.

gjb2048 commented 1 year ago

@jwalits Any news on your testing please?

jwalits commented 1 year ago

Hey @gjb2048, sorry for the late update. This patch is working well - we're seeing the sections update correctly without having to refresh the page.

Thanks again for getting the patch out quickly. Much appreciated.

gjb2048 commented 1 year ago

Dear @jwalits, thanks for letting me know.

jwalits commented 1 year ago

Marking this issue as resolved. Thanks again for the quick action @gjb2048