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

feat: Document @Layout #3618

Merged
merged 9 commits into from
Sep 2, 2024
Merged

Conversation

caalador
Copy link
Contributor

Add documentation on using
@Layout auto layouting in
Flow and using server side
layout for Hilla views.

Add documentation on using
`@Layout` auto layouting in
Flow and using server side
layout for Hilla views.
@caalador caalador requested review from mshabarov and platosha August 27, 2024 11:12
@github-actions github-actions bot added the Language unchecked English language and AsciiDoc formatting checks haven't been done label Aug 27, 2024
@russelljtdyer russelljtdyer added Language checking English language and AsciiDoc formatting checking is in progress and removed Language unchecked English language and AsciiDoc formatting checks haven't been done labels Aug 28, 2024
@russelljtdyer russelljtdyer added Language checked English language and AsciiDoc formatting checks done and removed Language checking English language and AsciiDoc formatting checking is in progress labels Aug 28, 2024
Copy link
Contributor

@mshabarov mshabarov left a comment

Choose a reason for hiding this comment

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

These two articles should mention that officially the auto main layout is implied to be either in React (Hilla) or in Java (Flow), not both.

We can admit that both layouts are possible with some efforts, e.g. by wrapping React layout into Flow component, but at the moment it can be undocumented feature.

Otherwise looks good to me, can approve.

@mshabarov mshabarov merged commit 7639183 into latest Sep 2, 2024
3 of 4 checks passed
@mshabarov mshabarov deleted the feat/Server-auto-Layout-hilla-server-layout branch September 2, 2024 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Language checked English language and AsciiDoc formatting checks done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants