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
Header text corrections #46
Conversation
This pull request has been linked to Shortcut Story #10470: The header container should fit the body. |
Codecov Report
@@ Coverage Diff @@
## main #46 +/- ##
=======================================
Coverage 52.32% 52.32%
=======================================
Files 47 47
Lines 3232 3232
=======================================
Hits 1691 1691
Misses 1359 1359
Partials 182 182
Flags with carried forward coverage won't be shown. Click here to find out more. Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good 👍 . well done. just a few suggestions if you can a get chance to check that out.
<h1 className="pb-1 text-xl sm:text-3xl md:text-6xl xl:text-7xl">Ensign</h1> | ||
<h2 className="font-light text-base sm:text-base md:text-2xl lg:text-4xl">Powering Realtime Experiences</h2> | ||
<h2 className="font-light text-base sm:text-base md:text-2xl lg:text-4xl">& Data-driven Apps</h2> | ||
<section className="text-white xl:max-w-7xl mx-auto"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what do you think if you use max-w-7xl
on all screens?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! I've made the change.
<Main /> | ||
<Diagram /> | ||
<BuildApps /> | ||
<div className="max-w-xl pl-2 pb-20 mx-auto sm:max-w-2xl lg:max-w-3xl xl:max-w-6xl"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you try max-w-7xl instead?
Scope of changes
Updated the header so that text remains centered as screen size increased. Adjusted text to be more in line with rotational.io.
Fixes SC-10470 & SC-10475
Type of change
Acceptance criteria
Describe how reviewers can test this change to be sure that it works correctly. Add a checklist if possible.
Author checklist
Reviewer(s) checklist