Migrate `GlDeprecatedSkeletonLoading` to `GlSkeletonLoader`
What does this MR do and why?
Related to #244669 (closed)
Screenshots or screen recordings
Before | After |
---|---|
How to set up and validate locally
- Short-circuit the condition to force the loading state to show up:
diff --git a/app/assets/javascripts/ide/components/ide_side_bar.vue b/app/assets/javascripts/ide/components/ide_side_bar.vue index f32d35bf774..beedc078879 100644 --- a/app/assets/javascripts/ide/components/ide_side_bar.vue +++ b/app/assets/javascripts/ide/components/ide_side_bar.vue @@ -35,7 +35,7 @@ export default { side="left" class="multi-file-commit-panel flex-column" > - <template v-if="loading"> + <template v-if="true"> <div class="multi-file-commit-panel-inner" data-testid="ide-side-bar-inner"> <div v-for="n in 3" :key="n" class="multi-file-loading-container"> <gl-skeleton-loader />
- Open the Web IDE in any project.
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Edited by Paul Gascou-Vaillancourt