Skip to content

no navigation-level0-item__hover__color #15848

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

Closed
DanielRuf opened this issue Jun 5, 2018 · 8 comments
Closed

no navigation-level0-item__hover__color #15848

DanielRuf opened this issue Jun 5, 2018 · 8 comments
Assignees
Labels
Fixed in 2.1.x The issue has been fixed in 2.1 release line Fixed in 2.2.x The issue has been fixed in 2.2 release line Fixed in 2.3.x The issue has been fixed in 2.3 release line good first issue Issue: Clear Description Gate 2 Passed. Manual verification of the issue description passed Issue: Confirmed Gate 3 Passed. Manual verification of the issue completed. Issue is confirmed Issue: Format is not valid Gate 1 Failed. Automatic verification of issue format is failed Reproduced on 2.2.x The issue has been reproduced on latest 2.2 release Reproduced on 2.3.x The issue has been reproduced on latest 2.3 release

Comments

@DanielRuf
Copy link
Contributor

DanielRuf commented Jun 5, 2018

We have @navigation-level0-item__active__color but no @navigation-level0-item__hover__color

Keep in mind that we also have hover on some devices and screens (resized browser window).

@magento-engcom-team magento-engcom-team added the Issue: Format is not valid Gate 1 Failed. Automatic verification of issue format is failed label Jun 5, 2018
@engcom-backlog-nickolas engcom-backlog-nickolas added Reproduced on 2.2.x The issue has been reproduced on latest 2.2 release Reproduced on 2.3.x The issue has been reproduced on latest 2.3 release Issue: Clear Description Gate 2 Passed. Manual verification of the issue description passed feature request labels Jul 11, 2018
@engcom-backlog-nickolas

Hello @DanielRuf, thank you for your report.
GitHub is intended for bug reports, please use https://github.com/magento/community-features for other suggestions like improvements or feature requests.

@engcom-backlog-nickolas engcom-backlog-nickolas removed their assignment Jul 11, 2018
@engcom-backlog-nickolas engcom-backlog-nickolas added the Issue: Confirmed Gate 3 Passed. Manual verification of the issue completed. Issue is confirmed label Jul 11, 2018
@DanielRuf
Copy link
Contributor Author

This is an actual bugreport. There is a missing variable.

@DanielRuf DanielRuf reopened this Jul 11, 2018
@DanielRuf
Copy link
Contributor Author

I can also provide a PR then if no other developer wants to contribute to solve this easy issue.

@engcom-backlog-nickolas

@DanielRuf, thank you for your feedback. You are right, my bad.

@hitesh-wagento
Copy link
Contributor

I am working on this PR

@hitesh-wagento hitesh-wagento self-assigned this Jul 11, 2018
@magento-engcom-team magento-engcom-team added the Fixed in 2.2.x The issue has been fixed in 2.2 release line label Jul 13, 2018
@magento-engcom-team
Copy link
Contributor

Hi @DanielRuf. Thank you for your report.
The issue has been fixed in #16732 by @hitesh-wagento in 2.2-develop branch
Related commit(s):

The fix will be available with the upcoming 2.2.6 release.

hitesh-wagento added a commit to hitesh-wagento/magento2 that referenced this issue Jul 14, 2018
@sidolov
Copy link
Contributor

sidolov commented Jul 18, 2018

Hi @DanielRuf. Thank you for your report.
The issue has been fixed in #16797 by @hitesh-wagento in 2.1-develop branch
Related commit(s):

The fix will be available with the upcoming 2.1.15 release.

@sidolov sidolov added the Fixed in 2.1.x The issue has been fixed in 2.1 release line label Jul 18, 2018
magento-engcom-team added a commit that referenced this issue Jul 18, 2018
…color #15848 #16797

 - Merge Pull Request #16797 from hitesh-wagento/magento2:2.1-develop-PR-port-16732
 - Merged commits:
   1. faa9815
@magento-engcom-team
Copy link
Contributor

Hi @DanielRuf. Thank you for your report.
The issue has been fixed in #16796 by @hitesh-wagento in 2.3-develop branch
Related commit(s):

The fix will be available with the upcoming 2.3.0 release.

@magento-engcom-team magento-engcom-team added the Fixed in 2.3.x The issue has been fixed in 2.3 release line label Jul 20, 2018
magento-engcom-team added a commit that referenced this issue Jul 20, 2018
…r__color #15848 #16796

 - Merge Pull Request #16796 from hitesh-wagento/magento2:2.3-develop-PR-port-16732
 - Merged commits:
   1. 85caf63
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fixed in 2.1.x The issue has been fixed in 2.1 release line Fixed in 2.2.x The issue has been fixed in 2.2 release line Fixed in 2.3.x The issue has been fixed in 2.3 release line good first issue Issue: Clear Description Gate 2 Passed. Manual verification of the issue description passed Issue: Confirmed Gate 3 Passed. Manual verification of the issue completed. Issue is confirmed Issue: Format is not valid Gate 1 Failed. Automatic verification of issue format is failed Reproduced on 2.2.x The issue has been reproduced on latest 2.2 release Reproduced on 2.3.x The issue has been reproduced on latest 2.3 release
Projects
None yet
Development

No branches or pull requests

5 participants