Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sidebar: number-indicator not showing anymore when empty #1481

Merged

Conversation

akashyeole
Copy link
Member

@akashyeole akashyeole commented Sep 19, 2024

By creating this pull request you agree to the terms in CONTRIBUTING.md.
https://github.com/Infineon/.github/blob/master/CONTRIBUTING.md
--- DO NOT DELETE ANYTHING ABOVE THIS LINE ---

CONTRIBUTING.md also tells you what to expect in the PR process.

Description
Changed property height >>> max-height .

Related Issue
Closes #1478

📦 Published PR as canary version: 25.6.4--canary.1481.73e1ba0d587d8501e7e6c61d263333d311c43ca7.0

✨ Test out this PR locally via:

npm install @infineon/infineon-design-system-stencil@25.6.4--canary.1481.73e1ba0d587d8501e7e6c61d263333d311c43ca7.0
# or 
yarn add @infineon/infineon-design-system-stencil@25.6.4--canary.1481.73e1ba0d587d8501e7e6c61d263333d311c43ca7.0

@akashyeole akashyeole self-assigned this Sep 19, 2024
@akashyeole akashyeole linked an issue Sep 19, 2024 that may be closed by this pull request
@akashyeole akashyeole added the minor minor version bump label Sep 19, 2024
Copy link
Contributor

github-actions bot commented Sep 19, 2024

PR Preview Action v1.4.8
Preview removed because the pull request was closed.
2024-10-03 12:42 UTC

Copy link
Contributor

--STORYBOOK-PREVIEW--

@tishoyanchev
Copy link
Contributor

When number-indicator is empty, NaN is rendered.
number-indicator=""
NaN

@tishoyanchev
Copy link
Contributor

Also, this condition is very unclear.
!this.numberIndicator !== undefined

@tishoyanchev
Copy link
Contributor

Also, even when number-indicator is not set, its wrapper is still rendered in the DOM.
remains

@akashyeole akashyeole added patch patch version bump and removed minor minor version bump labels Oct 3, 2024
@tishoyanchev
Copy link
Contributor

@akashyeole

  • every sidebar-item instance throws an error
  • sidebar__nav-item-indicator wrapper still occupies space even when number-indicator is not present.

@tishoyanchev tishoyanchev changed the title fix(1478): fix deformed style of number indicator when empty Sidebar: number-indicator not showing anymore when empty Oct 3, 2024
@tishoyanchev tishoyanchev merged commit 31b4573 into master Oct 3, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch patch version bump
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sidebar: number-indicators are loaded by default
2 participants