-
Notifications
You must be signed in to change notification settings - Fork 70
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] [while spin] Show the nams in plain text #6
Comments
Thank you for the proposal, Asim!
Could add some more details? I don't think I understand, but i would like
to.
…On Sun, May 17, 2020 at 6:39 PM Asim M Al Twijry ***@***.***> wrote:
Hello 👋
Thanks for the great work!
Is it possible to shuffle the names above the wheel in plain text?
Because when we add +200 name, you won't be able to read the names from
the wheel.
https://youtu.be/yL5clbrvmyY?t=480
Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#6>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAD2AIJUP3CV6464GS6IQVLRSCGUHANCNFSM4NDU4F7Q>
.
|
Here's an example
https://drive.google.com/file/d/1Ewh_kkJ98dipRevxCilRH2a8KeqUK7iT/view?usp=drivesdk
…On Mon, 18 May 2020, 05:18 Martin Omander, ***@***.***> wrote:
Thank you for the proposal, Asim!
Could add some more details? I don't think I understand, but i would like
to.
On Sun, May 17, 2020 at 6:39 PM Asim M Al Twijry ***@***.***
>
wrote:
> Hello 👋
> Thanks for the great work!
> Is it possible to shuffle the names above the wheel in plain text?
> Because when we add +200 name, you won't be able to read the names from
> the wheel.
> https://youtu.be/yL5clbrvmyY?t=480
> Thanks
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <#6>, or unsubscribe
> <
https://github.com/notifications/unsubscribe-auth/AAD2AIJUP3CV6464GS6IQVLRSCGUHANCNFSM4NDU4F7Q
>
> .
>
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#6 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA3U36LQXRS6GM2FLY6HCBLRSCLGFANCNFSM4NDU4F7Q>
.
|
Ah, very good, now I get it. Thank your for sending this! I think it looks
great. Will add it to the list of things to build next.
On Sun, May 17, 2020 at 8:02 PM Asim M Al Twijry <notifications@github.com>
wrote:
… Here's an example
https://drive.google.com/file/d/1Ewh_kkJ98dipRevxCilRH2a8KeqUK7iT/view?usp=drivesdk
On Mon, 18 May 2020, 05:18 Martin Omander, ***@***.***>
wrote:
> Thank you for the proposal, Asim!
>
> Could add some more details? I don't think I understand, but i would like
> to.
>
>
>
> On Sun, May 17, 2020 at 6:39 PM Asim M Al Twijry <
***@***.***
> >
> wrote:
>
> > Hello 👋
> > Thanks for the great work!
> > Is it possible to shuffle the names above the wheel in plain text?
> > Because when we add +200 name, you won't be able to read the names from
> > the wheel.
> > https://youtu.be/yL5clbrvmyY?t=480
> > Thanks
> >
> > —
> > You are receiving this because you are subscribed to this thread.
> > Reply to this email directly, view it on GitHub
> > <#6>, or unsubscribe
> > <
>
https://github.com/notifications/unsubscribe-auth/AAD2AIJUP3CV6464GS6IQVLRSCGUHANCNFSM4NDU4F7Q
> >
> > .
> >
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <
#6 (comment)
>,
> or unsubscribe
> <
https://github.com/notifications/unsubscribe-auth/AA3U36LQXRS6GM2FLY6HCBLRSCLGFANCNFSM4NDU4F7Q
>
> .
>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#6 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAD2AINVBRJ5MJYBTXV6SETRSCQM3ANCNFSM4NDU4F7Q>
.
|
The Wheel has a method getNameAtPointer(). Naive approach would be to call a function at an interval and update DOM innerText, depending upon the interval, the value will not be in exact sync with entry at the pointer. Also updating DOM innerText too frequently will have performance implications. A slightly better performance approach would be to raise events as the value at pointer changes, however, this will still have issues at the high speed of wheel rotating. Unless the values are not updated at high speed and only updated at certain low speed. The correct way to solve this would be to use the same animation approach used to spin the wheel. That way both the name at the pointer and the name value will be always in sync, as the browser paints them. |
Excellent analysis, johnberry09! Thank you. |
Hello 👋
Thanks for the great work!
Is it possible to shuffle the names above the wheel in plain text?
Because when we add +200 name, you won't be able to read the names from the wheel.
https://youtu.be/yL5clbrvmyY?t=480
Thanks
The text was updated successfully, but these errors were encountered: