Skip to content

event-list not working with dynamic content and custom height #1684

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

Open
2 tasks done
FabLax opened this issue Feb 10, 2025 · 2 comments · May be fixed by #1689
Open
2 tasks done

event-list not working with dynamic content and custom height #1684

FabLax opened this issue Feb 10, 2025 · 2 comments · May be fixed by #1689
Labels
type: bug Something isn't working Workflow: Issue created JIRA issue is created and will be analyzed

Comments

@FabLax
Copy link

FabLax commented Feb 10, 2025

Prerequisites

  • I have read the Contributing Guidelines.
  • I have not leaked any internal/restricted information like screenshots, videos, code snippets, links etc.

What happened?

When dynamically changing the content of the ix-event-list (e.g. using ngFor or @ for), newly added event-list-items don't get the custom item-height applied. Setting item-height="S" or "L" is working, but not with custom height like "60".

Suppose:
<ix-event-list item-height="60"> with dynamic content

Expected:
all ix-event-list-item inside have the same height (60px in rem)

Actual:
only initial rendered ix-event-list-item have the height, newly added ones have the default height

What type of frontend framework are you seeing the problem on?

Angular

Which version of iX do you use?

v2.6.1

Code to produce this issue.

import { Component } from '@angular/core';

@Component({
  selector: 'app-example',
  template: `
  <ix-event-list item-height="60">
  <ng-container *ngFor="let item of items">
    <ix-event-list-item color="color-primary">Text {{item}}</ix-event-list-item>
  </ng-container>
  </ix-event-list>
  <ix-button (click)=onAdd()>Add</ix-button>
  `,
})
export default class EventList {
  items = [1, 2, 3];

  onAdd() {
    this.items.push(this.items.length + 1);
  }
}
@FabLax FabLax added the triage We discuss this topic in our internal weekly label Feb 10, 2025
@danielleroux danielleroux added type: bug Something isn't working Workflow: Issue created JIRA issue is created and will be analyzed Contribution welcome 👨‍💻 This issue is open for contributions from the community and removed triage We discuss this topic in our internal weekly labels Feb 12, 2025
Copy link
Contributor

github-actions bot commented Feb 12, 2025

🤖 Hello @FabLax

Your issue will be analyzed and is part of our internal workflow.
To get informed about our workflow please checkout the Contributing Guidelines

JIRA: IX-2367

Copy link
Contributor

The issue has been labeled as Contribution welcome 👨‍💻. This issue is open for contributions from the community.

If you'd like to work on this issue, please comment here letting us know that you would like to submit a pull request for it. This helps us to keep track of the pull request and make sure there isn't duplicated effort.
Thank you!

@nuke-ellington nuke-ellington removed the Contribution welcome 👨‍💻 This issue is open for contributions from the community label Mar 4, 2025
This was referenced Apr 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working Workflow: Issue created JIRA issue is created and will be analyzed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants