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

public 이미지를 assets 으로 옮기기 #239

Merged
merged 12 commits into from
Jul 14, 2024

Conversation

seheon99
Copy link
Contributor

@seheon99 seheon99 commented Jul 4, 2024

변경 개요

구현 내용

관련 이슈

@seheon99 seheon99 requested a review from jaewoongs July 4, 2024 10:46
@seheon99 seheon99 self-assigned this Jul 4, 2024
Copy link

vercel bot commented Jul 4, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
fit-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 14, 2024 3:27am

@seheon99 seheon99 marked this pull request as draft July 4, 2024 13:02
- Replace static image URLs with imported image modules in the MatchingSequenceSwiper component. This shift allows for stronger type checking and potential optimizations by the module bundler.
- Simplify the props passed to the `MainDescriptionCard` by using a single `image` object instead of separate URL, width, and height properties, streamlining component interfaces and reducing redundancy.

These changes enhance maintainability and prepare the component for more flexible image management, such as easier transitions to different image formats or dynamic loading techniques.
seheon99 added 2 commits July 14, 2024 12:10
- Update the `StyledSection` to use a column layout with centered alignment, improving the structural hierarchy and visual alignment on various screen sizes.
- Adjust `ImageContainer` to be fully responsive by setting its width to 100% and constraining its maximum width to 1200px. This ensures the image scales properly within larger viewports while maintaining a reasonable size limit.

These changes are aimed at enhancing the visual appeal and responsiveness of the MainSection2, ensuring it looks optimal on different devices.
- Encapsulate the CardSwiper component within a new SwiperContainer for improved structure and responsiveness. This change adds an extra layer of flexibility in managing the positioning and styling of the swiper.
- Remove outdated `overflow: hidden;` from the Block styled component, simplifying the CSS and eliminating unused styles.
- Minor adjustments to styling to accommodate these structural changes, including better organization and separation of styled components for clarity and maintenance.

These modifications aim to enhance the visual presentation and manageability of the swiper component within the IntroRecommendBlock, ensuring it better suits various device layouts.
@seheon99 seheon99 marked this pull request as ready for review July 14, 2024 03:15
- Amend the Dockerfile to copy `next-env.d.ts` and `svgr.d.ts` files along with the existing `next.config.js` and `tsconfig.json`. This ensures that all necessary TypeScript definition files are available in the Docker image, supporting proper type-checking and configuration during the build process.
- These changes aim to enhance build stability and maintain consistency across development and production environments by ensuring all relevant configuration files are included in the Docker build context.

This adjustment ensures that the Docker environment mirrors the local development setup more closely, potentially reducing issues related to environment discrepancies.
@jaewoongs jaewoongs merged commit c75d5b4 into dev Jul 14, 2024
5 checks passed
@jaewoongs jaewoongs deleted the refactor/236-optimize-assets branch July 14, 2024 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

메인화면 이미지 크기 줄이기
2 participants