Skip to content
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

[DO NOT MERGE] Test "Contact Us" button in the page header #728

Closed
wants to merge 1 commit into from

Conversation

dzole0311
Copy link

Why are you creating this Pull Request?

@dzole0311 dzole0311 changed the title [DO NOT MERGE] Test prev page header [DO NOT MERGE] Test "Contact Us" button in the page header Feb 10, 2025
Copy link

netlify bot commented Feb 10, 2025

Deploy Preview for ghg-demo ready!

Name Link
🔨 Latest commit 9e2c93e
🔍 Latest deploy log https://app.netlify.com/sites/ghg-demo/deploys/67a9daf0e9a166000792e6e4
😎 Deploy Preview https://deploy-preview-728--ghg-demo.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 site configuration.

dzole0311 added a commit to NASA-IMPACT/veda-ui that referenced this pull request Feb 10, 2025
**Related Ticket:** closes
#1388

### Description of Changes

During the recent header redesign, we created a "legacy" version of the
old header to keep consistency across existing deployments (like GHG
Center) and to ease the development process. However, a USWDS-styled
button had accidentally been included in this legacy header and it's
causing visual inconsistency with the original GHG Center design. This
PR removes the USWDS button from the legacy header and replaces it with
the previous styled button (that was removed here
https://github.com/NASA-IMPACT/veda-ui/pull/1247/files#diff-3dc664e136e6e14925e5f449b907df7edc49fd1d2744cf0cb1aea27a63646846L51).
 
### Notes & Questions About Changes
_{Add additonal notes and outstanding questions here related to changes
in this pull request}_

### Validation / Testing

1. Verify that the new header is shown as usual in the veda UI
1. Verify that the "Contact Us" button is not a USWDS button in the [GHG
Center preview
deployment](US-GHG-Center/veda-config-ghg#728)
and verify that it matches some of the earlier deployments of GHG Center
(example of an older deployment:
https://deploy-preview-455--ghg-demo.netlify.app/)
dzole0311 added a commit to NASA-IMPACT/veda-ui that referenced this pull request Feb 10, 2025
**Related Ticket:** closes
#1388

During the recent header redesign, we created a "legacy" version of the
old header to keep consistency across existing deployments (like GHG
Center) and to ease the development process. However, a USWDS-styled
button had accidentally been included in this legacy header and it's
causing visual inconsistency with the original GHG Center design. This
PR removes the USWDS button from the legacy header and replaces it with
the previous styled button (that was removed here
https://github.com/NASA-IMPACT/veda-ui/pull/1247/files#diff-3dc664e136e6e14925e5f449b907df7edc49fd1d2744cf0cb1aea27a63646846L51).

_{Add additonal notes and outstanding questions here related to changes
in this pull request}_

1. Verify that the new header is shown as usual in the veda UI
1. Verify that the "Contact Us" button is not a USWDS button in the [GHG
Center preview
deployment](US-GHG-Center/veda-config-ghg#728)
and verify that it matches some of the earlier deployments of GHG Center
(example of an older deployment:
https://deploy-preview-455--ghg-demo.netlify.app/)
@dzole0311 dzole0311 closed this Feb 13, 2025
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.

1 participant