Skip to content
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: signing button postion #563

Merged
merged 18 commits into from
Mar 23, 2020
Merged

fix: signing button postion #563

merged 18 commits into from
Mar 23, 2020

Conversation

hanwencheng
Copy link
Contributor

This PR based on #562 .

Improve on the signing button. and also fix the buttonn display on big screens (e.g. iPhone X).

Signing button

before fix the "sign transaction" button is floating with fix position, which will cover the transaction information. now it is on the end of the screen.

before fix after fix
Screenshot_20200317_183615_io parity signer Screenshot_20200317_183711_io parity signer

Signing Message button in big screen

The "signing message" button do not have fix position, but it extends to the whole screen width when displayed in big screen, now change to fit screen.

before fix after fix
Screenshot_20200317_183401_io parity signer Screenshot_20200317_183431_io parity signer

sjeohp-zz
sjeohp-zz previously approved these changes Mar 19, 2020
@hanwencheng hanwencheng dismissed sjeohp-zz’s stale review March 19, 2020 17:53

The base branch was changed.

@hanwencheng hanwencheng changed the base branch from hanwen-navigation-v5 to master March 19, 2020 17:53
@hanwencheng hanwencheng merged commit 1afd936 into master Mar 23, 2020
@hanwencheng hanwencheng deleted the hanwen-signing-button branch March 23, 2020 10:36
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