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

Feature/cra 18 #19

Open
wants to merge 26 commits into
base: master
Choose a base branch
from
Open

Feature/cra 18 #19

wants to merge 26 commits into from

Conversation

DawidZlobecki
Copy link
Contributor

Single announcement

DawidZlobecki and others added 9 commits February 7, 2022 22:36
* feature CRA/2 :navbar menu plus logo

* feature/CRA-2:addupdate

* fix: feature/CRA-2 some fixes and changes

* fix: some fixes

* fix: some fixes

* fix: fixed bugs with stepbystepitem and some fixes

* fix: fix

* fix: fixed problem with displaying hamburger and menu

* fix: logo.png was moved to assets

* fix: set the navbar elements into links and downsized a logo

* rename class

* fix: fixed button - adds class on click

* added sliding to navbar in mobile mode

* WIP

* feat: add styles for hamburger menu
add background image
change logo image
add image for section in main

* refactor: change module name

* refactor: change module name

* feat: add custom hook
change styles for navbar
change navbar and menu file
add style for swiper

* fix: change className for hamburger

* fix: change eslint rule

Co-authored-by: JustD1d1t <starzyk.dawid.0826@gmail.com>
Co-authored-by: dawidzlobecki <dawidzlobecki2@gmail.com>
Co-authored-by: MaciejP <m.ponicki1995@gmail.com>
Co-authored-by: Maciej Ponicki <72940506+MaciekPon@users.noreply.github.com>
* feat: Services list

* feat: add DividerSection

* Refactor: minor changes

* Refactor: minor changes

* Refactor: minor changes

* Refactor: minor changes

* Refactor: minor changes
@DawidZlobecki DawidZlobecki self-assigned this Feb 8, 2022
DawidZlobecki and others added 8 commits February 8, 2022 21:28
* Merge branch 'master' into feature/CRA-15

* feat: add Encourage sectionto main page
add another dog image
add Section component
* refactor: add entry point in utils

* refactor(auth-context): move AuthContext from js to jsx

* refactor: move opinions and steps to constants

* refactor: move steplist to main components

* refactor: minor changes

* feature: styles for forms

* refactor: add rwd styles

* refactor: self closing tags, class style instead tag

* refactor: move erro span to component

* refactor: errorBox gets children props instead error, export/import
* Merge branch 'master' into feature/CRA-15

* feat: add Encourage sectionto main page
add another dog image
add Section component

* fix: change file name

* feat: add section to main page
add styles for section.jsx

* feat: add changes from code review
change image name in assets

* fix: change images names

* fix: change image name in encourage section
@ruljin ruljin added the enhancement New feature or request label Feb 9, 2022
Copy link
Contributor

@ruljin ruljin left a comment

Choose a reason for hiding this comment

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

Changes requested

src/mock/auth.js Outdated Show resolved Hide resolved
src/mock/auth.js Outdated
name: "userName",
description:
"Lorem ipsum dolor sit amet consectetur adipisicing elit. Ratione, placeat?",
announcement: [1, 2, 3, 8, 9],
Copy link
Contributor

Choose a reason for hiding this comment

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

☝️

src/mock/auth.js Outdated Show resolved Hide resolved
src/mock/auth.js Outdated Show resolved Hide resolved
src/modules/announcements/pages/Announcement.jsx Outdated Show resolved Hide resolved
src/modules/announcements/pages/Announcement.jsx Outdated Show resolved Hide resolved
@MaciekPon MaciekPon self-assigned this Feb 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants