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

dynamic import in riot tags #2641

Closed
1 of 7 tasks
Elastic1 opened this issue Nov 25, 2018 · 1 comment
Closed
1 of 7 tasks

dynamic import in riot tags #2641

Elastic1 opened this issue Nov 25, 2018 · 1 comment

Comments

@Elastic1
Copy link

@Elastic1 Elastic1 commented Nov 25, 2018

  1. Describe your issue:
    Dynamic import does not work in riot tag. it moved to global scope after compile.

  2. Can you reproduce the issue?
    https://next.plnkr.co/edit/h5mT5p4wIP0QWJ91

  3. On which browser/OS does the issue appear?
    Windows 10
    Chrome 70

  4. Which version of Riot does it affect?
    3.13.1

  5. How would you tag this issue?

  • Question
  • Bug
  • Discussion
  • Feature request ?
  • Tip
  • Enhancement
  • Performance
GianlucaGuarini added a commit to riot/compiler that referenced this issue Nov 25, 2018
GianlucaGuarini added a commit that referenced this issue Nov 25, 2018
@GianlucaGuarini
Copy link
Member

@GianlucaGuarini GianlucaGuarini commented Nov 25, 2018

fixed in riot@3.13.2 thank you

https://next.plnkr.co/edit/0Qggh9DBUuMpBc5L?preview

Loading

GianlucaGuarini added a commit to riot/compiler that referenced this issue Apr 19, 2019
* master: (43 commits)
  3.6.0
  added: changelog
  #closes riot/custom-elements#1
  3.5.3
  updated: changelog
  closes riot/custom-elements#1
  bump release
  3.5.2
  fixes riot/riot#2641
  fixed: riot urls
  update release
  3.5.1
  added: babel deprecation warning message
  updated: invert the babel parser import
  3.5.0
  updated: changelog
  updated: babel 7 hint
  update: handle also the newest babel versions
  updated: dependencies
  Update README.md
  ...
GianlucaGuarini added a commit that referenced this issue May 30, 2019
* master: (266 commits)
  added: link to the new site
  Update issue templates
  Update issue templates
  Delete ISSUE_TEMPLATE.md
  Update issue templates
  Update issue templates
  removed: dead links from readme file
  fixed: unit test on async test
  3.13.2
  updated: dev dependencies
  fixes #2641
  updated: lock file
  3.13.1
  updated: lib size
  Added link to chrome extension boilerplate repository
  fixes #2635
  updated: lock file
  added: polymer 3 lib size
  3.13.0
  updated: framework sizes
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants