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

chore: responsive layout modification of cards sections #3212

Merged
merged 5 commits into from
Jun 4, 2020

Conversation

freefaller69
Copy link
Contributor

Description

Rework responsive behavior in cards sections for frameworks, features, and community.
Refactor inner border implementation in frameworks section.
Remove media queries that are no longer needed.

Screen Shot 2020-06-01 at 3 29 54 PM

Screen Shot 2020-06-01 at 3 31 00 PM

Screen Shot 2020-06-01 at 3 32 40 PM

Screen Shot 2020-06-01 at 3 30 06 PM

Screen Shot 2020-06-01 at 3 31 45 PM

Screen Shot 2020-06-01 at 3 32 49 PM

Screen Shot 2020-06-01 at 3 30 25 PM

Screen Shot 2020-06-01 at 3 31 14 PM

Screen Shot 2020-06-01 at 3 32 56 PM

Motivation & context

Issue type checklist

  • Chore: A change that does not impact distributed packages.
  • Bug fix: A change that fixes an issue, link to the issue above.
  • New feature: A change that adds functionality.

Is this a breaking change?

  • This change causes current functionality to break.

Process & policy checklist

  • I have added tests for my changes.
  • I have tested my changes.
  • I have updated the project documentation to reflect my changes.
  • I have read the CONTRIBUTING documentation and followed the standards for this project.

@SamanthaAO
Copy link
Contributor

I am noticing that feature is still overflowing from its row in the wrapper starting around 1330px screen width.
Screen Shot 2020-06-02 at 12 59 31 PM

@SamanthaAO
Copy link
Contributor

The comment above has been discussed and will be addressed in an upcoming PR, so that the other styling reformat changes can go in.

@freefaller69 freefaller69 force-pushed the users/v-mbenso/responsive-cards branch from bd0fdce to fbccd22 Compare June 3, 2020 21:09
@freefaller69 freefaller69 force-pushed the users/v-mbenso/responsive-cards branch from 6fee71c to a59c475 Compare June 4, 2020 16:59
@freefaller69 freefaller69 merged commit b28a5cf into feature/fast-frame Jun 4, 2020
@freefaller69 freefaller69 deleted the users/v-mbenso/responsive-cards branch June 4, 2020 17:41
chrisdholt pushed a commit that referenced this pull request Jun 4, 2020
* framework section inner border and responsiveness

* refactor community card section responsiveness

* remove stray tag, condense styling

* fix feature section hover behavior

* Adjust grid properties on feature-cards

Co-authored-by: Michael Benson <[email protected]>
chrisdholt pushed a commit that referenced this pull request Jun 11, 2020
* framework section inner border and responsiveness

* refactor community card section responsiveness

* remove stray tag, condense styling

* fix feature section hover behavior

* Adjust grid properties on feature-cards

Co-authored-by: Michael Benson <[email protected]>
chrisdholt pushed a commit that referenced this pull request Jun 16, 2020
* framework section inner border and responsiveness

* refactor community card section responsiveness

* remove stray tag, condense styling

* fix feature section hover behavior

* Adjust grid properties on feature-cards

Co-authored-by: Michael Benson <[email protected]>
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.

6 participants