Comments (13)
There are inherent issues with using Class based componets in general. It's the motivation for why React came up with hooks.
Please read....https://reactjs.org/docs/hooks-intro.html#classes-confuse-both-people-and-machines
However, we found that class components can encourage unintentional patterns that make these optimizations fall back to a slower path. Classes present issues for today’s tools, too. For example, classes don’t minify very well, and they make hot reloading flaky and unreliable. We want to present an API that makes it more likely for code to stay on the optimizable path.
I'm sure this is also what Evan knows and understands and is trying to avoid. 😃
Scott
from rfcs.
I tried to create a sketch of the API design,
Demo available https://stackblitz.com/edit/vue-ts-class-api-0001
// A quick sketch of the API Design
// Not an actual Vue app
abstract class Component<TProps, TState>{
protected abstract data(): TState;
protected state: TState;
abstract render();
constructor(protected props: TProps, protected onRefresh: (c: Component<TProps, TState>) => void) {
this.constructProxyFromData();
// This is to simulate Vue rendering logic
this.onRefresh(this);
}
private constructProxyFromData(){
const obj = this.data();
this.state = new Proxy(obj as any, {
set: (obj, prop, val) =>{
obj[prop] = val;
this.onRefresh(this);
return true;
}
}) as any;
}
}
class Counter extends Component<{
counter: number
}, {
counter: number
}>{
data(){
return {
counter: this.props.counter
}
}
render(){
// This would be the template in real Vue app
return `<h1>Counter ${this.state.counter}</h1>`;
}
click(){
// Simulates a button click
this.state.counter++;
}
}
const appDiv = document.getElementById("app");
// This would be written as <Counter counter="1" /> in template
const counter = new Counter({
counter: 1
}, (c)=> { // simulates Vue rendering logic
appDiv.innerHTML = c.render();
});
// Simulates click
setInterval(()=>counter.click(), 1000);
from rfcs.
I believe this is no longer relevant as we have better TypeScript support now. As the issue poster, I won't mind if this issue is closed. Thank you to everyone involved on this discussion.
cc @LinusBorg Please feel free to either close or keep this issue opens depending on your insight. Thank you!
from rfcs.
Hi, thanks for wanting to contribute with an RFC.
However, you posted this as an issue whereas RFC are done as Pull requests.
So please follow these steps:
- fork this repository.
- in your fork, write out our proposal in a markdown file in the
/active-rfcs
directory, preferably in its own branch - open a Pull request from your fork to this main repository.
- In the PRs comment body, give short Summary of your PR, then link to the rendered version of the md file that you added.
- Close this issue, as discussions should happen in the PR comments.
Please see existing RFCs for examples. If you have any more questions, feel free to ask here.
from rfcs.
Also, your RFC should address points from the abandoned class API RFC that we already had, and how your approach intends to solve them or why they don't apply etc.
from rfcs.
Hi @LinusBorg and thank you for responding!
I am creating this as an issue because I wanted to have a discussion first about this with others, following:
Gathering feedback before submitting
It's often helpful to get feedback on your concept before diving into the level of API design detail required for an RFC. You may open an issue on this repo to start a high-level discussion, with the goal of eventually formulating an RFC pull request with the specific implementation design.
At the meantime, while gathering more feedback - I am currently trying to implement a PoC for this API.
If a discussion in a PR is preferred, I would close this issue and starts working on the PR.
Thank You!
from rfcs.
Ah, I see. Well, that's totally fine. It's just that you already wrote the issue in the formulaic way of the actual RFC, so I thought you wanted to submit it as an RFC directly.
It's fine to discuss here, just make sure to give some context at the beginning of your post that this is not yet meant as an RFC. Otherwise people might be confused, as evidenced by me :-P
from rfcs.
Oops, sorry for the confusion @LinusBorg - I edited the issue description, I hope it doesn't cause any more confusion. Sorry!
from rfcs.
Maybe TypeScript Transformer Plugin(typescript.CustomTransformers
) is a possible way?
from rfcs.
Hi @lovetingyuan, what would be the use case for the transformer plugin as I never used it?
from rfcs.
Using mobx we can build a ui framework-independent data layer. You are free to choose to use vue or react.
And It has TypeScript Friendly Class API.
from rfcs.
Also, your RFC should address points from the abandoned class API RFC that we already had, and how your approach intends to solve them or why they don't apply etc.
Now that class
component is not support in Vue3, is there anyway to migrate vue2 class component to vue3 component? @LinusBorg
from rfcs.
The vue-class-component is still being supported and has seen a release that's compatible with Vue 3.
so just continue using this.
the class API just won't be integrated into Vue core, which the dropped RFC was about.
from rfcs.
Related Issues (20)
- Adding a .v shortcut for .value HOT 4
- [FEAT] Required slots HOT 1
- Add abstraction for System Modifier Key: ctrl+c on Windows equals cmd+c on macOS HOT 1
- Pure TypeScript props declaration for the Options API, like we have defineProps() for the Composition API? HOT 4
- <script setup> : Make it possible to call defineProps many times HOT 10
- All Transition and TransitionGroup JavaScript hooks should be async
- Make runtime Props validation optional HOT 5
- Teleport as a composable/function
- An option to generate scoped styles using `:where` HOT 6
- Provide/inject for EffectScope
- Compiler: whitelist `import` global in template HOT 2
- Native CSS Modules
- Add impassive event modifier !passive HOT 2
- Stop script setup from executing any further HOT 1
- AsyncFunctionalComponent HOT 4
- Add a '.noinherit' modifier HOT 2
- Feature Request: `$emit` to return a Promise notifying when the event handler has run HOT 5
- Vue SSR app renderToString catch errors/warnings
- [Warnings] Silence Vue warnings in DEV
- expose 支持传入响应式对象 HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from rfcs.