Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion src/sections/subscribe/subscribe.style.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,10 +11,11 @@ const SubscribeWrapper = styled.div`

h2{
font-weight: 600;
font-size:1.875rem;
font-size: clamp(1.25rem, 1.9vw, 1.875rem);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Could we keep the heading size continuous across the 912px breakpoint? At 913px and above, this clamp() can resolve to its 1.25rem minimum, while at 912px the media query overrides it with 1.75rem. This means shrinking the viewport by just 1px can cause a 20px → 28px jump. Since this is a responsive fix, could we keep the font sizing continuous across the breakpoint while preserving the wrapping behavior below 912px?

margin:1.25rem 0;
color:${props => props.theme.text};
transition: 0.8s cubic-bezier(0.2, 0.8, 0.2, 1);
white-space: nowrap;
Comment thread
AnkitRewar11 marked this conversation as resolved.
}
h2 span{
color:${props => props.theme.primaryColor};
Expand Down Expand Up @@ -58,6 +59,7 @@ const SubscribeWrapper = styled.div`
h2{
font-size:1.75rem;
margin: 1.5625rem 0;
white-space: normal;
}
input{
width:90%;
Expand Down