tutorialkit icon indicating copy to clipboard operation
tutorialkit copied to clipboard

`Toggle Terminal` button should be hidden when `terminal: false`

Open AriPerkkio opened this issue 1 year ago • 3 comments

Describe the bug

When terminal is disabled in metadata, the Toggle Terminal button is still visible. It just shows empty blank terminal when toggled.

Link to a StackBlitz project which shows the error

No response

Steps to reproduce

  1. Define in metadata:
---
type: lesson
title: Test
terminal: false
---
  1. Open page and click Toggle Terminal

Expected behavior

Whole Toggle Terminal button should be hidden.

Screenshots

image

Platform

  • TutorialKit version: 0.2.1

Additional context

No response

AriPerkkio avatar Sep 02 '24 06:09 AriPerkkio

Hello @AriPerkkio. I was trying to recreate this issue but it seems to be working fine. Could you please help me out?

https://github.com/user-attachments/assets/924cf1a8-c0f2-4161-a268-cb8b4e9336e1

Charan167 avatar Sep 28 '24 14:09 Charan167

but it seems to be working fine

Great, thanks for checking this! 🙌

I'll check later if this issue is still valid. Feel free to proceed with the other issue mentioned in https://github.com/stackblitz/tutorialkit/pull/343#pullrequestreview-2322783870.

AriPerkkio avatar Sep 30 '24 05:09 AriPerkkio

I'll check later if this issue is still valid.

Yup, this is still valid. Added failing test case at https://github.com/stackblitz/tutorialkit/tree/test/terminal-false.

Task assigned to @RonithManikonda.

tk-terminal-false.webm

AriPerkkio avatar Oct 14 '24 08:10 AriPerkkio