Skip to content

Conversation

@stephprince
Copy link
Contributor

Add front page banner and link to webpage with more info about supporting NWB

@netlify
Copy link

netlify bot commented Aug 22, 2025

Deploy Preview for nwborg ready!

Name Link
🔨 Latest commit bdd49db
🔍 Latest deploy log https://app.netlify.com/projects/nwborg/deploys/68a89992eb47950008626e96
😎 Deploy Preview https://deploy-preview-114--nwborg.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.

@stephprince
Copy link
Contributor Author

@rly @oruebel I think this covers most of the items we discussed. I wasn't sure about the exact layout of all of the stats figures and downloads video so feel free to edit as needed

@oruebel
Copy link
Contributor

oruebel commented Aug 22, 2025

@stephprince I just took at the page. The only minor suggestion I have is that it would be nice to reduce the empty white-space at the top and bottom of the "How You Can Help" section a bit to make sure folks don't accidentally miss the content with the stats that appear below it by not scrolling far enough. Otherwise the page looks great 👍

rly
rly previously approved these changes Aug 22, 2025
@rly
Copy link
Contributor

rly commented Aug 22, 2025

I agree with @oruebel 's spacing suggestion and made the change. The page looks great!

@rly
Copy link
Contributor

rly commented Aug 22, 2025

Minor: The blue text in the graphs, e.g.,
image

is redundant with the text above it and could be removed. I know these are images, so it's not super easy to do.

@oruebel
Copy link
Contributor

oruebel commented Aug 22, 2025

is redundant with the text above it and could be removed.

TBH, I like having the figure in the text.

oruebel
oruebel previously approved these changes Aug 22, 2025
@oruebel
Copy link
Contributor

oruebel commented Aug 22, 2025

I agree with @oruebel 's spacing suggestion and made the change.

Thanks @rly for making that adjustment.

@oruebel oruebel requested a review from bendichter August 22, 2025 06:35
@bendichter bendichter dismissed stale reviews from oruebel and rly via bdd49db August 22, 2025 16:23
@bendichter bendichter marked this pull request as ready for review August 22, 2025 16:23
@bendichter bendichter enabled auto-merge August 22, 2025 16:24
@bendichter bendichter disabled auto-merge August 22, 2025 16:45
@bendichter bendichter enabled auto-merge (squash) August 22, 2025 16:45
@bendichter bendichter disabled auto-merge August 22, 2025 16:47
@bendichter bendichter merged commit a0f33a5 into main Aug 22, 2025
4 of 5 checks passed
@bendichter bendichter deleted the add-advocacy-page branch August 22, 2025 16:48
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.

4 participants