Skip to content

fix: hide Create button in sidebar (both expanded and collapsed states) - #29

Open
GenosseOtt wants to merge 3 commits into
mainfrom
fix/hide-create-button
Open

fix: hide Create button in sidebar (both expanded and collapsed states)#29
GenosseOtt wants to merge 3 commits into
mainfrom
fix/hide-create-button

Conversation

@GenosseOtt

Copy link
Copy Markdown
Contributor

Summary

Hides the Create button in the Headlamp sidebar in both states:

  • Expanded sidebar: text button (MuiButton-textSecondary + MuiButton-sizeLarge)
  • Collapsed sidebar: icon button (MuiIconButton-colorPrimary + MuiIconButton-sizeMedium)

Uses language-independent MUI CSS class selectors (no aria-label) so the hiding works regardless of browser locale.

The expanded sidebar shows a button with text 'Create' inside the
MuiDrawer with classes MuiButton-textSecondary + MuiButton-sizeLarge
but no aria-label. The existing aria-label selector only caught the
collapsed sidebar variant. Add a drawer-scoped selector to hide both.

Signed-off-by: Johannes Ott <johannes.ott@sap.com>
Remove aria-label based selectors which break with non-English
browsers. Use MUI CSS classes (MuiDrawer-paper + MuiButton-textSecondary
+ MuiButton-sizeLarge) which are stable across all locales.

Signed-off-by: Johannes Ott <johannes.ott@sap.com>
The collapsed sidebar uses MuiIconButton-colorPrimary + MuiIconButton-sizeMedium
instead of the text button. Hide both variants with language-independent
MUI class selectors.

Signed-off-by: Johannes Ott <johannes.ott@sap.com>
@GenosseOtt
GenosseOtt requested a review from Lasserich as a code owner August 6, 2026 20:38
@GenosseOtt
GenosseOtt requested review from Hubert-Szczepanski-SAP and removed request for Lasserich August 7, 2026 07:17
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.

1 participant