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.

Sign up for GitHub

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

Jump to bottom

fix handling of ParsedHTML in @page macro #239

Merged
merged 4 commits into from
Nov 10, 2023
Merged

fix handling of ParsedHTML in @page macro #239

merged 4 commits into from
Nov 10, 2023

Conversation

hhaensel
Copy link
Member

@hhaensel hhaensel commented Nov 9, 2023

Currently, routing of ParsedHTMLString is broken in two ways:

  • ParsedHTMLValues and vectors thereof are not correctly handled by html(); layout and model are ignored
  • content of functions that generate ParsedHTML or vectors thereof is parsed; it is therefore impossible to route content that is not correctly parsed, e.g. '@' characters (cf. @click handler doesn't work v-on:click works #164)

This PR fixes both shortcomings together with a respective fix in Genie.

@hhaensel
Copy link
Member Author

hhaensel commented Nov 9, 2023

Tests fail because Genie is not yet updated. With Genie's PR688 tests will pass.
so we will need a Genie compat here.

@hhaensel hhaensel mentioned this pull request Nov 9, 2023
Merged
@hhaensel hhaensel merged commit 245cfbc into master Nov 10, 2023
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@PGimenez PGimenez Awaiting requested review from PGimenez

@AbhimanyuAryan AbhimanyuAryan Awaiting requested review from AbhimanyuAryan

@essenciary essenciary Awaiting requested review from essenciary

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@hhaensel