Skip to content
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

Suggestion for Spreadsheet Bookmarklet #5

Open
Suleman-Elahi opened this issue Oct 14, 2020 · 5 comments
Open

Suggestion for Spreadsheet Bookmarklet #5

Suleman-Elahi opened this issue Oct 14, 2020 · 5 comments

Comments

@Suleman-Elahi
Copy link

Add functionality to jump cursor to the next row after pressing Enter. Right now, nothing happens, to go to new row, I have manually clicked in the next row and start editing.

@benstigsen
Copy link

I second this

@Gurbuzguven
Copy link

Hello, i have added that functionality take a look at my version.

@tijsmaas
Copy link
Contributor

In my attempt [ENTER] -> jump to next row, pressing [TAB] brings you to the next column, similar to excel. Live demo

Additional functionality:

  • Save&load to localstorage
  • Infinite rows (try press [ENTER] at the end)
  • Less than 1KB

@Gurbuzguven
Copy link

In my attempt [ENTER] -> jump to next row, pressing [TAB] brings you to the next column, similar to excel. Live demo

Additional functionality:

  • Save&load to localstorage
  • Infinite rows (try press [ENTER] at the end)
  • Less than 1KB

Hello i only have one suggestion, rewrite "eval(v.substring(1))" as "eval(v.toUpperCase().slice(1))", that way you prevent it to run harmful code

@tijsmaas
Copy link
Contributor

tijsmaas commented Sep 8, 2021

Great idea! Taking this route localstorage should also be renamed to something other than capital L, so that it wont be accessible. I managed to free up even more bytes, therefore unicode emoji for save&load can be put back in place. Live demo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants