New: completion icon added to transcript button (fixes #290)

[kirsty-hames]

Fixes #290

New

• On completion, a checkmark icon is displayed on the transcript button

Update

• Transcript button styled for consistency with .nav__btn text label/icon
• LESS styles structured in DOM order (skip to transcript btn, transcript btn, transcript body)

Not included in PR
Do you think we should display a plus icon for the inline transcript button to convey the button toggles/expands to reveal content? This would be consistent with the Accordion item button however I'm not wedded to this if people think it's unnecessary. See example below.

[joe-allen-89]

👍

[cahirodoherty-learningpool]

👀

[guywillis]

👍

Not included in PR
Do you think we should display a plus icon for the inline transcript button to convey the button toggles/expands to reveal content? This would be consistent with the Accordion item button however I'm not wedded to this if people think it's unnecessary.

I think it would make sense for a plus icon to be present especially as we have similar behaviour in the accordion.

[oliverfoster]

Yes to + buttons for expand/collapse generally, maybe not for external transcript? Perhaps? Dunno.

[github-actions]

🎉 This PR is included in version 6.5.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

[kirsty-hames]

Yes to + buttons for expand/collapse generally, maybe not for external transcript? Perhaps? Dunno.

Agreed. The external transcript button doesn't expand so there is no need for a + icon. New issue raised to display plus icon for inline transcript button - #292.