palantir/blueprint

Suggest Component should have an easy way to prevent an empty menu from showing.

Open

#3,425 opened on 2019年3月16日

GitHub で見る
 (2 comments) (2 reactions) (0 assignees)TypeScript (20,263 stars) (2,167 forks)batch import
P2Package: selectType: bughelp wanted

説明

Hey @giladgray @adidahiya image

In the above image I have a <Suggest/> with the prop

initialContent={null}

Instead of showing the weird little empty menu, I'd rather not show anything. There doesn't seem to be an easy way to do this besides implementing an entire itemListRenderer override. I think that passing initialContent={null} should completely hide the Menu.

To do that all that would seemingly need to change is here:

https://github.com/palantir/blueprint/blob/3d6f13e8af98225ad8ca5ea6309bb406fd218b3e/packages/select/src/components/query-list/queryList.tsx#L264

If we added the this

if (menuContent === null && createItemView === null) {
  return null
}

to the next line then that weird empty menu wouldn't show up.

コントリビューターガイド

技術スタック
typescriptreact
領域
frontend
Issue 種別
feature
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
under 1 hour
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
なし
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
85
調査方針
Review the file `packages/select/src/components/query list/queryList.tsx` around line 264. Check the current logic for rendering menu content and implement the suggested null check to hide the menu when `initialContent` is null. Ensure that the change does not break other scenarios where `menuContent` might be validly null in conjunction with `createItemView`.