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

QTable pop-up editing does not work with sorting #7552

Closed
martijn788 opened this issue Aug 3, 2020 · 2 comments
Closed

QTable pop-up editing does not work with sorting #7552

martijn788 opened this issue Aug 3, 2020 · 2 comments

Comments

@martijn788
Copy link
Contributor

martijn788 commented Aug 3, 2020

Describe the bug
When you are editing a sorted column, the rows are flipping on the background. therefore you are changing all the different rows instead of the one you want to change.

Codepen/jsFiddle/Codesandbox (required)
https://codepen.io/martijn788/pen/QWNLgJY?editable=true&editors=101%3Dhttps%3A%2F%2Fquasar.dev%2Fvue-components%2Ftable

(Quasar table pop-up editing example from the documentation)

To Reproduce
Steps to reproduce the behavior:

  1. sort on calories (desc)
  2. Try to edit the calories of the top row to 1243232342 (typing one for one)
  3. The rows are changing on the background

Expected behavior
I expect that the sortation will only change when the row is saved by clicking on the save button.

Screenshots
quasar

Platform (please complete the following information):
OS:
Node:
NPM:
Yarn:
Browsers: Chrome & Firefox
iOS:
Android:
Electron:

@pdanpdan
Copy link
Collaborator

pdanpdan commented Aug 3, 2020

Do not edit the column used as key and the column(s) used for sorting. Make a clone of them and sort by one and edit the other.

@pdanpdan pdanpdan closed this as completed Aug 3, 2020
@martijn788
Copy link
Contributor Author

#7553

I've created a pull request to update the documentation with the extra information you provided here.

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

No branches or pull requests

2 participants