em icon indicating copy to clipboard operation
em copied to clipboard

Inline Popup into CommandPalette

Open raineorshine opened this issue 7 months ago • 2 comments

  • [ ] Blocked by #2928
  • [ ] The Popup component is only used in CommandPalette, so inline it.
  • [ ] Rename PopupBase to Popup to reduce complexity.

raineorshine avatar May 13 '25 15:05 raineorshine

@raineorshine I already removed Popup in my PR, but haven't renamed PopupBase to Popup. Should this be done in the same PR?

yangchristina avatar May 31 '25 16:05 yangchristina

Separate PR, after #2928 is merged. Thanks!

raineorshine avatar May 31 '25 16:05 raineorshine