Apply the copy and layout trims from the 1 Oct client review - #7
Merged
Merged
Conversation
Changes the client walked through on the call, all of them removals or re-weightings rather than new surface: - Drop "Architectural & Design Consultancy" from the hero eyebrow and from under the footer wordmark. (He called it "collaborative designs" on the call; the recording shows he was pointing at this line.) - Instagram in the footer is the icon alone, enlarged, with the handle moved to aria-label so the link is still announced. - Add an Instagram link to the header beside Enquire. It stays visible below 620px where .nav__cta is hidden — the practice's first enquiry arrived through the profile, so it should not be the thing that drops on a phone. - Remove the inline phone beside "Start a project" in the hero and the CTA band. The floating Call button already covers that intent. - Contact page: the two numbers were inline-flex with a size step between them, so they ran onto one line and collided. They now stack at one size, which is also what he asked for — both numbers published equally. - Footer columns: auto-fit split the row evenly and left Contact ~204px, just under what the email needs, so it broke mid-word. Weight the columns so the width goes to the one that uses it. Verified against the production build: the dead rules are gone, the new ones survive minification, and the 821px rule is still scoped to its media query.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes the client walked through on the 1 Oct call. All of them are removals
or re-weightings — no new surface.
Copy
under the footer wordmark. He called it "collaborative designs" on the call;
the recording confirms he was pointing at this line.
moves to
aria-labelso the link is still announced, and it gets a 44px hitarea now that there is no text to click.
Contact affordances
where
.nav__ctais hidden — the practice's first enquiry arrived through theprofile, so it should not be the thing that drops on a phone.
CTA band. The floating Call button already covers that intent, and it carries
both numbers.
inline-flexwith a size step betweenthem, so they ran onto one line and collided. They now stack at one size.
Both numbers stay published, equally — the client was explicit about that, and
the
--altmodifier is gone entirely so there is no variant left to drift.Layout
auto-fitsplit the row evenly and left Contact ~204px,just under what
kalapritidesigns@gmail.comneeds, so it broke mid-word.Weighted the columns so the width goes to the one that uses it. Follow is now
a single icon plus a short link, so it had room to give.
Verification
npm run lintandnpm run buildclean.gone, the new ones survive, and the 821px rule is still scoped to its media
query (LightningCSS rewrites it to
width>=821px).font-size,font-family,font-weight,color, icon size, gap and height; stacked 6px apart.