Skip to content

feat(home): Optimized hero images on the home page #16623

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.

Sign up for GitHub

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

Open
sfanahata wants to merge 1 commit into
base: master
Choose a base branch
from
+1 −1
Open

feat(home): Optimized hero images on the home page #16623

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/components/home.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import {Banner} from 'sentry-docs/components/banner';
import {extractPlatforms, getDocsRootNode} from 'sentry-docs/docTree';
import AiInSentryHero from 'sentry-docs/imgs/AI-in-Sentry.jpeg';
import AiSentryHero from 'sentry-docs/imgs/ai-sentry-hero.png';
import AiSentryHero from 'sentry-docs/imgs/ai-sentry-hero.jpeg';
import PlugImage from 'sentry-docs/imgs/api.png';
import ChatBubble from 'sentry-docs/imgs/chat-bubble.png';
import TerminalImage from 'sentry-docs/imgs/cli.png';
Expand Down Expand Up @@ -60,7 +60,7 @@
{/* Hero Banner with Two-Column Layout */}
<div className="w-full relative hero-gradient">
{/* Single decorative squiggle at top right */}
<img

Check warning on line 63 in src/components/home.tsx

View workflow job for this annotation

GitHub Actions / Lint 

Using `<img>` could result in slower LCP and higher bandwidth. Consider using `<Image />` from `next/image` to automatically optimize images. This may incur additional usage or cost from your provider. See: https://nextjs.org/docs/messages/no-img-element
src={SquiggleSVG.src}
alt=""
className="pointer-events-none absolute overflow-hidden"
Expand Down Expand Up @@ -184,7 +184,7 @@
justifyContent: 'center',
}}
>
<img

Check warning on line 187 in src/components/home.tsx

View workflow job for this annotation

GitHub Actions / Lint 

Using `<img>` could result in slower LCP and higher bandwidth. Consider using `<Image />` from `next/image` to automatically optimize images. This may incur additional usage or cost from your provider. See: https://nextjs.org/docs/messages/no-img-element
src={ErrorMonitoringHero.src}
alt="Debugging in Sentry"
style={{
Expand All @@ -202,26 +202,26 @@
<p className="text-sm text-[var(--gray-11)] dark:text-white leading-relaxed">
Monitor, identify, and resolve errors and performance issues across your
applications using{' '}
<a href="/product/error-monitoring/" className="text-[#8b5cf6] underline">

Check warning on line 205 in src/components/home.tsx

View workflow job for this annotation

GitHub Actions / Lint 

Do not use an `<a>` element to navigate to `/product/error-monitoring/`. Use `<Link />` from `next/link` instead. See: https://nextjs.org/docs/messages/no-html-link-for-pages
Error Monitoring
</a>
,{' '}
<a

Check warning on line 209 in src/components/home.tsx

View workflow job for this annotation

GitHub Actions / Lint 

Do not use an `<a>` element to navigate to `/product/explore/trace-explorer/`. Use `<Link />` from `next/link` instead. See: https://nextjs.org/docs/messages/no-html-link-for-pages
href="/product/explore/trace-explorer/"
className="text-[#8b5cf6] underline"
>
Tracing
</a>
,{' '}
<a href="/product/session-replay/" className="text-[#8b5cf6] underline">

Check warning on line 216 in src/components/home.tsx

View workflow job for this annotation

GitHub Actions / Lint 

Do not use an `<a>` element to navigate to `/product/session-replay/`. Use `<Link />` from `next/link` instead. See: https://nextjs.org/docs/messages/no-html-link-for-pages
Session Replay
</a>
,{' '}
<a href="/product/drains/" className="text-[#8b5cf6] underline">

Check warning on line 220 in src/components/home.tsx

View workflow job for this annotation

GitHub Actions / Lint 

Do not use an `<a>` element to navigate to `/product/drains/`. Use `<Link />` from `next/link` instead. See: https://nextjs.org/docs/messages/no-html-link-for-pages
Logs
</a>
, and{' '}
<a href="/product/" className="text-[#8b5cf6] underline">

Check warning on line 224 in src/components/home.tsx

View workflow job for this annotation

GitHub Actions / Lint 

Do not use an `<a>` element to navigate to `/product/`. Use `<Link />` from `next/link` instead. See: https://nextjs.org/docs/messages/no-html-link-for-pages
more
</a>
.
Expand All @@ -245,7 +245,7 @@
justifyContent: 'center',
}}
>
<img

Check warning on line 248 in src/components/home.tsx

View workflow job for this annotation

GitHub Actions / Lint 

Using `<img>` could result in slower LCP and higher bandwidth. Consider using `<Image />` from `next/image` to automatically optimize images. This may incur additional usage or cost from your provider. See: https://nextjs.org/docs/messages/no-img-element
src={AiSentryHero.src}
alt="Fix Bugs Faster with Seer"
style={{
Expand All @@ -262,14 +262,14 @@
</h3>
<p className="text-sm text-[var(--gray-11)] dark:text-white leading-relaxed">
Debug applications automatically with{' '}
<a

Check warning on line 265 in src/components/home.tsx

View workflow job for this annotation

GitHub Actions / Lint 

Do not use an `<a>` element to navigate to `/product/ai-in-sentry/seer/`. Use `<Link />` from `next/link` instead. See: https://nextjs.org/docs/messages/no-html-link-for-pages
href="/product/ai-in-sentry/seer/"
className="text-[#8b5cf6] underline"
>
Seer
</a>
. Use{' '}
<a

Check warning on line 272 in src/components/home.tsx

View workflow job for this annotation

GitHub Actions / Lint 

Do not use an `<a>` element to navigate to `/product/ai-in-sentry/seer/root-cause-analysis/`. Use `<Link />` from `next/link` instead. See: https://nextjs.org/docs/messages/no-html-link-for-pages
href="/product/ai-in-sentry/seer/root-cause-analysis/"
className="text-[#8b5cf6] underline"
>
Expand Down
Binary file modified src/imgs/AI-in-Sentry.jpeg
View file
Open in desktop
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added src/imgs/ai-sentry-hero.jpeg
View file
Open in desktop
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file removed src/imgs/ai-sentry-hero.png
View file
Open in desktop
Binary file not shown.
Loading