Update content.js to use cards

djplaner / Content-Interface-Tweak

Improves both the task of creating content for Blackboard Learn, and reading that content.

https://djplaner.github.io/Content-Interface-Tweak/

GNU General Public License v3.0

0 stars 0 forks source link

Update content.js to use cards #31

Closed djplaner closed 3 years ago

djplaner commented 3 years ago

For sometime content.js has had some support for cards - but not quite contentWithCards level.

[ ] Get the content.js stuff working as current contentWithCards
[ ] update with latest cards.js

djplaner commented 3 years ago

Currently extractCardMetaData is not correctly able to replace the vtbegnerated stuff into ps

djplaner commented 3 years ago

addCardInterface is calling generateDateHtml with a unconverted date in date.start

Week 1 is getting converted into date.start=25 Feburary 2019.

The actual conversion is done in getTermDate, but there were two versions of getTermDate. An old one and a new one names getTErmDateCards. This was meant to be used by all card interface stuff. But it wasn't being used by the new code copied across.

However, turns out the old function is no longer being used. Move everything back to getTermDate and "remove" old getTermDate

djplaner commented 3 years ago

Remaining issues

Card Date is included double week

Work done in generateDateHtml....the week template had a hard coded week.

It's only doing 1 card per accordion

extractCardsFromContent is only getting 1 card. It's passed bbItems which has four APpears that the index counter looping through items is leaping to beyond the count

The loop variable wasn't defined using let. That fixed the problem.

djplaner commented 3 years ago

All working But the styles of cards is upset by being within the content interface styles.