Skip to content

Revamp /contact/ into /get-support/ - #966

Merged
jgravois merged 7 commits into
mainfrom
feat/get-support
Aug 27, 2026
Merged

Revamp /contact/ into /get-support/#966
jgravois merged 7 commits into
mainfrom
feat/get-support

Conversation

@jgravois

@jgravois jgravois commented Aug 25, 2026

Copy link
Copy Markdown
Member

closes #965

view it live: https://deploy-preview-966--cal-itp-mobility-marketplace.netlify.app/get-support/

  • replaces /contacts/ with /get-support/ and refreshes the design
  • shuffles some existing CSS (to make the styles available in more than one layout template)
  • redirects from /contact/ to the new page
  • adds some margin-bottom for the <h1> in /about to match /get-support/ (and Figma)

speaking of Figma, i posted a couple comments there with questions about a few implementation details. @cmajel feel free to reply wherever is easiest.

@netlify

netlify Bot commented Aug 25, 2026

Copy link
Copy Markdown

Deploy Preview for cal-itp-mobility-marketplace ready!

Name Link
🔨 Latest commit 53158d6
🔍 Latest deploy log https://app.netlify.com/projects/cal-itp-mobility-marketplace/deploys/6a9072dcf70cd70008e7b184
😎 Deploy Preview https://deploy-preview-966--cal-itp-mobility-marketplace.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@jgravois
jgravois marked this pull request as ready for review August 26, 2026 20:53
@jgravois
jgravois requested a review from a team as a code owner August 26, 2026 20:53
@jgravois
jgravois requested a review from cmajel August 26, 2026 20:53
@cmajel

cmajel commented Aug 27, 2026

Copy link
Copy Markdown
Member

@jgravois looks great!

My one suggestion is to remove the scrolling animation for a straight jump to limit motion on the page and when the #contactlink is shared directly. I'm comfortable making that change to the skip to main content link as well for consistency (which I believe is the only other place we have a jump link right now).

@jgravois

jgravois commented Aug 27, 2026

Copy link
Copy Markdown
Member Author

My one suggestion is to remove the scrolling animation for a straight jump to limit motion on the page

good call. turns out it was bootstrap that was setting 'smooth' scrolling. i overrode the default for the entire site in 02ef259

@jgravois
jgravois enabled auto-merge August 27, 2026 16:58
@jgravois
jgravois disabled auto-merge August 27, 2026 16:58

@Scotchester Scotchester left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks great for the most part. I noticed a couple teeny nits that snuck by Christine :)

Comment thread src/get-support.liquid Outdated
Comment thread src/styles/gallery-cards.css Outdated
}
.gallery-card {
border: 1px solid var(--dsdl-gray-30);
border-radius: 40px;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I see this was carried over from the cards on the homepage, but the cards on this new Get Support page are supposed to have 20px radii. Could maybe make it a variable like --gallery-card-radius that has a default in :root that gets overridden in contact.css?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

much nicer! 53158d6

@jgravois
jgravois enabled auto-merge August 27, 2026 17:25

@Scotchester Scotchester left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@jgravois
jgravois removed the request for review from cmajel August 27, 2026 18:12
@jgravois
jgravois merged commit 770f5ec into main Aug 27, 2026
6 checks passed
@jgravois
jgravois deleted the feat/get-support branch August 27, 2026 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement revamped 'Get Support' page

3 participants