Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

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

Sign up
Appearance settings

Draft: Feature/svelte support#829

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft
MischaPanch wants to merge2 commits intomain
base:main
Choose a base branch
Loading
fromfeature/svelte_support

Conversation

@MischaPanch
Copy link
Contributor

@MischaPanchMischaPanch commentedDec 10, 2025
edited
Loading

Based on the work of@tylersatre, we can now implement svelte support in a similar way and using the same abstractions as done for vue. Implementing svelte support will also give a practical test for how well the abstractions work (I expect them to work well for this).

Several attempts at supporting svelte were already done in the past, including#339 and#504 .

We can likely reuse a lot of the code there, and combine it with the companion server abstraction introduced on this branch.

I am creating the PR as a place for further discussion and also as a good way for@opcode81 and me to review the companion abstraction.

Thank you@tylersatre for the great work on this, highly appreciated!

tylersatre reacted with laugh emoji
tylersatreand others added2 commitsDecember 9, 2025 19:36
(allowing the principle upon which the Vue.js implementation is based to beapplied to similar cases)
@MischaPanchMischaPanch marked this pull request as draftDecember 10, 2025 09:36
@openhands-ai
Copy link

Looks like there are a few issues preventing this PR from being merged!

  • GitHub Actions are failing:
    • Tests on CI
  • There are merge conflicts

If you'd like me to help, just leave a comment, like

@OpenHands please fix the merge conflicts on PR #829 at branch `feature/svelte_support`

or

@OpenHands please fix the failing actions on PR #829 at branch `feature/svelte_support`

Feel free to include any additional details that might help me get this PR into a better state.

You can manage your notificationsettings

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

1 more reviewer

@linxiaotutututu1123linxiaotutututu1123linxiaotutututu1123 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@MischaPanch@linxiaotutututu1123@tylersatre

[8]ページ先頭

©2009-2025 Movatter.jp