Skip to content

Ai use case section#4842

Open
britt6612 wants to merge 10 commits intohpe-products-sectionfrom
AI-use-case-section
Open

Ai use case section#4842
britt6612 wants to merge 10 commits intohpe-products-sectionfrom
AI-use-case-section

Conversation

@britt6612
Copy link
Collaborator

Deploy Preview

What does this PR do?

Adds AI use case section

Where should the reviewer start?

What testing has been done on this PR?

In addition to the feature you are implementing, have you checked the following:

General UX Checks

  • Small, medium, and large screen sizes
  • Cross-browsers (FireFox, Chrome, and Safari)
  • Light & dark modes
  • All hyperlinks route properly

Accessibility Checks

  • Keyboard interactions
  • Screen reader experience
  • Run WAVE accessibility plugin (Chrome)

Code Quality Checks

  • Console is free of warnings and errors
  • Passes E2E commit checks
  • Visual snapshots are reasonable

How should this be manually tested?

Any background context you want to provide?

What are the relevant issues?

Screenshots (if appropriate)

Should this PR be mentioned in Design System updates?

Is this change backwards compatible or is it a breaking change?

@changeset-bot
Copy link

changeset-bot bot commented Mar 7, 2025

⚠️ No Changeset found

Latest commit: 2cfd8f0

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@britt6612 britt6612 changed the base branch from master to add-marketing-page March 7, 2025 05:42
@netlify
Copy link

netlify bot commented Mar 7, 2025

Deploy Preview for rad-shortbread-897892 ready!

Name Link
🔨 Latest commit 5e4456c
🔍 Latest deploy log https://app.netlify.com/sites/rad-shortbread-897892/deploys/67ca875d5712db0008442051
😎 Deploy Preview https://deploy-preview-4842--rad-shortbread-897892.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.

@netlify
Copy link

netlify bot commented Mar 7, 2025

Deploy Preview for unrivaled-bublanina-3a9bae ready!

Name Link
🔨 Latest commit 5e4456c
🔍 Latest deploy log https://app.netlify.com/sites/unrivaled-bublanina-3a9bae/deploys/67ca875daa7b6900088d8c43
😎 Deploy Preview https://deploy-preview-4842--unrivaled-bublanina-3a9bae.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.

@taysea
Copy link
Collaborator

taysea commented Mar 18, 2025

Seems like this should share the background image with the above image rather than starting again

Screenshot 2025-03-18 at 1 37 01 PM

actions={
// button sizes are off in design they have 42px height and we have 36px
<Button
label="View the usecase brief"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same comment as above about adding aria-label that also includes title. For example "View the usecase brief for [title]"

alt="hpe cray supercomputing"
fit="cover"
// this is not ideal
style={{
Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't see where this is applying

description="Exascale technologies to unlock the next frontier of discovery, innovation, and achievement."
actions={
<Button
label="View the usecase brief"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same comment re: specific aria-label

@britt6612 britt6612 changed the base branch from add-marketing-page to hpe-products-section March 19, 2025 03:39
britt6612 and others added 6 commits March 26, 2025 13:20
…UseCase.tsx

Co-authored-by: Taylor Seamans <taylor.seamans@yahoo.com>
…UseCase.tsx

Co-authored-by: Taylor Seamans <taylor.seamans@yahoo.com>
…UseCase.tsx

Co-authored-by: Taylor Seamans <taylor.seamans@yahoo.com>
…UseCase.tsx

Co-authored-by: Taylor Seamans <taylor.seamans@yahoo.com>
…UseCase.tsx

Co-authored-by: Taylor Seamans <taylor.seamans@yahoo.com>
…UseCase.tsx

Co-authored-by: Taylor Seamans <taylor.seamans@yahoo.com>
@britt6612
Copy link
Collaborator Author

Seems like this should share the background image with the above image rather than starting again

Screenshot 2025-03-18 at 1 37 01 PM

good point I will change so they are part of the same section

@britt6612 britt6612 requested a review from taysea March 26, 2025 19:24
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