Skip to content

add event component#31

Open
victoriacheng15 wants to merge 3 commits intomainfrom
feature/event-component
Open

add event component#31
victoriacheng15 wants to merge 3 commits intomainfrom
feature/event-component

Conversation

@victoriacheng15
Copy link
Collaborator

#18

image

@victoriacheng15 victoriacheng15 force-pushed the feature/event-component branch from a78067a to 3b5d995 Compare January 31, 2025 01:50
@victoriacheng15
Copy link
Collaborator Author

victoriacheng15 commented Jan 31, 2025

add /src/data/eventConfig.ts to hold event details values, so we can edit event details from the file in the future.

@lindan4 lindan4 requested a review from a team February 8, 2025 01:10

p {
margin: 0;
font-size: 1.25rem;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Out of curiosity, why are using rem sizes?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

rem is relative to the root font size of the document, making it more consistent and scalable across different screen sizes. This helps with accessibility and responsiveness. Plus, it's my habit to use rem for better maintainability.

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