-
Notifications
You must be signed in to change notification settings - Fork 983
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
[FIX #3123] ui: fix network field overflow on long network names in profile #3345
[FIX #3123] ui: fix network field overflow on long network names in profile #3345
Conversation
Hey @ikysylevych, thanks for making your first pull request in status-react! ❤️ |
Hi @ikysylevych - thanks for the contribution! |
@ikysylevych If you feel like it you can extract this as a component ! We certainly have this need for others texts. |
@jeluard I'm still learning the project structure, don't want to break things. But I hope I could extract it to a separate component soon. |
Automated test results:test_send_eth_from_wallet_sign_later:x:Test Steps & Error message:
test_browse_link_entering_url_in_dapp_view:white_check_mark::Test Steps & Error message:
test_network_switch:white_check_mark::Test Steps & Error message:
test_qr_code_and_its_value:x:Test Steps & Error message:
test_transaction_send_command_wrong_password:white_check_mark::Test Steps & Error message:
test_send_transaction_from_daap:white_check_mark::Test Steps & Error message:
test_send_stt_from_wallet_via_enter_contact_code:white_check_mark::Test Steps & Error message:
test_transaction_send_command_one_to_one_chat:white_check_mark::Test Steps & Error message:
test_transaction_send_command_group_chat:x:Test Steps & Error message:
test_send_eth_to_request_in_one_to_one_chat:white_check_mark::Test Steps & Error message:
test_send_eth_to_request_in_group_chat:x:Test Steps & Error message:
test_send_eth_to_request_from_wallet:white_check_mark::Test Steps & Error message:
|
(when-not (string/blank? value) | ||
[react/text {:style styles/settings-item-value | ||
:number-of-lines 1 | ||
:uppercase? component.styles/uppercase?} value]) |
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.
can you put value on the next line please ?
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.
it's done. thanks.
2a8ad8d
to
d604209
Compare
Branch: PR-3345 |
@ikysylevych Could you squash your commits? |
d604209
to
2f7e1b0
Compare
@jeluard I squashed my commits. Thanks! |
…ames in profile settings Signed-off-by: Julien Eluard <julien.eluard@gmail.com>
2f7e1b0
to
5f5412d
Compare
@ikysylevych Thanks! |
fixes #3123
Summary
Introduced proportions between field title and value and ellipsis endings on text overflow.
Note: In case of longer network names, this bug may be probably reproducible even with latest ios/android devices.
status: ready