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

Clearing date or number column's cell value is not possible. #4881

Closed
gedinakova opened this issue May 23, 2019 · 1 comment · Fixed by #5542
Closed

Clearing date or number column's cell value is not possible. #4881

gedinakova opened this issue May 23, 2019 · 1 comment · Fixed by #5542
Assignees
Labels
🐛 bug Any issue that describes a bug 🧨 severity: medium grid: cell-editing grid: general version: 7.2.x version: 7.3.x ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged.

Comments

@gedinakova
Copy link
Contributor

Description

Describe the issue.

  • igniteui-angular version: 7.2.x, 7.3.x
  • browser: N/A

Steps to reproduce

  1. Run DEV demos
  2. Navigate to Grid Column Moving sample
  3. Double-click in a cell in "DateCreated" column
  4. Click the "X" button to clear the value
  5. Click on another row to commit the change

Result

The cell's value is reverted to the initial one.

Expected result

The cell should be empty.

Note:

In a number column, the value is converted to a zero which may not be the desired result.

@nrobakova nrobakova assigned nrobakova and unassigned Lipata Jul 26, 2019
@nrobakova nrobakova added the 🛠️ status: in-development Issues and PRs with active development on them label Jul 26, 2019
@nrobakova nrobakova added ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged. and removed 🛠️ status: in-development Issues and PRs with active development on them labels Jul 29, 2019
@nrobakova
Copy link
Contributor

This bug is fixed only for the date column. For the number column this behavior is by design.
As it is written in the link:
"For number data type, default template is using igxInput type="number", so if you try to update cell to a value which can not be parsed to a number your change is going to be discarded, and the value in the cell will be set to 0."

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Any issue that describes a bug 🧨 severity: medium grid: cell-editing grid: general version: 7.2.x version: 7.3.x ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants