use-ink / contracts-ui

Web application for deploying wasm smart contracts on Substrate chains that include the FRAME contracts pallet
https://contracts-ui.substrate.io/
GNU General Public License v3.0
61 stars 45 forks source link

Improve tooltip descriptions for `RefTime` and `ProofSize` #422

Closed cmichi closed 1 year ago

cmichi commented 1 year ago

We have info-tooltips here:

screenshot-contracts-ui substrate io-2023 03 03-19_47_17

The "RefTime Limit" tooltip currently says:

The maximum amount of gas to use for execution (the old gas limit). If the transaction requires more, it will fail.

The issue here is that the user typically won't know what "gas" means here, and also not what "old gas" means.

For "ProofSize Limit":

The maximum amount of gas to use for PoV. If the transaction requires more, it will fail.

The issue here is that the user typically won't know what PoV refers to, I would formulate it like below.

I suggest for "RefTime Limit"

The amount of computational time used for execution. If the transaction requires more, it will fail.

For the "ProofSize Limit" tooltip I suggest:

The amount of storage space used, in bytes. If the transaction requires more, it will fail.

I would avoid introducing the term "weight" here and use the more general "amount".

statictype commented 1 year ago

Sounds good @cmichi! I suggest keeping "maximum" around since we are talking about the limits not the values themselves.

The maximum amount of computational time that can be used for execution, in picoseconds The maximum amount of storage space that can be used, in bytes.