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

enhace description of multisig label #294

Merged
merged 2 commits into from Jan 31, 2018

Conversation

Projects
None yet
2 participants
@farmerwampum
Contributor

farmerwampum commented Jan 30, 2018

On small screen I was confused what "msig" said. Thought it was "message"
Should be plenty of room

@prusnak

This comment has been minimized.

Member

prusnak commented Jan 31, 2018

Yes, this was shortened prior we reworked the layout. There is more space now, so we can apply this change.

However, change of the string is not enough. You need to increase the capacity of desc from 16 to 20 to accomodate for 4 extra chars and also change the assignments below from desc[5], desc[6], desc[11], desc[12] to desc[9], desc[10], desc[15], desc[16], because of the extra 4 chars you added to the beginning of the string.

@prusnak prusnak merged commit 2391beb into trezor:master Jan 31, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

zulucrypto added a commit to zulucrypto/trezor-mcu that referenced this pull request Mar 19, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment