Skip to content

Conversation

@AntoLC
Copy link
Collaborator

@AntoLC AntoLC commented Sep 3, 2025

Purpose

A section in the homepage was not displaying correctly anymore. This commit fixes the issue.

Demo

image
@AntoLC AntoLC self-assigned this Sep 3, 2025
@AntoLC AntoLC added bug Something isn't working frontend labels Sep 3, 2025
@AntoLC AntoLC force-pushed the fix/display-bug-homepage branch from 41c3a60 to be6af9f Compare September 3, 2025 08:50
@AntoLC AntoLC requested a review from Ovgodd September 3, 2025 08:52
@github-actions
Copy link

github-actions bot commented Sep 3, 2025

Size Change: -4 B (0%)

Total Size: 3.64 MB

Filename Size Change
apps/impress/out/_next/static/8a028a96/_buildManifest.js 0 B -868 B (removed) 🏆
apps/impress/out/_next/static/5a73cff5/_buildManifest.js 867 B +867 B (new file) 🆕

compressed-size-action

Copy link
Collaborator

@Ovgodd Ovgodd left a comment

Choose a reason for hiding this comment

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

Just a little thing ;)

{title}
</Text>
<Text
as="div"
Copy link
Collaborator

@Ovgodd Ovgodd Sep 4, 2025

Choose a reason for hiding this comment

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

I would recommend using a p tag instead of a div for better semantics.
If you want to keep the same visual style without the default margin applied to p, you can simply add:
wdyt ?

Suggested change
as="div"
$css={css`
margin: 0;
`}
as="p"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I don't think we should use a "p" tag here, because it contains "div" and others "p" tag in it.

image
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ahh yes, I agree with you, div is a better choice.

@AntoLC AntoLC requested a review from Ovgodd September 4, 2025 08:37
Copy link
Collaborator

@Ovgodd Ovgodd left a comment

Choose a reason for hiding this comment

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

good!

A section in the homepage was not displaying correctly anymore. This commit fixes the issue.
@AntoLC AntoLC force-pushed the fix/display-bug-homepage branch from be6af9f to 39ef6d1 Compare September 4, 2025 08:44
@AntoLC AntoLC merged commit 39ef6d1 into main Sep 4, 2025
21 of 22 checks passed
@AntoLC AntoLC deleted the fix/display-bug-homepage branch September 4, 2025 08:59
@AntoLC AntoLC mentioned this pull request Sep 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working frontend

3 participants