-
Notifications
You must be signed in to change notification settings - Fork 462
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
Discussion: Lit-Element for Justgage #333
Comments
Hey @robertsLando. Thanks for researching into this. I like the idea of removing Raphael as a dependence. I am wondering if Lit-Element can provide us with the drawing features Raphael does? Can you research into pros/cons? I totally support this initiative, unless it limits us in achieving stuff from roadmap for 2.0, and future developments. |
@robertsLando did you see the demo that @robertsLando is pointing to: https://github.com/thomasloven/round-slider In effect, the demo shows how Lit-Element supports SVG (https://www.w3.org/Graphics/SVG/). You can see the setup here:
https://github.com/thomasloven/round-slider/blob/master/src/main.js#L1-L6 And where the magic happens: Looks great to me! |
Yes it looks amazing, we should also check if it is possibile to get events like Raphael does.
We could start a dev branch with the porting to LitElement?
Daniel
… On 1 Dec 2019, at 02:35, deezone ***@***.***> wrote:
@robertsLando did you see the demo that @robertsLando is pointing to: https://github.com/thomasloven/round-slider
In effect, the demo shows how Lit-Element supports SVG (https://www.w3.org/Graphics/SVG/). You can see the setup here:
import {
LitElement,
html,
css,
svg,
} from "lit-element";
https://github.com/thomasloven/round-slider/blob/master/src/main.js#L1-L6
And where the magic happens:
https://github.com/thomasloven/round-slider/blob/master/src/main.js#L295-L336
Looks great to me!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
Ooops, I flagged @robertsLando when I meant @toorshia :) What are your thoughts on my comments @toorshia ? |
@deezone just saw it, I see in example he wrote his custom "renderArc" method, right? Does this mean we would have to re-create all the drawing methods rom Raphael we are currently using? |
Yes but should not be that hard in my opinion, the only thing I actually don’t know is how to support Raphael events we currently bind too without using Raphael
Daniel
… On 12 Dec 2019, at 11:15, Bojan Djuricic ***@***.***> wrote:
@deezone just saw it, I see in example he wrote his custom "renderArc" method, right? Does this mean we would have to re-create all the drawing methods rom Raphael we are currently using?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
Allrighty then, let's start a dev branch for LitElement! 👍 @robertsLando can you take a crack at this? |
I think that we could start another repo because it would be a complete rewrite, almost everything will change in the code. What about a Justgage2?
Daniel
… On 12 Dec 2019, at 15:36, Bojan Djuricic ***@***.***> wrote:
Allrighty then, let's start a dev branch for LitElement! 👍 @robertsLando can you take a crack at this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
Whatever you say, chief. You're the engineering brains around here :) |
Unfortunally now I would not have much time to spend on it because I have many others repo to keep updated, I'm focusing on fixing bugs/compatibility issues but no time for new features/projects right now. Let's keep this open for the future, I would be glad to implement this :) |
Sure thing 👍:) |
Hi @toorshia and @deezone, I have found this interesting project on github that uses Lit-Element that is
It is really interesting because it allows to write really clean code for web components and maybe could allow us to also remove Raphael as dependence.
Any thoughts about it? The example I have linked shows a round slider that is not so far from this.
The text was updated successfully, but these errors were encountered: