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

Contact #5

Merged
merged 20 commits into from
Mar 7, 2025
Merged

Contact #5

merged 20 commits into from
Mar 7, 2025

Conversation

czarekmilek
Copy link
Collaborator

Contact Page

  • Initialized view
  • Added basic form
  • Replaced background logo used in other views with a higher quality one
  • Added simple slide-in from the left animation for the hero section
  • Extracted reusable components

Footer

  • Extracted Footer into an adjacent component

Hooks

  • Added new hook for custom tab titles for pages

Other

  • Replaced Vite logo with KSI logo in tab header
  • Added unique tab titles for each existing page
  • Organized project structure a little

jakjank and others added 20 commits January 21, 2025 20:37
- Removed fixed values from parameters
- Positioning using `relative`, not `absolute`
- Added `logo_ksi.svg` of a higher quality than the old logo
-  Only `Contact` view has the new logo version
- Updated `Footer` to always be visible in front of other components
- Deleted unused react image
- Added new hook for setting tab title
- Added new folders for components and pages
- Extracted `HeroSection`, `BackgroundImage` and `ContactForm` components from `Contact` page
- Updated `Contact` page to use the new components
- Moved `Contact` page to the new pages folder
@czarekmilek czarekmilek added enhancement New feature or request good first issue Good for newcomers labels Mar 5, 2025
@czarekmilek czarekmilek self-assigned this Mar 5, 2025
Copy link
Collaborator

@porfinogeneta porfinogeneta left a comment

Choose a reason for hiding this comment

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

git

@porfinogeneta porfinogeneta merged commit 9b37bce into master Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants