madeleineostoja / madeleineostoja/postcss-responsive-type

Responsive line-height with default parameters does not work

Open
#30 6 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
JavaScript
Stars
368
Forks
21
PR merge metrics
No merged PRs in 30d

Description

Hi. I'm using postcss-responsive-type@1.0.0.
I just wanted to use responsive line-height.

```css
html {
line-height: responsive;
}
```

Unfortunately it generates an error:

```
postcss-responsive-type: $FILE_PATH/default.css:5:1: this combination of units is not supported
```

and invalid css:

```css
html {
line-height: calc(1.2em + 0.6000000000000001 * ((100vw - undefined) / NaN));
}

@media screen and (min-width: 1280px) {
html {
line-height: 1.8em;
}
}

@media screen and (max-width: 420px) {
html {
line-height: 1.2em;
}
}
```

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reproducing the responsive line-height example with the default parameters and trace how those defaults are read when generating the calc() expression. Done means the example produces valid CSS without undefined, NaN, or the unsupported-units error while retaining the expected media-query values.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
tooling
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.