Skip to content

Update explorer.htm & CodeQL workflow#1964

Merged
valdok merged 3 commits intoBeamMW:masterfrom
Maxnflaxl:master
Mar 26, 2024
Merged

Update explorer.htm & CodeQL workflow#1964
valdok merged 3 commits intoBeamMW:masterfrom
Maxnflaxl:master

Conversation

@Maxnflaxl
Copy link
Member

@Maxnflaxl Maxnflaxl commented Mar 17, 2024

@Maxnflaxl Maxnflaxl requested a review from valdok March 17, 2024 13:57
@Maxnflaxl Maxnflaxl changed the title Update explorer.htm Update explorer.htm & CodeQL workflow Mar 17, 2024
@valdok
Copy link
Member

valdok commented Mar 19, 2024

This is very nice, I like it.

But I have a couple of suggestions.

  1. In the contract state window, the "call history". In case a contract call invokes calling other contract (such as Nephrite method call sometimes invokes calling Oracle and DaoVault contract) - all this should be shown as a group call, separated from others.

In previous version there's a slightly more bold margin that separates those groups, and the Height is shown only for top-level call. In this version there'e no visible margin, and the Height field is at every row. Which is more confusing IMHO.

Ideally it'd be great to somehow put a bold visible separator between such groups. Also that'd be cool to "dim" other contracts invoked (i.e. maybe use smaller fonts), emphasize this specific contract being-viewed.

  1. In a contract list view if a contract holds or owns many different asset types - they're placed under "+x assets", which is very handy.
    Is it possible to give a user control of their expansion? I mean, a button at the top of the page, where user select to either expand, collapse, or semi-expand (i.e. expand unless too many items) those lists?

@dbadol
Copy link
Contributor

dbadol commented Mar 19, 2024

@valdok
Thanks for your comments! All very valid. I will work on that and submit an updated version.

The only reason I removed the rowspan display for multiple contract calls is that it was making table sorting and filtering much more complicated. I thus replaced it with a simple height repetition (not ideal, but quick and simple). Now, your comment makes me think that we can still make the result much more readable just by adjusting some display parameters. I will do it.

For the asset lists, the list is already automatically collapsed if there is more than 4 assets. But yes, it would be handy to have a general control somewhere. I will add it.

@valdok
Copy link
Member

valdok commented Mar 21, 2024

@dbadol
More on this topic. Basically in those tables there's more info that can be shown.
For example the call history of a contract, we currently show how much funds a contract locks/unlocks, and how much assets it emits/burns.

In addition to this we could show how much excess funds are put in/out of the current tx. In simple case where only 1 contract is invoked and no assets are emitted/burned that would match the numbers that contract locks/unlocks.
But it could be more complex.

Another example: in blocks history, there're many more details can be shown. Such as current cumulative values (total number of UTXOs, delta UTXOs, and etc.)

The reason we didn't include all possible values in the table is the lack of screen space.
But if we could give the user ability to show/hide all the columns - that would be very beneficial.
Ideally for each such a table there should be a set of checkboxes with column names, each could be checked/unchecked by the user, and the user preferences could be stored locally.

@valdok valdok merged commit 036c60b into BeamMW:master Mar 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants