Lbry-desktop: Inputs allow negative value

Created on 6 Aug 2017  路  9Comments  路  Source: lbryio/lbry-desktop

The Issue

The claim deposit input inside publish page allows a negative value.

Steps to reproduce

  1. Open publish page
  2. Go to Content URL > deposit
  3. Add a negative value.

Expected behaviour

It should probably allow 0 or minimum but not negative.

Actual behaviour

A negative value it's assigned,

System Configuration

Configuration | Version
-- | --
App | 0.14.3
Daemon (lbrynet) | 0.14.2
Wallet (lbryum) | 3.1.4
Platform | Linux (Linux-4.10.0-28-generic-x86_64-with-elementary-0.4.1-loki)

Screenshots


negative

All 9 comments

Don't think #398 has anything to do with this one.

ok, I was just pointing out a similar issue,
they are kind of related because both of them are: ux-issue

Less then two. I understand that this is not critical issue. I think you can change it to something like "pending" instead of negative value.

Fixed in Publish page, as well as in all LBC/USD hybrid inputs.

are you sure?? ^^ what about? -> #471

Merged #471.

Also I think maybe would be better to set the formFieldPrice default prop min to 0 :upside_down_face:

If themin isn't a multiple of step, Chrome has some undesirable behavior when using the arrows (e.g. it will step 5 down to 4.91, with min=0.01 and step=0.1).

Was this page helpful?
0 / 5 - 0 ratings

Related issues

tumispro picture tumispro  路  16Comments

jsigwart picture jsigwart  路  12Comments

tzarebczan picture tzarebczan  路  13Comments

kauffj picture kauffj  路  14Comments

kauffj picture kauffj  路  12Comments