Skip to content

Conversation

BrunoQuaresma
Copy link
Collaborator

@BrunoQuaresma BrunoQuaresma self-assigned this Oct 4, 2023
Copy link
Member

@code-asher code-asher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Just the one issue with the refresh button.

Copy link
Member

@code-asher code-asher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Beautiful! It seems like the LoadedScriptsAlert is not working, but I am pretty sure that was a problem before as well, so I say merge it in and we can create a new issue for that. I guess lifecycleState is not getting updated somehow.

@code-asher
Copy link
Member

Ah sorry one more thing, was the removal of the stories intentional?

@BrunoQuaresma BrunoQuaresma merged commit a5f8300 into main Oct 5, 2023
@BrunoQuaresma BrunoQuaresma deleted the bq/remove-opacity-from-terminal branch October 5, 2023 21:06
@github-actions github-actions bot locked and limited conversation to collaborators Oct 5, 2023
@BrunoQuaresma
Copy link
Collaborator Author

It seems like the LoadedScriptsAlert is not working,

I will investigate that.

Ah sorry one more thing, was the removal of the stories intentional?

Yeap. I was not able to storybook multiple components so I just gave up but I will try it again.

@BrunoQuaresma
Copy link
Collaborator Author

BrunoQuaresma commented Oct 5, 2023

Opened the issue to investigate the lifecycle not being updated: #10088

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove opacity from disconnected/ended web Terminal
2 participants