Skip to content

45 profile portrait responsive#71

Merged
khoadnguyen merged 4 commits intodevfrom
45-profile-portrait-responsive
Oct 9, 2018
Merged

45 profile portrait responsive#71
khoadnguyen merged 4 commits intodevfrom
45-profile-portrait-responsive

Conversation

@iamalx
Copy link
Copy Markdown
Collaborator

@iamalx iamalx commented Oct 3, 2018

#45 Made profile page responsive: landscape and portrait mode for ipad; and for ipad to cell phone. Ionic components are hard to manage and work with. Spend a lot of time fixing bugs because of ionic component's predefined CSS properties.

iamalx added 2 commits October 3, 2018 01:30
… to work with for table, so changed item content and placed display: flex for buttons instead of grid
@iamalx iamalx added the code review This PR is ready for code review label Oct 3, 2018
@iamalx iamalx added this to the Week 4 milestone Oct 3, 2018
@iamalx iamalx self-assigned this Oct 3, 2018
<ion-input placeholder="Peter Horton" text-center></ion-input>
</ion-item>

<ion-label class= "label"color="dark">Name</ion-label>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@awstin17 did some work with fontawesome icons, perhaps we could use that styling here as well?

Copy link
Copy Markdown
Contributor

@khoadnguyen khoadnguyen left a comment

Choose a reason for hiding this comment

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

Nice job. 🌊

margin: 10px 10px auto 10px;
}

@media screen and (min-width: 768px) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Your styling formatting and media query layout is great, can you discuss your methodology in class during CR?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

sure!

@khoadnguyen khoadnguyen merged commit 1111a15 into dev Oct 9, 2018
@khoadnguyen khoadnguyen deleted the 45-profile-portrait-responsive branch October 31, 2018 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code review This PR is ready for code review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants