-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
allow hiding marketcap percentage #2818
Conversation
Pull Request Test Coverage Report for Build 600fcbec-7c3c-43d2-a770-5db8a2308340
💛 - Coveralls |
should be merged after blockscout/blockscout#2818
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only a comment
apps/block_scout_web/lib/block_scout_web/templates/address/index.html.eex
Outdated
Show resolved
Hide resolved
#, elixir-format | ||
#: lib/block_scout_web/templates/address/_tabs.html.eex:32 | ||
#: lib/block_scout_web/templates/address/overview.html.eex:97 | ||
#: lib/block_scout_web/templates/address_validation/index.html.eex:13 | ||
#: lib/block_scout_web/views/address_view.ex:313 | ||
msgid "Blocks Validated" | ||
msgstr "" | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ayrat555 what is the rationale to change the placement of these strings in the internalization file? Please let's leave them as is. Otherwise, there will be merging conflicts.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@vbaranov to fix merging conflicts. Let me try to change only line numbers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ayrat555 in case of disabled % of market cap, the header of the table doesn't fill the full width of the container
@vbaranov I addressed your comment |
Motivation
resolves #2567
Changelog