Automattic / sensei

Sensei LMS - Online Courses, Quizzes, & Learning
https://senseilms.com
GNU General Public License v2.0
539 stars 198 forks source link

Summary option for archive pages in Twenty Twenty theme breaks Courses archive page. #5570

Open aaronfc opened 2 years ago

aaronfc commented 2 years ago

Steps to Reproduce

  1. Enable Twenty Twenty theme.
  2. Make sure you have a page selected in Sensei LMS > Settings > Course Archive Page.
  3. Go to Appearance > Customise > Theme Options.
  4. Set "On archive pages, posts show:" to "Summary".
  5. Navigate to the Course archive page.

What I Expected

To see the Course Archive page correctly.

What Happened Instead

The content of the Course Archive page has been stripped out of HTML tags and it's just a single line string.

PHP / WordPress / Sensei LMS version

Sensei LMS: 4.6.3 Theme: Twenty Twenty

Screenshot / Video

Looks like this

image

Should look like this (ignore misalignments)

image

Context / Source

StefMattana commented 2 years ago

Followup from the user:

Indeed changing to «full text» fixed the problem. The courses are listed. But there is no border on this page and I thought the courses will be shown in the new course listing grid that you proposed. Instead it replaced the new block and puts out the old courses-overview listing.

I am not really happy with the solution full text, because this breaks my portfolio category page. Also I am not happy with the course listing cause in Google people can find me with «Kurse» and they come to this page wich is still not formatted correct (a part of finally listing the courses)

I don't think we can suggest a workaround or another solution (even if it's temporary) to mitigate this. Noting the user doesn't want to change the theme.

github-actions[bot] commented 2 years ago

Support References

This comment is automatically generated. Please do not edit it.