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

embedded CSS regression riot@5 to riot@6 and later #2972

Closed
2 of 7 tasks
qlbit-com opened this issue Nov 3, 2022 · 4 comments
Closed
2 of 7 tasks

embedded CSS regression riot@5 to riot@6 and later #2972

qlbit-com opened this issue Nov 3, 2022 · 4 comments

Comments

@qlbit-com
Copy link

  1. Describe your issue:

I'm having strange regression in compiled code which has embedded CSS.
It works when compiler riot@5 is used, and fails with compiler riot@6 , see plunker below.

code is a menu component which uses onMouseOver to change the display style on sibling element to show\hide nested menu items.

Please advise if that is improper technique.

  1. Can you reproduce the issue?
    https://plnkr.co/edit/1GSWeSbs47tSMUHg

  2. On which browser/OS does the issue appear?
    win10\Chrome , win10\Edge, win10\Opera

  3. Which version of Riot does it affect?
    works with riot5 compiler, fails with riot6 compiler.

  4. How would you tag this issue?

  • Question
  • Bug\regression
  • Discussion
  • Feature request
  • Tip
  • Enhancement
  • Performance
@qlbit-com
Copy link
Author

to reproduce in plunker in index.html change:
<script type="text/javascript" src="https://unpkg.com/riot@5/riot+compiler.min.js"></script>
to
<script type="text/javascript" src="https://unpkg.com/riot@7/riot+compiler.min.js"></script>

@qlbit-com
Copy link
Author

when testing with webpack:
regression observed between compiler version 5.4.2 and 6.0.0,
all later versions of the compiler between 6.0.0 and 6.4.1 showing this issue.

@GianlucaGuarini
Copy link
Member

Thank you for reporting it. This issue is related to #2967

GianlucaGuarini added a commit to riot/compiler that referenced this issue Nov 4, 2022
@GianlucaGuarini
Copy link
Member

Fixed in riot@7.1.0 thank you for submitting this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants