rowyio / rowy

Low-code backend platform. Manage database on spreadsheet-like UI and build cloud functions workflows in JS/TS, all in your browser.
http://rowy.io
Other
6.14k stars 487 forks source link

Edit functionality for options of single select and multi select fields #1248

Closed mnmt7 closed 1 year ago

mnmt7 commented 1 year ago

Solves #1247

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
rowy-os ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 25, 2023 0:50am
1 Ignored Deployment | Name | Status | Preview | Comments | Updated (UTC) | | :--- | :----- | :------ | :------- | :------ | | **rowy-typedoc** | ⬜️ Ignored ([Inspect](https://vercel.com/rowy/rowy-typedoc/F1Mbmz1YH4XzB4JtLsqRiXkqqhRN)) | | | May 25, 2023 0:50am |
vercel[bot] commented 1 year ago

@mnmt7 is attempting to deploy a commit to the Rowy Team on Vercel.

A member of the Team first needs to authorize it.

harinij commented 1 year ago

@mnmt7 Thanks for submitting the PR. One quick update - can the UI be updated to also auto-save after an edit and clicked out of the text box? I.e someone should be able to double click on an option, edit it and click out of the cell to have the updated value saved instead of the current option to explicitly click the tick option on the right.

Screenshot 2023-05-24 at 12 19 29 pm
mnmt7 commented 1 year ago

@harinij I have added the functionality to auto-save when the user clicks outside the text field. Right now, the labels are editable when the user single-clicks on them. I just wanted to confirm whether it should be a single-click or double-click. Additionally, since the fields are now auto-saved when the user clicks outside the text field, should I remove the tick option or not?

harinij commented 1 year ago

@mnmt7 Thanks, the updates look good as it is, we will get it merged soon.