Skip to content

Commit

Permalink
feat: add break-all to tx reason field (#340)
Browse files Browse the repository at this point in the history
# What ❔

This PR adds a simple break-all to the reason field of the Transactions
view, so that when the transaction returns the error, it doesn't mess up
the UI - the error text just breaks into next lines.

## Why ❔

When the transaction fails, if the error text is too long, it will make
the whole tx UI scrollable for no reason, so we just add break-all to
fix the UI.
This is a fix for this issue:
#329
Example of a failed transaction and error text that makes the table
scrollable:
https://sepolia.explorer.zksync.io/tx/0xe8d8ec3d36bc6cbbcf3453fa855fb64b56c547411000a631fd41a6c622ce1d79#overview

## Checklist

<!-- Check your PR fulfills the following items. -->
<!-- For draft PRs check the boxes as you complete them. -->

- [x] PR title corresponds to the body of PR (we generate changelog
entries from PRs).
- [x] Tests for the changes have been added / updated.
- [x] Documentation comments have been added / updated.
  • Loading branch information
tx-nikola authored Dec 4, 2024
1 parent 198b1ff commit 9317a70
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -324,7 +324,7 @@ const gasUsedPercent = computed(() => {
@apply py-2;
}
.transaction-reason-value {
@apply text-error-600 whitespace-normal;
@apply text-error-600 whitespace-normal break-all;
}
}
</style>

0 comments on commit 9317a70

Please sign in to comment.