Skip to content

Explain Code: Do not show "empty-state" when loading

Denys Mishunov requested to merge dmishunov/fix-empty-state-explain-code into master

What does this MR do and why?

The GitLab Chat: Adding empty state (!121703 - merged) introduced the empty state to GitLab chat. The condition to render this state had to be fine-tuned for the Explain Code feature to not show it when the request is being sent out already (isLoading: true state). This MR fixed it.

Screenshots or screen recordings

Before After
GitLab chat empty-state-gitlab-chat-before empty-state-chat-after
Explain Code empty-state-ec-before empty-state-ec-after

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Denys Mishunov

Merge request reports