Skip to content
This repository has been archived by the owner on Apr 15, 2019. It is now read-only.

Fix responsiveness of the layout - Closes #529 #617

Merged
merged 8 commits into from
Aug 21, 2017

Conversation

slaweet
Copy link
Contributor

@slaweet slaweet commented Aug 18, 2017

Closes #529

@slaweet slaweet self-assigned this Aug 18, 2017
@slaweet slaweet requested a review from reyraa August 18, 2017 13:22
@slaweet slaweet added this to Pending Review in Version 1.1.0 Aug 18, 2017
@reyraa
Copy link
Contributor

reyraa commented Aug 21, 2017

Everything is all fine. thank you. there's just two small issue you may want to fix.
I think there's enough space for menu to stay in the right side of logo:
screen shot 2017-08-21 at 10 18 12

I think transaction ID is taking more space and is less informative than the others. if we hide it in this media query, we'll have a better user experience:
screen shot 2017-08-21 at 10 18 39

Copy link
Contributor

@reyraa reyraa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you Vit.

@slaweet slaweet merged commit 593f7bf into development Aug 21, 2017
@slaweet slaweet deleted the 529-fix-responsive-layout branch August 21, 2017 15:36
@slaweet slaweet moved this from Pending Review to Done in Version 1.1.0 Aug 21, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
No open projects
Version 1.1.0
Merged Pull Requests
Development

Successfully merging this pull request may close these issues.

None yet

2 participants