Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Create Aspect Ratio Mixin #445

Merged
merged 20 commits into from
Oct 16, 2019
Merged

Conversation

rebeccahongsf
Copy link
Contributor

@rebeccahongsf rebeccahongsf commented Jun 12, 2019

NOT READY

To Do:

  • Create Aspect Ratio Mixin Code
  • Create Aspect Ratio Mixin Documentation

Summary

Needed By (Date)

N/A

Urgency

N/A

Steps to Test

  1. Pull this branch
  2. Compile style guide
  3. Review documentation in the example section (Section 6 Mixins)

Affected Projects or Products

  • Does this PR impact any particular projects, products, or modules?

Associated Issues and/or People

See Also

@rebeccahongsf rebeccahongsf self-assigned this Jun 12, 2019
@buttonwillowsix
Copy link

@rebeccahongsf I will take this over for now. :)

@buttonwillowsix buttonwillowsix self-assigned this Jun 14, 2019
@yvonnetangsu
Copy link
Member

@rebeccahongsf @sherakama Hey there - just want to check in and see what's the status of this PR 😄 We could use this mixin for the media field component (will be used in cards). Let me know if I could help with reviewing etc.

@sherakama
Copy link
Member

@yvonnetangsu

If you need this in your work feel free to merge it into your working branch.

@sherakama
Copy link
Member

@yvonnetangsu @rebeccahongsf

I took a stab at cleaning up the branch and merging it with the latest work from the master branch. I also moved a couple of things around and deleted the duplicate example template.

Please let me know if this makes sense as I wasn't 100% sure what the desired outcome was supposed to be.

@yvonnetangsu
Copy link
Member

yvonnetangsu commented Oct 9, 2019

@rebeccahongsf @sherakama - I did some touch up and added 3 use cases in the example section (Core -> Helpers -> Aspect Ratio Container). Please let me know what you think.

@sherakama sherakama merged commit 8b8215b into master Oct 16, 2019
@sherakama sherakama deleted the 310-aspect-ratio-container-mixins branch October 16, 2019 18:43
@sherakama
Copy link
Member

Thanks! Looks great.

yvonnetangsu added a commit that referenced this pull request Oct 16, 2019
* master:
  Update _hero.scss (#513)
  Create Aspect Ratio Mixin (#445)

# Conflicts (resolved manually)
#	kss/builder/decanter/kss-assets/css/kss.css
#	kss/builder/decanter/scss/_aspect-ratio.scss
yvonnetangsu added a commit that referenced this pull request Oct 29, 2019
* master: (28 commits)
  Updated packages and configuration files. (#527)
  Use rem instead of em for big button font size so it doesn't blow up in sections if we set a large base font size; finetune big button padding for smaller breakpoints (#522)
  Change sizes of link icons to use px instead of em to prevent occasional clipping (#523)
  D8CORE-628: Add caption option to hero banner. (#517)
  Use percentage for html root font size instead of px so people can scale up using browser font size preferences (#526)
  Mixins documention update — @centered-column (#448)
  Adds Tugboat and fixes npm dependencies. (#518)
  Update _hero.scss (#513)
  Create Aspect Ratio Mixin (#445)
  Adding global footer variants and link tweaks for legibility, change layout at MD breakpoint to match Figma (#500)
  Updated package and lock files.
  Add parameter to @link-icon to adjust vertical position of icon and different options for $animate parameter (#504)
  Delete card-2019-08-28.twig (#506)
  Update _input.scss (#507)
  Scale down type-a and type-b font size at xs and sm breakpoints, repurpose splash font and minor typography tweaks (#498)
  Add 4 new @modular-spacing steps (#6 to #9) and modify 2 existing steps (#3 and #4) (#503)
  add su- prefix to Decanter core and component variables, deprecate old names (#452)
  Includevar (#496)
  Put appearance back for select dropdowns (#479)
  Split input.scss into several files and add new input-base placeholder (#476)
  ...

# Conflicts (resolved manually)
#	core/dist/css/decanter.css
#	core/src/scss/components/card/_card.scss
#	core/src/scss/components/card/index.scss
#	core/src/scss/utilities/placeholders/components/_card.scss
#	core/src/templates/components/card/card.twig
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants