Skip to content
This repository has been archived by the owner on Mar 8, 2023. It is now read-only.

Input number in Content-Type Manager is broken for bigInteger, decimal and float #136

Closed
petersg83 opened this issue Apr 28, 2020 · 5 comments · Fixed by strapi/strapi#7178

Comments

@petersg83
Copy link

Describe the bug
As you can see on the gif:

  • I can't edit a bigInteger field
  • Arrows don't work on Decimal and Float
  • (not in the video but it displays 2,4 instead of 2.4 once saved. I would say that the point is more standard than the coma)

Steps to reproduce the behavior

  1. Create a content-type (let's say Country)
  2. Add 3 fields number with : bigInteger, decimal and float
  3. Go to the content-type manager to create a country
  4. See the bigInteger input and try the arrows of the 2 other fields
  5. You should see the error

Expected behavior

  • I can edit bigInteger
  • Arrows can increment and decrement the number of decimal and float (I let you decide how to handle the precision)

Screenshots
ezgif com-video-to-gif

System

  • Node.js version: v12.16.2
  • NPM version: 6.14.4
  • Strapi version: beta.20.1
  • Database: sqlite
  • Operating system: macOS
@derrickmehaffy
Copy link
Member

Bigint, Decimal, and float shouldn't have increase/decrease buttons in their edit panes and should just be input boxes

@lauriejim
Copy link
Contributor

@petersg83 is this issue related to Buffet or to the Content Manager?

@derrickmehaffy
Copy link
Member

@petersg83 is this issue related to Buffet or to the Content Manager?

It's possible that it's a combo of both

@petersg83
Copy link
Author

I don't really know. I went to see the code but it was too complicated for me to understand quickly. I think a front dev can better answer

@lauriejim lauriejim transferred this issue from strapi/strapi Jun 11, 2020
@ericliu-el
Copy link
Contributor

@lauriejim I went through some code logic and I found that the issue is more about strapi's side instead of buffet's issue.

So I submitted my first PR on strapi. Please feel free to check it out and help review it.

I'm very interested in strapi project and I want to help with it if possible. Thank you.

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

Successfully merging a pull request may close this issue.

4 participants