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

Validation Error #3

Closed
asifpix opened this issue May 1, 2021 · 6 comments
Closed

Validation Error #3

asifpix opened this issue May 1, 2021 · 6 comments

Comments

@asifpix
Copy link
Contributor

asifpix commented May 1, 2021

Hi,

First of all thanks for the nice walker. I have used it in my WordPress theme and when I went to validate my site it gets validation error duplicate attributes. That is why it I came here to let you know.

When your menu depth is higher than 0 then it will get validation error.

Best Regards

@AlexWebLab
Copy link
Owner

AlexWebLab commented May 2, 2021

Hello there,
Thank you for your feedback. I never heard of this kind of error. Can you help me to reproduce it? If you tell me the exact steps to get it then I can fix it. If you have an online example it would be even better. Thanks.

@asifpix
Copy link
Contributor Author

asifpix commented May 3, 2021

Hi Alex,

First of create a menu which has sub menu item or dropdown item in Appearance > Menus
By creating a dropdown menu using your walker it will output two class. One is class = "dropdown-item" and another one is class = "nav-link"
But it should be output in a single class like this class = "dropdown-item nav-link"
Screenshot_1

I have fixed the error though. If you wish I can send a pull request of this fix.

Best Regards
Asif

@AlexWebLab
Copy link
Owner

Oh yeah I can see that now. I have an idea on how to fix it but I'm curious to see also your solution. Feel free to send a pull request. Thanks.

@asifpix
Copy link
Contributor Author

asifpix commented May 4, 2021

Thanks Alex, I have sent a pull request.
Best Regards

@AlexWebLab
Copy link
Owner

Thank you Asif, I've merged your pull request and fixed some class name conflicts.

@asifpix
Copy link
Contributor Author

asifpix commented May 6, 2021

You are most welcome Alex. Stay fine.

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

No branches or pull requests

2 participants