diff --git a/packages/cursorless-org-docs/src/docs/contributing/MissingLanguageScopes.tsx b/packages/cursorless-org-docs/src/docs/contributing/MissingLanguageScopes.tsx index e74ab7e762..11c722bbc2 100644 --- a/packages/cursorless-org-docs/src/docs/contributing/MissingLanguageScopes.tsx +++ b/packages/cursorless-org-docs/src/docs/contributing/MissingLanguageScopes.tsx @@ -43,17 +43,21 @@ function Language({ }): React.JSX.Element | null { const scopeSupport = languageScopeSupport[languageId] ?? {}; - const unsupportedFacets = scopeSupportFacets.filter( - (facet) => scopeSupport[facet] === ScopeSupportFacetLevel.unsupported, - ); - const unspecifiedFacets = scopeSupportFacets.filter( - (facet) => scopeSupport[facet] == null, - ); + let unsupportedFacets = scopeSupportFacets + .filter( + (facet) => scopeSupport[facet] === ScopeSupportFacetLevel.unsupported, + ) + .sort(); + let unspecifiedFacets = scopeSupportFacets + .filter((facet) => scopeSupport[facet] == null) + .sort(); - const unsupportedScopes = facetsToScopes(unsupportedFacets, showPrivate); - const unspecifiedScopes = facetsToScopes(unspecifiedFacets, showPrivate); + if (!showPrivate) { + unsupportedFacets = unsupportedFacets.filter((f) => !isPrivate(f)); + unspecifiedFacets = unspecifiedFacets.filter((f) => !isPrivate(f)); + } - if (unsupportedScopes.length === 0 && unspecifiedScopes.length === 0) { + if (unsupportedFacets.length === 0 && unspecifiedFacets.length === 0) { return null; } @@ -67,19 +71,19 @@ function Language({ - {renderFacets("Unsupported", unsupportedScopes)} - {renderFacets("Unspecified", unspecifiedScopes)} + {renderFacets("Unsupported", unsupportedFacets)} + {renderFacets("Unspecified", unspecifiedFacets)} > ); } function renderFacets( title: string, - scopes: string[], + facets: string[], ): React.JSX.Element | null { - const [open, setOpen] = useState(scopes.length < 4); + const [open, setOpen] = useState(facets.length < 10); - if (scopes.length === 0) { + if (facets.length === 0) { return null; } @@ -91,7 +95,7 @@ function renderFacets( return (