[aspect-ratio] Latest aspect ratio broke aspect-ratio classes
See original GitHub issuehttps://github.com/carbon-design-system/carbon/pull/5255/files
The bx--aspect-ratio-1x1
class no longer does anything.
reproduction:
Go here: https://carbon-website-hxvmb5n2c.now.sh/data-visualization/chart-types
Add the css rule:
.bx--aspect-ratio--1x1 {
padding-bottom: 100%;
}
This is the rule that was being applied before we updated Carbon.
Issue Analytics
- State:
- Created 3 years ago
- Comments:5 (5 by maintainers)
Top Results From Across the Web
aspect-ratio - CSS-Tricks
When content breaks out of the ratio. Simply put, if you have an element with an aspect ratio and the content is so...
Read more >aspect-ratio - CSS: Cascading Style Sheets - MDN Web Docs
The aspect-ratio CSS property sets a preferred aspect ratio for the box, which will be used in the calculation of auto sizes and...
Read more >Aspect Ratio - Tailwind CSS
Utilities for controlling the aspect ratio of an element. ... Tailwind lets you conditionally apply utility classes in different states using variant ...
Read more >aspect-ratio - QuirksBlog - QuirksMode
The script that runs in this page changes the value of --aspectRatio . The trick here is that percentual padding is calculated relative...
Read more >The Complete Guide to Understanding Video Aspect Ratios
Today, we are going to break down what exactly video aspect ratios are, commonly used ratios, and video resolutions, as well as the...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
@joshblack I’m still not sure why the aspect ratio worked until recently, but definitely agree the base class should have been there from the start.
@vpicone to clarify, this was always the intended usage of these classes going back to the first implementation of this package. It is not a new requirement or sudden dependency.