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

Side navigation #467

Conversation

Elkrival
Copy link
Collaborator

@Elkrival Elkrival commented Nov 3, 2023

closes #436

This pr adds the stylings for the SideNavigation component.
This sets up the theme with the colors used in the figma designs.
It adds rendering tests and testing library set up.

Screenshot 2023-11-02 at 3 50 25 PM

* Added testing library
* Refactored Sidebar components and added tests
* Added color theme to material ui
* Updated Sidebar component into smaller pieces
* Added measurements stylings

* Cleaned up tests

* Renamed tests
* Added await syntax
* Renamed var
* Syntax
@JackVCurtis
Copy link

JackVCurtis commented Nov 3, 2023

@Elkrival CI failure is an easy fix. In package.json set:

  "jest": {
    "setupFilesAfterEnv": [
      "<rootDir>/jest-setup.js"
    ],
    "moduleNameMapper": {
      "\\.(css|less|scss|sass)$": "identity-obj-proxy"
    }
  },

and in jest-setup.js just put:

import '@testing-library/jest-dom'

Then delete the jest-dom import from Sidebar.test.jsx

* updated jest preferences
* Fixed test
Copy link

@JackVCurtis JackVCurtis left a comment

Choose a reason for hiding this comment

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

🚢

@Elkrival Elkrival merged commit 319eb8c into 428-update-material-ui-version-to-the-latest-stable-version Nov 3, 2023
3 checks passed
@Elkrival Elkrival deleted the side-navigation branch November 3, 2023 13:41
Elkrival added a commit that referenced this pull request Nov 8, 2023
* Sidebar

* Added testing library
* Refactored Sidebar components and added tests
* Added color theme to material ui
* Updated Sidebar component into smaller pieces
* Added measurements stylings

* Cleaned up tests

* Renamed tests

* Await

* Added await syntax

* Var

* Renamed var
* Syntax

* js dom set up

* updated jest preferences
* Fixed test
Elkrival added a commit that referenced this pull request Nov 8, 2023
* Sidebar

* Added testing library
* Refactored Sidebar components and added tests
* Added color theme to material ui
* Updated Sidebar component into smaller pieces
* Added measurements stylings

* Cleaned up tests

* Renamed tests

* Await

* Added await syntax

* Var

* Renamed var
* Syntax

* js dom set up

* updated jest preferences
* Fixed test
Elkrival added a commit that referenced this pull request Nov 8, 2023
* Sidebar

* Added testing library
* Refactored Sidebar components and added tests
* Added color theme to material ui
* Updated Sidebar component into smaller pieces
* Added measurements stylings

* Cleaned up tests

* Renamed tests

* Await

* Added await syntax

* Var

* Renamed var
* Syntax

* js dom set up

* updated jest preferences
* Fixed test
Elkrival added a commit that referenced this pull request Nov 8, 2023
* Login Page

* Added Login Page UI designs

3Fixed Test
Elkrival added a commit that referenced this pull request Nov 8, 2023
* Login Page

* Added Login Page UI designs

3Fixed Test
Elkrival added a commit that referenced this pull request Nov 27, 2023
* Login Page

* Added Login Page UI designs

3Fixed Test
Elkrival added a commit that referenced this pull request Nov 27, 2023
* Login Page

* Added Login Page UI designs

3Fixed Test
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