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

Tabs Error in the Console on Home page when logged in #584

Open
eleanorreem opened this issue Sep 7, 2023 · 3 comments
Open

Tabs Error in the Console on Home page when logged in #584

eleanorreem opened this issue Sep 7, 2023 · 3 comments
Labels
bug fix Fix a bug complexity: moderate Time needed to do this ticket will be moderate e.g. 1-2 days help wanted Extra attention is needed

Comments

@eleanorreem
Copy link
Contributor

Describe the bug
When you are logged in and on the home page, you get an error in the console.

next-dev.js:20 MUI: The `value` provided to the Tabs component is invalid.
None of the Tabs' children match with "-1".
You can provide one of the following values: 0, 1, 2, 3, 4.

This comes from the SecondaryNav component. When we are on the home page, we are not in any of the tabs and the value is set as -1. -1 is not an index in the tabs.

To Reproduce

  1. Go to / in bloom
  2. Click on Login in the header
  3. Log in with credentials/ create an account
  4. Once logged in return to the home page
  5. Look at the console and you will see the error

Expected behavior
We have no error in the console. Ideally we would not set a value at all. If you look at these [docs](https://mui.com/material-ui/api/tabs/) - it looks like you can set the value to false and it will select nothing and we won’t get the error.

Desktop (please complete the following information):
All OS and Browsers

@eleanorreem eleanorreem added bug fix Fix a bug complexity: moderate Time needed to do this ticket will be moderate e.g. 1-2 days labels Sep 7, 2023
@kyleecodes kyleecodes added the hacktoberfest Hacktoberfest issues label Oct 3, 2023
@IanWorley
Copy link
Contributor

Hey I know I have another issue assign to me but could I take this one also.

@kyleecodes
Copy link
Member

Thanks for your interest @IanWorley! You can have multiple issues as long as they are not both no-code hacktober issues in 1 repo. I've assigned you this issue. Please let us know if you have any questions!

@IanWorley
Copy link
Contributor

Hey can someone take me off this issue I had something come up.

@kyleecodes kyleecodes removed the hacktoberfest Hacktoberfest issues label Nov 3, 2023
@kyleecodes kyleecodes added the help wanted Extra attention is needed label Mar 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug fix Fix a bug complexity: moderate Time needed to do this ticket will be moderate e.g. 1-2 days help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants