Skip to content

Add editor component search loading skeleton#2468

Merged
Mbeaulne merged 1 commit into
masterfrom
06-29-add_editor_component_search_loading_skeleton_matt
Jun 29, 2026
Merged

Add editor component search loading skeleton#2468
Mbeaulne merged 1 commit into
masterfrom
06-29-add_editor_component_search_loading_skeleton_matt

Conversation

@Mbeaulne

@Mbeaulne Mbeaulne commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator

Description

Improves the component search UX with two key additions:

  1. Search skeleton loading state — While a search query is debouncing or pending, a skeleton placeholder (5 rows with icon, title, and subtitle bones) replaces the previous spinner-in-header approach, giving users a clearer sense of incoming results.

  2. AI rerank progress indicator — When AI reranking is active, an animated inline status message cycles through verbs ("Scanning", "Comparing", "Scoring", "Ranking") every 1.2 seconds to communicate that the AI is working, rather than showing a bare spinner.

Additionally, component icons in the sidebar now reflect their source kind with distinct colors:

  • published → emerald
  • registered → teal
  • user / owned → orange
  • default → blue

Components with a published_by field now display a "Published by {author}" subtitle beneath the component name.

Related Issue and Pull requests

Type of Change

  • Bug fix
  • New feature
  • Improvement
  • Cleanup/Refactor
  • Breaking change
  • Documentation update

Checklist

  • I have tested this does not break current pipelines / runs functionality
  • I have tested the changes on staging

Screenshots (if applicable)

Test Instructions

  1. Open the component search sidebar and type a query — confirm the skeleton appears immediately while the debounce is pending, then resolves into results.
  2. Trigger AI reranking and confirm the animated progress label appears below the search box, cycling through verbs roughly every 1.2 seconds.
  3. Inspect components from different sources (published, registered, user-owned) and confirm their icons render in the correct colors.
  4. Confirm that components with a published_by value display the "Published by …" subtitle.

Additional Comments

The isSearching flag is derived by comparing the local (immediate) input value against the deferred committed query, so the skeleton is shown for the full debounce window without requiring any extra async state.

@github-actions

github-actions Bot commented Jun 29, 2026

Copy link
Copy Markdown

🎩 Preview

A preview build has been created at: 06-29-add_editor_component_search_loading_skeleton_matt/8bed990

@Mbeaulne Mbeaulne force-pushed the 06-29-add_editor_component_search_loading_skeleton_matt branch 8 times, most recently from e457ad3 to c102a63 Compare June 29, 2026 16:57
@Mbeaulne Mbeaulne marked this pull request as ready for review June 29, 2026 17:15
@Mbeaulne Mbeaulne requested a review from a team as a code owner June 29, 2026 17:15
@Mbeaulne Mbeaulne force-pushed the 06-29-add_editor_component_search_loading_skeleton_matt branch from c102a63 to bb5303a Compare June 29, 2026 17:20

Mbeaulne commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator Author

Merge activity

  • Jun 29, 5:50 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Jun 29, 5:58 PM UTC: Graphite rebased this pull request as part of a merge.
  • Jun 29, 6:01 PM UTC: @Mbeaulne merged this pull request with Graphite.

@Mbeaulne Mbeaulne changed the base branch from 06-29-limit_editor_component_search_result_noise_matt to graphite-base/2468 June 29, 2026 17:51
@Mbeaulne Mbeaulne changed the base branch from graphite-base/2468 to master June 29, 2026 17:56
@Mbeaulne Mbeaulne force-pushed the 06-29-add_editor_component_search_loading_skeleton_matt branch from bb5303a to 8bed990 Compare June 29, 2026 17:57
@Mbeaulne Mbeaulne merged commit 713cd2a into master Jun 29, 2026
17 checks passed
@Mbeaulne Mbeaulne deleted the 06-29-add_editor_component_search_loading_skeleton_matt branch June 29, 2026 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants