Skip to content

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

feat(template): make virtual views not set a height by default #1849

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
wants to merge 1 commit into
base: main
Choose a base branch
Loading
from

Conversation

eneajaho
Copy link
Member

No description provided.

@eneajaho eneajaho requested a review from hoebbelsB January 27, 2025 14:13
@github-actions github-actions bot added </> Template @rx-angular/template related 📚 Docs Web Documentation hosted on github pages labels Jan 27, 2025
Copy link

nx-cloud bot commented Jan 27, 2025

View your CI Pipeline Execution ↗ for commit a356749.

Command Status Duration Result
nx affected -t lint build test component-test e... ✅ Succeeded 1m 39s View ↗
nx build docs ✅ Succeeded 1m 15s View ↗
nx-cloud record -- npx nx format:check ✅ Succeeded 1s View ↗

☁️ Nx Cloud last updated this comment at 2025-01-27 14:17:05 UTC

Comment on lines +94 to +101
You can use the `--rx-vw-h` and `--rx-vw-w` CSS variables in order to define the placeholder dimensions after the virtual view is rendered at least once.

```html
<div style="min-height: var(--rx-vw-h, 100px); min-width: var(--rx-vw-w, 50px);"></div>
```

If the virtual view is not rendered at least once, the 100px and 50px values will be used as fallback the first time, and after that, the values will be updated to the actual dimensions of the virtual view.

Copy link
Member

Choose a reason for hiding this comment

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

maybe this deserves its own headline

RxVirtualViewObserver,
RxVirtualViewPlaceholder,
} from '@rx-angular/template/virtual-view';
import { RxVirtualView, RxVirtualViewContent, RxVirtualViewObserver, RxVirtualViewPlaceholder } from '@rx-angular/template/virtual-view';
Copy link
Member

Choose a reason for hiding this comment

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

meh formatting?

@hoebbelsB
Copy link
Member

@eneajaho can we finalize this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📚 Docs Web Documentation hosted on github pages </> Template @rx-angular/template related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.