viresh-ratnakar / exolve

Online interactive crossword software in JavaScript
MIT License
73 stars 15 forks source link

This app is amazing, thank you #9

Closed ballantynedewolf closed 4 years ago

ballantynedewolf commented 4 years ago

Can I ask, why is the grid definition appended to the URL? Ballantyne

ballantynedewolf commented 4 years ago

And another question: why are annotations not available from Reveal This, only from Reveal All?

viresh-ratnakar commented 4 years ago

Thanks for the kind words about Exolve!

Can I ask, why is the grid definition appended to the URL?

The current solution state is saved and updated in the URL, so you can bookmark or otherwise save the URL to capture your state (and can open the URL on another device to transfer state). The state is also saved in a cookie, but that has some issues (cookie limits, auto-deletions), so I found it safer to also save in the URL. https://github.com/viresh-ratnakar/exolve/blob/master/README.md#saving-state

On Tue, Sep 24, 2019 at 11:52 PM ballantynedewolf notifications@github.com wrote:

And another question: why are annotations not available from Reveal This, only from Reveal All?

That's a good point. I'll try to fix this.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/viresh-ratnakar/exolve/issues/9?email_source=notifications&email_token=AJQ562FQ7IJUJOJOQFNRVT3QLMDA7A5CNFSM4I2GPGXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7Q2FIQ#issuecomment-534880930, or mute the thread https://github.com/notifications/unsubscribe-auth/AJQ562AKV3GB5GAMGZY4HCDQLMDA7ANCNFSM4I2GPGXA .

viresh-ratnakar commented 4 years ago

"Reveal/Clear this" now shows/clears the annotation (if available) for the clues associated with the current cell, in version v0.24.

Antagony1060 commented 4 years ago

I've just started experimenting with adding annotations, and I'm very impressed once again. One thing I think would be an improvement, though, is having the 'Check' buttons also reveal the annotation, but only for correct answers.

ballantynedewolf commented 4 years ago

Thanks. I should've read your docs more closely! I was just so excited by how perfect the app is.😁 Jonathan

On Thu., 26 Sep. 2019, 01:14 Viresh Ratnakar, notifications@github.com wrote:

Thanks for the kind words about Exolve!

Can I ask, why is the grid definition appended to the URL?

The current solution state is saved and updated in the URL, so you can bookmark or otherwise save the URL to capture your state (and can open the URL on another device to transfer state). The state is also saved in a cookie, but that has some issues (cookie limits, auto-deletions), so I found it safer to also save in the URL.

https://github.com/viresh-ratnakar/exolve/blob/master/README.md#saving-state

On Tue, Sep 24, 2019 at 11:52 PM ballantynedewolf < notifications@github.com> wrote:

And another question: why are annotations not available from Reveal This, only from Reveal All?

That's a good point. I'll try to fix this.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub < https://github.com/viresh-ratnakar/exolve/issues/9?email_source=notifications&email_token=AJQ562FQ7IJUJOJOQFNRVT3QLMDA7A5CNFSM4I2GPGXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7Q2FIQ#issuecomment-534880930 , or mute the thread < https://github.com/notifications/unsubscribe-auth/AJQ562AKV3GB5GAMGZY4HCDQLMDA7ANCNFSM4I2GPGXA

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/viresh-ratnakar/exolve/issues/9?email_source=notifications&email_token=AGBJIAOFB3CGC5B2DLZ76H3QLN57FA5CNFSM4I2GPGXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7SIXCQ#issuecomment-535071626, or mute the thread https://github.com/notifications/unsubscribe-auth/AGBJIAL36N55FT24ATC4FCDQLN57FANCNFSM4I2GPGXA .

viresh-ratnakar commented 4 years ago

With v0.25 now, "Check this" and "Check all" do everything that "Reveal this" and "Reveal all" do, if they find no errors.

For "Check all" this is slightly different from what Antagony1060 requested (as it will not reveal annos for correct entries if there is a mistake elsewhere), but was easier to implement.

ballantynedewolf commented 4 years ago

Enjoying the updates in 0.26. This is a very good app IMO.

Can I suggest a feature: a text box for typing in anagram fodder, with a Fisher-Yates shuffle function called by an html button. Solvers can then shuffle the letters til they get some inspiration.

Jonathan

@@-? ( - the fashion and sober design will make you special

On Thu, 26 Sep 2019 at 01:14, Viresh Ratnakar notifications@github.com wrote:

Thanks for the kind words about Exolve!

Can I ask, why is the grid definition appended to the URL?

The current solution state is saved and updated in the URL, so you can bookmark or otherwise save the URL to capture your state (and can open the URL on another device to transfer state). The state is also saved in a cookie, but that has some issues (cookie limits, auto-deletions), so I found it safer to also save in the URL.

https://github.com/viresh-ratnakar/exolve/blob/master/README.md#saving-state

On Tue, Sep 24, 2019 at 11:52 PM ballantynedewolf < notifications@github.com> wrote:

And another question: why are annotations not available from Reveal This, only from Reveal All?

That's a good point. I'll try to fix this.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub < https://github.com/viresh-ratnakar/exolve/issues/9?email_source=notifications&email_token=AJQ562FQ7IJUJOJOQFNRVT3QLMDA7A5CNFSM4I2GPGXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7Q2FIQ#issuecomment-534880930 , or mute the thread < https://github.com/notifications/unsubscribe-auth/AJQ562AKV3GB5GAMGZY4HCDQLMDA7ANCNFSM4I2GPGXA

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/viresh-ratnakar/exolve/issues/9?email_source=notifications&email_token=AGBJIAOFB3CGC5B2DLZ76H3QLN57FA5CNFSM4I2GPGXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7SIXCQ#issuecomment-535071626, or mute the thread https://github.com/notifications/unsubscribe-auth/AGBJIAL36N55FT24ATC4FCDQLN57FANCNFSM4I2GPGXA .

viresh-ratnakar commented 4 years ago

Sure, will add an option to create this.

viresh-ratnakar commented 4 years ago

I added a scratch pad in v0.27 (with a control to shuffle the selected text in it -- or all text if none has been selected).

The scratch pad is hidden, but can be toggled with the "Tools" link (I renamed the erstwhile "Controls" link to "Tools," reusing it for this).

There are tooltips that describe the functionality on hovering over the Tools link as well as the Shuffle control.

viresh-ratnakar commented 4 years ago

Closing this issue, please reopen if needed.