Skip to content

CardView: Fix the load panel overlapping the whole page#31005

Merged
Alyar666 merged 10 commits intoDevExpress:25_2from
Alyar666:cardview_loader_issue_25_2
Sep 10, 2025
Merged

CardView: Fix the load panel overlapping the whole page#31005
Alyar666 merged 10 commits intoDevExpress:25_2from
Alyar666:cardview_loader_issue_25_2

Conversation

@Alyar666
Copy link
Contributor

@Alyar666 Alyar666 commented Sep 4, 2025

No description provided.

@Alyar666 Alyar666 requested a review from pomahtri September 4, 2025 21:34
@Alyar666 Alyar666 self-assigned this Sep 4, 2025
@Alyar666 Alyar666 requested a review from a team as a code owner September 5, 2025 07:10
@Alyar666 Alyar666 force-pushed the cardview_loader_issue_25_2 branch from f1b8b00 to 0b9a7eb Compare September 8, 2025 07:59
templatesRenderAsynchronously: this.options.oneWay('templatesRenderAsynchronously').value,
}));

private readonly commonProps = {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor (change only there will be other changes)
Let's add here comment that

private calculatePosition(rootElement: HTMLElement): PositionConfig {
const window = getWindow();

if (rootElement.offsetHeight > window.innerHeight) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For future
It's not reactive

@Alyar666 Alyar666 merged commit ba99766 into DevExpress:25_2 Sep 10, 2025
424 of 426 checks passed
Alyar666 added a commit to Alyar666/DevExtreme that referenced this pull request Sep 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants