Skip to content

Cam 1358 - #123

Open
martinseul wants to merge 3 commits into
developfrom
cam-1358
Open

Cam 1358#123
martinseul wants to merge 3 commits into
developfrom
cam-1358

Conversation

@martinseul

Copy link
Copy Markdown
Member

added section 1 of the about page

@netlify

netlify Bot commented Aug 19, 2026

Copy link
Copy Markdown

Deploy Preview for hydroprocess ready!

Name Link
🔨 Latest commit ec96ee1
🔍 Latest deploy log https://app.netlify.com/projects/hydroprocess/deploys/6a85c22a37543b000835a741
😎 Deploy Preview https://deploy-preview-123--hydroprocess.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.

@martinseul
martinseul requested a review from igarousi August 19, 2026 14:50

@igarousi igarousi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please see my review comments. The margins don't quite match the mockup yet. Please use the mockup as the spacing reference and add more margins.

continental scales. Below you can find the team, funding resources, publications, and
data citations behind each project.
</p>
</div>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

  • Please move the content slightly further to the right by increasing the space between the content and the left edge, matching the mockup.
  • When adjusting the left margin, please make sure the same margin is maintained on the right side.
  • The content currently is too close to the top edge. Please add more top margin/padding to match the vertical spacing shown in the mockup.

Synthesizing hydrologic process knowledge to determine global drivers of dominant
processes.
</a>
<section class="about-citation">

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

  • This section needs more breathing room overall. Please increase the padding/margins around this section so the box is larger. See the mockups.
  • The two cards under this section should also have a bit more internal spacing and height, matching the mockups.
  • Please make sure the content inside the box aligns with the content of the "about-page" container.

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.

2 participants