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

Add sections 1 #30

Merged
merged 24 commits into from
Sep 22, 2019
Merged

Add sections 1 #30

merged 24 commits into from
Sep 22, 2019

Conversation

Aichnerc
Copy link
Member

Checklist:

  • Have you added an explanation of what your changes do and why you'd like them to be included?
  • Have you updated or added documentation for the change?
  • Have you tested your changes with successful results?

Type of Changes:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

What is the current behavior? (link to any open issues here)

What is the new behavior (if this is a feature change)?

The section Call To Action has been added.
More padding has been added to the Footer component.
Images and Parallax have been implemented.
Particle effects have been added.
The particle js library has been removed.
A text within the CallToAction component has been improved.
The Services section has been added.
All components have been cleaned up
abit.
The weddings section has been added. A fancy
button has been added as well.
The wedding section component has been
placed in the wrong folder.
The Weddings component is now called Additional
and will hold additional services we offer. Including
Social Media, Tutorials, Weddings, ...
A new button type has been added.
More data and a row/col system has been
added to the Additional section.
The old png logo has been replaced by SVG images.
React-loadable and React-loadable-visibility modules
have been added.
An id has been added to the logos to distinctly
manage its color apart from other SVG files.
An image has been added to the Hero component.
The content of the hero component has been improved.
The react-loadable modules have been implemented.
Static components were assigned the PureComponent
type of components.
The code quality of the Additional component section
has been improved.
The code quality of the CallToAction component section
has been improved.
The navbar logo color has been fixed in light mode.
@Aichnerc Aichnerc added enhancement Enhancement of an existing feature or request feature Completely new feature or request labels Sep 16, 2019
@Aichnerc Aichnerc added this to the 1.0.0 milestone Sep 16, 2019
@Aichnerc Aichnerc requested a review from kleberbaum September 16, 2019 13:53
@Aichnerc Aichnerc self-assigned this Sep 16, 2019
Copy link
Member Author

@Aichnerc Aichnerc left a comment

Choose a reason for hiding this comment

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

After a line-length change it should be okay.

Enforcement of the max. chars per line policy.
Ref: A function with a name starting with an
uppercase letter should only be used as a
constructor.
Copy link
Member

@kleberbaum kleberbaum left a comment

Choose a reason for hiding this comment

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

Should be ok to merge after a few changes.

Particle.js has been removed from the packages
used in this project.
@Aichnerc Aichnerc requested a review from kleberbaum September 22, 2019 17:17
Copy link
Member

@kleberbaum kleberbaum left a comment

Choose a reason for hiding this comment

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

Should be ok to merge.

@kleberbaum kleberbaum merged commit d7d6808 into master Sep 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement of an existing feature or request feature Completely new feature or request
Development

Successfully merging this pull request may close these issues.

2 participants