* Split ui-api, update API version
* Update (awaiting API changes to go through)
* Cleanup ui-api exports
* Bump API (SubscriptionPromise support)
* Linting fixes
* Update renames
* Do display balances.Transfer (shows for non-owned accounts)
* Bump API
* helpful error message when input address is not in intentions list
* check address format is valid, set error msg if not
* fix: state validate
* fix: translatable strings, address format validation from decoded address
* Import Labelled Component for error reporting
* Translatable labelled error reporting components
* handle assertion errors when decoding nominee address
* create ui-app general InputError component
* Use Input Error component to format input errors
* edits to setting state for readability and DRYness
* edits for style guidelines, remove 'any' types
* fix: Fixes Issue #144. Balance to submit with transaction changed to string
Placeholder indicates acceptable range of values to user in the interim until form validation incorporated.
Max value must be the balance of the account minus one, otherwise the Polkadot Node will give error `Runtime: proposer's balance too low`.
Min value must be above 0.000000000000001, otherwise the Polkadot Node will give error `Runtime: proposer's balance too low`
* fix: Add validations to string input. Add unit tests
Note that entering a large number after all inputs are valid (including code dragged and dropped) causes error `Uncaught Error: byte array longer than desired length`. This is because it only starts encoding after all input fields are valid, but the maximum balance is 64 bits for poc-1 and 128 bits for latest master branch
* fix: Add internationalisation, maxLength to input, validate depending on chain used
- add internationalisation
- add maxLength to input field depending on whether poc-1 (64bit) or latest (128bit) chain used
- pass down api chain data via props
- add unit tests for balance input field, new tests not all passing but don't understand why. since using unsigned 64bit and 128bit
- add balance input field validation to only accept numbers
- when i add `&& maxBN64Bit.gte(inputBN)` or `&& maxBN128Bit.gte(inputBN)` it doesn't appear
to work correctly as in the UI it doesn't allow a length as much as expected
* fix: Must pass in as string as String(xyz) converts to scientific notation x...e+21 for large numbers
* fix: Remove String(xyz) that was converting to scientific notation. Throw on it or non-string input.
* fix: Prop value is BN converted to string with toString(10) according to BN.js docs
* fix: Translate the placeholder
* refactor: Move external dependencies above local. Add missing whitespace
* fix: Scientific value detection on e+ not just letter e
* feat: Add input field validation message notification to extrinsic balance
* refactor: Move expected responses generation into enum in separate file
* WIP - trying with input type number and onKeyDown
* WIP - split into onKeyDown and onKeyUp
* feat: Add scientific notation equivalent in info message. Yet to use converted value when submit
* fix: Allow user to select all that they've entered in input field
* feat: Converts from scientific notation with submit. Need to fix input as crashes on 9e19
* fix: Remove debugging console.log so it doesn't crash
* fix: Calculate size based on actual bitlength before checking BN value against max
* fix: Allow use of decimal point only in scientific notation. Prevent successive decimal point entry
* fix: Update error message with example
* feat: Create InputNumber instead of extending Balance. Requires PR with mods to polkadot-js/apps
* fix: Remove accidently added vscode config
* fix: Remove step and min props since not using input type number
* refactor: Remove debugging console logs from extrinsics
* fix: Remove disused step
* fix: Remove comments and add whitespace
* fix: Remove unnecessary check if balance is defined when it must be
* fix: Fix logic
* fix: Show error when user enters e or e+ without a number afterwards
* test: Update existing tests. Still must add new tests
* test: Fix tests for scientific vs exponential notation
* test: Add remaining tests
* test: Refactor tests in order used
* refactor: clean up comments and duplicate code
* fix: Replace implementation with restrictions on key combos instead of horrible previous approach
* fix: Prevent notification with info from displaying when there are errors. Remove console logs
* fix: Prevent user pasting balance in exponential notation with invalid exponent. Only allow + symbol entry immediately after letter e
- Only allow user to enter + symbol immediately after letter e for exponential input.
- Prevent error when user pastes input with invalid exponent (i.e. when user pastes into the input field for Balance 232e11+ it was crashing with error `error: RangeError: Invalid array length
at scientificToDecimal`)
* refactor: Add comment to input cursor and move
* remove todo note
* fix: Prevent user entering 0's at start
* restore: Restore Balance instead of incorrect implementation of InputNumber
* WIP - initially add generic identity function where Balance wraps InputNumber type
* remove generic ident fn
* feat: Add GenericInputNumber
* fix: Add InputNumber generic wrapper of Input and Notification for use in Balance or elsewhere
* fix: Address review comments.
* fix: Fix linting errors and remove useless new line
* fix: Add WarnMessage. Remove UI message for it to be account balance minus 1
* fix: Fix bitLength error when small amount entered. Fix NaN test. Allow input of 0 with warning.
* fix: Allow select all, cut, copy, paste whilst preventing input of a, x, c, v
* fix: Fix linting error that appeared in CI
* fix: Remove conflicting restriction on use of 0
* test: Add enzyme test for default load of Balance component
* test: Add basic enzyme tests to simulate Info, Warning, and Error Notifications
* fix: Address some review comments. See commit details
* Refactor nested condition into `isValidLength` instead of `(values[index].type === 'Balance' ? checkValueBitLength(...`
* Fix alphabetical ordering
* Change to `Between 0 DOT` instead of `Between 1 DOT`
* Add License heaer to chainSpec.ts
* Replace `if (!(typeof input === 'string')) ` with `isString`
* Add blank lines for all blocks in isValidBalanceExpectedResponses.ts
* fix: Remove NaN check since unnecessary. Replace with test for just single e input
* Remove check for NaN since unnecessary. NaN is not string type so will show error associated with the test for string
* Add initial check for input value that contains just 'e' since it should not show same error message as say '1e'
* fix: Revert to just allow integers and decimals without scientific notation
* docs: fix comments
* refactor: Split isValid ternary across multiple lines for readability
* refactor: Remove unnecessary blank line
* fix: Remove nested if condition
* refactor: Split isCopy function definiton from function body implementation similar to in ui-signer
* refactor: Split function definition from function body for all keyboard utils
* fix: Replace event any type with KeyboardEvent type for all keyboard functions
* fix: Remove TranslateFunction with just string as type of messages
* fix: Replace deprecated keyCode and which with key.
* fix: Move isValidLength into isValidBalance
* fix: Split imported packages
* fix: Split imported packages in Balance spec
* fix: Change placeholder to request positive integer input and update messages
* refactor: Split function body onto new line
* fix: Split function body onto new line for readability
* refactor: Split ternary across multiple lines
* fix: Use BN instead of Number for info message
* fix: pass up destructured event properties
* fix: Remove SUIEvent type definition
* fix: Reduce maxLength in digits that user allowed to enter by 1
* merge latest from master
* fix: Remove transfer terminology from Balance placeholder
* Move logic to child
* fix: Remove assignment to optional
* fix: Remove DOT references since number can be anything and substrate chains have own token
* fix: Add missing blank lines and reorder alphabetically
* Non-numeric keys outright without error notification. Use MAX_SAFE_INTEGER built-in
* fix: Reordering
* fix: Remove showing error to user when they enter value over max bitlength
* fix: Restrict user enter value greater than max safe integer
* fix: Do not pass translate down from Balance. Fix tests
* test: Move tests to InputNumber
* Return only value from onChange in InputNumber. Bug since isValid keydowns allowed
* fix: Fixes review comments. See commit message details:
* `onChange` in InputNumber is just `onChange: (value: BN) => void` without an `isValid` property. `isValid` is now set to `true` in Balance as we've taken care of all the invalid cases (i.e. automatically reverting the input if the user pastes an invalid value, or preventing input of non-decimal values in keyDown)
* New input value is reverted to previous valid input field value if the input field value becames invalid (i.e. if they enter a value that's above the maximum allowable). i.e. 0.1111111111111111111111111111111 is allowed, but if any additional 1's are entered (i.e. 0.1111111111111111111111111111111111111) at the end it exceeds max for 128 bit, and we automatically prevent these invalid values from being entered (instead of informing the user through a notification).
* Refer to input as "input" instead of "balance" (i.e. for i18n keys, etc)
* Remove duplicate `KEYS.BACKSPACE`
* Add unit tests for isNonDecimal and isNonInteger
* Remove `KEYS.ZERO` from `KEYS_ALLOWED` and add to isValidDecimal and isValidInteger checks instead
* Split isValidNumber into isValidNumber and isValidKey. Since keyDown has access to the key and can prevent the input (but only has access to the previous input field value before the new key was entered), we check if the the new key entered is valid in isolation of the whole valued by calling isValidKey. Then in onChange where we have access to the new input field value we validate it calling isValidNumber.
* In InputNumber.tsx the Input component accepts `defaultValue={defaultValue || '0'}`, and also we're also overriding the value with the previous valid value with `value={revertedValue}` if the user input is found to be invalid. However React complains with the following errors in the browser Chrome Inspector:
```
Warning: Input contains an input of type text with both value and defaultValue props. Input elements must be either controlled or uncontrolled (specify either the value prop, or the defaultValue prop, but not both). Decide between using a controlled or uncontrolled input element and remove one of these props. More info: https://fb.me/react-controlled-components
```
```
Warning: A component is changing a controlled input of type text to be uncontrolled. Input elements should not switch from controlled to uncontrolled (or vice versa). Decide between using a controlled or uncontrolled input element for the lifetime of the component.
```
I have tried as many ways as possible to just use one or the other (i.e. defaultValue or value) to resolve these warnings but so have not being able to replicate the same functionality that works.
* fix: Remove new line
* fix: Remove complexity. See detailed commit comments
Progress update:
* ASSUMPTION: InputNumber may be used in future for useful error, warning, and info messages, so leave the boilerplate props along with Notification.
* ASSUMPTION: Displaying an info message in scientific notation i.e. 3e+7
causes more confusion than benefit since most users are non-techy. In future this when we have Issue #86 Settings App, it could be added as an optional extra that could be toggled by the user along with Issue #224 E-Notation
* ACTIONS:
* [x] - Remove display of scientific notation equivalent
* [x] - Remove warning 'number.warn.zero' when user enters 0 value
* [x] - Remove info notification 'number.info.scientific' when user enters valid value
* [x] - Remove `toScientificNotation` function, which is incorrectly passed the TranslationFunction `t` anyway.
* [x] - Remove `IsValidWithMessage` type and replace return value with just a boolean of whether its valid or not
* [x] - Move `isValidNumber` into React componet, and do not pass the TranslationFunctio to it!
* [x] - Move `isValidKey` into React componet, and do not pass the TranslationFunctio to it!
* [x] - Move `isNonDecimal` into React componet, and do not pass the TranslationFunctio to it!
* [X] - Remove `isNonInteger` and isNonInteger.ts and isNonInteger.spec.js since no longer using
* [X] - Remove `error`, `warn`, `info` from InputNumber state, since we're filtering out everything not valid. Instead add them as optional props (so Balance or another component can display notifications if required)
* [X] - Move tests for isValidKey into InputNumber React component, and remove isValidKey.spec.js
* [X] - Move tests for isValidNumber into InputNumberReact component, and remove isValidNumber.spec.js
* [X] - Move tests for isNonDecimal into InputNumberReact component, and remove isNonDecimal.spec.js
* [X] - Update InputNumber.spec.js to remove useless tests that check that the Notification component isn't shown
* [X] - Make methods `private` as relevant
* [X] - move chainSpec and keyboard subfolder files into InputNumber. Move contents of constants into InputNumber except for BIT_LENGTH_128
* [X] - Move all the different files that are only used in the number input into functions inside that input. Then we also get rid of calling external functions (like isNumberValid) with t as a param. Keep it small, keep it together.
* [X] - Remove `cursorIndexInputField` function since we're not trying to check the current cursor position in the input field anymore
* [X] - Assume that based on @jacogr's comment: "I'm struggling to see where we are actually now doing errors in the Balance input - it filters everything non-valid.".
* Action: Change isValid from being true, to being assigned to the value of !isError
* Note: DO NOT update the Extrinsics component to use the 'fees' observable (as it's used in the Transfer App) so the Balance input field is invalid (appears "red" and Submit Transaction button disabled) when the value entered plus fees exceeds the sender's balance. Reasoning: We cannot determine the size of the tx in there at all - there is a FIXME for the hardcoding of the size (already wrong for poc-3), in the Transfer App. So without the final signed size, it currently cannot be done. `fee = baseTxFee + (tx-size-in-bytes * feePerByte)`. We cannot get the tx-size-in-bytes atm. (hardcoded to 156 in Transfer, which assumes only publicKeys and is not the case when we get to poc-3). There is an issue logged that goes through all the fees. Signer needs to have a method exposed that takes all the params as given, encodes the extrinsic and returns the total fee or size. So nothing stopping us, it just has not been done properly. (Signer does not even know in the modal what the final size would be - that is the first stop, showing the fees in there. Then the apps, then maybe extrinsics if it still has a use)
* [X] - "Seems SUI docs are currently broken. But in the cases where you absolutely need to point to an input with an attachment, their Label does exactly that and allows attachment with a pointer to an Input. So at most Notification should just warp that. And it really should only be used is completely exceptionally circumstances. (And I really couldn't see any such circumstances from playing with the actual inputs. So at this point seems like a good, although unused in practice idea.)"
* [X] - Remove Notification component, and all the error, warn, and info props, as we won't be showing notifications
* [ ] - Integrate into the Transfer App `send a value of` where actual transfers will be done (see @jacogr comment and screenshot). Incorporate the Balance/InputNumber into the Transfer App, which currently just uses an <Input> field and isError={!hasAvailable} to determine highlight the input field "red" and disable the Submit button when they don't have sufficient balance.
* ACTION: Unable to get this to work. If I change the Input to Balance in Transfer.tsx to the following, then when I modify the value in the Transfer tabs "send a value of" input field, it shows an `Error: Assertion failed` in the console
```
{/* <Input
defaultValue='0'
isError={!hasAvailable}
label={t('amount', {
defaultValue: 'send a value of'
})}
min={0}
onChange={this.onChangeAmount}
type='number'
/> */}
<Balance
defaultValue={{ value: '0' }}
isError={!hasAvailable}
label={t('transfer.amount', {
defaultValue: 'send a value of'
})}
onChange={this.onChangeAmount}
/>
```
Error mentioned above:
```
InputNumber.tsx?299c:129 Error: Assertion failed
at assert (bn.js?cda7:6)
at BN._initArray (bn.js?cda7:145)
at BN.init [as _init] (bn.js?cda7:82)
at new BN (bn.js?cda7:39)
at Transfer.onChangeAmount.amount (Transfer.tsx?f63a:45)
at Balance.onChange.value (Balance.tsx?8315:42)
at InputNumber.onChange.value (InputNumber.tsx?299c:127)
at Object.Input.onChange.event [as onChange] (Input.tsx?a939:36)
at apply (_apply.js?59ad:15)
at baseInvoke (_baseInvoke.js?199b:21)
```
* fix: Move BIT_LENGTH_128 into InputNumber
* refactor: Remove disused keys from constant. Remove useless isShift fn. Use Spread operator.
* refactor: Simplify comments
* refactor: remove useless maxSafeIntegerBN
* refactor: remove useless check for isZero
* refactor: Simplify isNonDecimal function
* refactor: remove maxValue from being maxLength arg. remove useless defaultMaxLength fn
* fix: Use InputNumber in Transfer instead of Balance
* refactor: move non-number specific keyboard functions to Input instead of InputNumber
* refactor: Reorder alphabetically exports
* refactor: Remove disused withApi and translate from Balance and simplify defaultValue
* refactor: spread operator to pass down props before overrides if any
* test: Add more isValidNumber tests. Add maxValue and isValidBitLength tests
* fix: fix maxValue function
* test: Add maxLength test for 128 bit
* refactor: remove unnecessary line in test
* fix: Remove support for decimals since BN.js only supports integers
* refactor: optimise by using isNaN instead of isNonInteger for single character checks
* fix: revert using spread operator to pass down props
* fix: Add bitLength prop to InputNumber. Move maxConservativeLength function. See detailed comments
* bitLength passed down as prop from Transfer and Balance to InputNumber is 128 bit for latest chain specification
* Move maxLength method out of InputNumber class and rename maxConservativeLength also to avoid name conflict with the maxLength prop
* Change argument of maxConservativeLength to the maxValueLength to simplify
* Update InputNumber tests
* Add default bit length for InputNumber is 32 bit for normal numbers
* fix: Add BitLength type and BitLengthOption enum
* fix: prevent user entering value preceded by 0
* refactor: Replace ternary with more readable
* feat: Add 8 and 16 to BitLength for future-proofing and consistency
* Cleanup app-accounts, align with addresses
* Remove 'restore' action (copy gone wrong)
* Bump dependencies
* Bump deps (again, api was not included properly...)
* Fix Address add flicker
* Clenaup unused
* use getDerivedState in Tabs
* Mark functions private
* Creator autoFocus on Input
* No extra {} on proptype
* Add validators App
Fix#191
* Pull validator breakidown into staking overview
* Fix layout
* Add AddressRow component
* Minor CSS fix
* Use standard address components in the staking app
* Use new components and cleanup
* Refactor and layout work on the validators
* Re-use already available components
* Remove app-validators in tsconfig
* Render nominators
* Use row defaults applicable
* Small css fixes
* Add validators App
Fix#191
* Pull validator breakidown into staking overview
* Fix layout
* Add AddressRow component
* Minor CSS fix
* Use standard address components in the staking app
* Use new components and cleanup
* Refactor and layout work on the validators
* Re-use already available components
* Remove app-validators in tsconfig
* WIP: Fixes#260. Autofocus password input field. Press tab after enter password to switch focus to submit
* feat: Clicks Submit Transaction when user presses Enter after entering password
* fix: Remove console logs and no need to focus
* test: Add test setup. setState not changing currentItem from undefined
* fix: Change tabIndex values to number instead of string
* fix: Use Refs instead of JS DOM access
* fix: Add missing newline
* test: Test modal show after setState. Skipped pending resolution of Enzyme issue
* fix: Remove string from tabIndex prop type
* fix: Remove use of SUIEvent. Use React refs not HTML DOM queries
* test: Update tests but skipping since awaiting resolution of enzyme issue
Reference: https://github.com/airbnb/enzyme/issues/1794
* WIP
* fix: Simply call onSend instead of long-winded approach with Refs
* test: Fix tests to use promises instead of callbacks
* fix: Fix tslint error to handle promises correctly
* fix: Remove return since broke functionality. Use await to handle promise. Use key instead of keyCode
* fix: Part fix#264. Trim address. Add CopyButton. Align CopyButton with monospace
* fix: Remove CopyButton
* fix: Add empty div so double clicking to copy address only selects the address itself
* fix: Add key prop to div to avoid error
* fix: Add PostCSS Mixins. Reuse font family for hashes. Add missing license header.
* Add PostCSS Mixins dependency to allow reuse to make CSS DRYer
* Create generic mixin for common font family used by different hashes displayed in UI (i.e. block, data, address)
* Create extension of the generic mixin for the block hash
* Add missing license header
* fix: Rename mixin to hash font traits
* fix: Remove hash-block-font-family since we are only using it once
* fix: Revert Mixins and variable
* added account meta to forget modal
* added modal size (tiny) and styling to center account summary info
* removed article - not needed
* Bump dependencies
* FIXME around RxJs (with types) upgrade
* fix: Fixes#290. Toggles Address Add/Edit menu correctly (#292)
* fix: Fixes#290. Toggles Address Add/Edit menu correctly
* fix: Rename to hasNoAddresses
* fixed modal title and added blank line
* WIP: Not sure if this is proper use of Link / Button
* remove console log
* remove space
* trying to create a forgetting modal
* first commit with new forgetting file - unsure this is right
* hopefully fixed the incorrect use of translate?
* fixed other requested changes with translate
* WIP: Not sure if this is proper use of Link / Button
* remove console log
* remove space
* moved logic to index and use accountsObservable
* fixed travis recs
* added translate import and t(), changed button to simple link, updated div to main
* use 4e4e4e-d75ea1-ffffff-89acb4-98aaae colors
* tone down lateness & anchor, use variables
* use descriptive variable names and dark blue color for anchors
* remove unused color
* DRY Card display (CardBar component, consistency)
* Use SummaryCard on staking
* Simplify summaries (no duplication from explorer)
* Display lateness percentage value
* Basic democacry (Referendum) overview
* Summary in-place
* Fix package name
* Proposals & Referendums are showing up
* Some basic Extrinsic formatting. (WIP)
* Basic referendum/proposal summary in place
* Referendum outcome chart
* Voting in place. It works.
* Remove extra console.log/error
* Only show balances as label
* Move observables to ui-react-rx
* Move observables into ApiObservable
* Allow AddressSummary to render passed-in balance(s)
* Render Balance consistently in summaries
* Style Mini balance
* Use ApiObservable via HOC
* Nominee balances expanded
* Render nominators for account
* Add full balance when intending
* Format stakingBalance as votingBalance + nominatingBalance
* Fix version number change (revert)
* Bump api 0.15.x: rxjs 6 & class-ified providers
* API bump
* Type cleanup
* Update version of packages
* RxJs 6 style Observable imports
* Remove ZERO const
* Move AddressSummary component into ui-app
* Add children rendering to summary
* User AddressSummary in staking
* Mini summary component
* Show nominating in modal
* Check nomination against intentions (not validators)
* Update input description
* refactor: Rename InputStorage key to method to be consistent with InputExtrinsic
* fix: Add filter to both section and method that considers isHidden prop
* refactor: Move into meaningful private functions
* fix: Fix linting errors
* test: Add Unit Tests to check shouldDisplaySection function
* fix: Remove unnecessary import
* restore file naming
* refactor: Move shouldDisplaySection into utils folder
* refactor: Move more common functionality into util
* remove unused isObject
* restore by removing unnecessary types. propose private type for storage
* fix: Remove explicit passing through the type for storage since its default
* feat: Use shared logic for showing section and methods in InputRpc
* refactor: Move shared types into util folder with section visibility type
* fix: type is public or private
* WIP: methods[name] TypeScript error
Element implicitly has an 'any' type because type '{}' has no index signature
* fix: Fix TSLint errors using jacos solution
* fix: Detect chain specs that enable test environment and dev accounts Issue #130
* test: refactor moving chain checker into meaningful function name
* fix: Detect check specs requiring backward compatibility encoding for Issue #130
* remove comment
* remove comment and restore source maps
* fix: Fix the license information in test files
* refactor: Move cases into array with meaningful array. Cater for numbers
* refactor: Reduce how condensed the code is
* refactor: Use Polkadot isUndefined helper
* refactor: Move RegEx instantiation outside function so not called every time
* refactor: Move new utility functions and associated tests into ui-react-rx/src/util
* fix: Update to cater for renaming of poc-2 to staging
See commit updates since `poc-2` has been renamed to `staging` 3426552c76
* fix: Fixes#88 buttonbar style cornering on Accounts and Addresses pages
- Add right corners to Forget
- Add left corners to Reset
* refactor: Move overrides for buttonbar corner styles into shared resource
* fix: Add newline to end of file
* refactor: Refactor using PostCSS variables, nesting, and import
* refactor: Move variables into partials subfolder. Apply CSS when preceding sibling is spacer
* fix: Remove unnecessary comma to pass CI linting
* fix: Add initial divider with no padding to apply corners conditionally
There is no CSS selector combinator that allows conditional styling of an element conditional upon "subsequent" sibling elements existing. This is a work-around whereby a dummy divider with no padding is added to the start of the button group such that the custom corner overrides only apply to button groups that contain a 1st and 3rd element that are dividers. Only options for conditional styling dependent on "preceding" sibling elements currently exist that are compatible across browsers (i.e. 'Adjacent sibling combinator' by use of `+` and 'General sibling combinator' by use of `~`, see below reference for details).
Another alternative solution that was considered by not deemed generic enough was to conditionally apply the custom cornering only to button groups that are contained within a classname that contains the text "--Editor", on the assumption that the Forget, Reset, Save button group would only be used within a nesting containing such a wildcard value (as is currently the case)
```
div [class*="--Editor"] {
.ui--Button-Group {
.ui.buttons {
.button:nth-child(1) {
border-top-right-radius: $small-corner;
border-bottom-right-radius: $small-corner;
}
.ui.button.compact.mini.basic + .button:nth-child(3) {
border-top-left-radius: $small-corner;
border-bottom-left-radius: $small-corner;
}
}
}
}
```
Reference: https://developer.mozilla.org/en-US/docs/Web/CSS/CSS_Selectors
* fix: Resolve linting CI errors
* fix: Add missing dependencies to package.json to pass CI
* refactor: Move widthless divider into base Group component
* refactor: Move button group custom cornering styles
* feat: Fixes#75. Add edit icon from semantic-ui to account name input field
Adding an edit icon that is embedded in the account name input field makes it more obvious to the user that the input field isn't read-only, and that there is a way to "click before doing" (i.e. editing the input field that is identified as being editable before the ability to save changes is granted). Some users may click the edit icon itself thinking it toggles the ability to edit the input field, whilst others may just recognise that the edit icon means the input field is editable. In either case just having the edit icon should suffice as being the initial "call to action" for a user wishing to edit the input field, and since the Reset and Save buttons illuminate when the account name is modified, they'll recognise that a further step is required to save their changes.
* resolve introduced ESLint and Flow errors
* fix: Move attributes up so they are in alphabetical order
* fix: Remove unnecessary classenames. Edit icon stays within input field
* Be explicit in State returns
* Cleanup Param properties
* Cleanup casting
* Cleanup type manipulations
* Don't allow null for InputAddress defaultValue
* Convert to @polkadot/param formatters
* WIP
* Renderinbg operational
* Update with checks
* Show 'empty' with no array values
* Pass-through label properly
* Pass through isDisabled property
* Pull in ui-app components only
* Undefined check
* Small overall layout updates (buttons)
* Password hide/unhide icons
* Automatically assign name to inputs
* Swap to using split-out ui-keyring
* First part of split into ui-keyring complete
* Create/Edit account operational
* Cleanup keyring display
* InputAddress defaultValue
* Updates
* s/app-testing/app-extrinsics/
* Run components as apps using url
* Redirect to /home
* Expand Portal with NotFound app component
* New block numbers & hashes are being shown
* Only display best block once
* best block -> best
* TODO for calcHash
* Explorer aligns with blockNumber/hash
* Break signing into seperate modal
* Cleanups with @flow
* Adjust Dropdown for methods
* Remove (for now) unused app-home
* Update deps
