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

When using riot-viewBox inside an svg tag it is replaced by viewbox. #2086

Closed
jpeter06 opened this issue Nov 23, 2016 · 7 comments
Closed

When using riot-viewBox inside an svg tag it is replaced by viewbox. #2086

jpeter06 opened this issue Nov 23, 2016 · 7 comments

Comments

@jpeter06
Copy link

@jpeter06 jpeter06 commented Nov 23, 2016

  1. Describe your issue:
    When using riot-viewBox inside an svg tag
    it is replaced by viewbox (B is in lowercase).

  2. Can you reproduce the issue?
    https://jsfiddle.net/jpeter06/nhfabqj7/

  3. On which browser/OS does the issue appear?
    Chrome, IE, ...

  4. Which version of Riot does it affect?
    3.0.0

  5. How would you tag this issue?

  • Question
  • Bug
  • Discussion
  • Feature request
  • Tip
  • Enhancement
  • Performance
@GianlucaGuarini
Copy link
Member

@GianlucaGuarini GianlucaGuarini commented Nov 26, 2016

@jpeter06 just avoid the riot prefix it should be fine https://jsfiddle.net/gianlucaguarini/cjcb3oyw/

@jpeter06
Copy link
Author

@jpeter06 jpeter06 commented Nov 29, 2016

I was just trying to avoid the console error:
riot+compiler.min.js:2 Error: attribute viewBox: Expected number, "{viewBox}".

@GianlucaGuarini
Copy link
Member

@GianlucaGuarini GianlucaGuarini commented Nov 29, 2016

Ok this issue appears only on chrome, I think it should be really simple to solve

@sntran
Copy link

@sntran sntran commented Jan 3, 2017

This still throws an error in the console:

Error: <svg> attribute viewBox: Expected number, "{viewBox}".
@GianlucaGuarini
Copy link
Member

@GianlucaGuarini GianlucaGuarini commented Jan 9, 2017

@sntran i don't see any issue or error in the console browser + riot version?

@sntran
Copy link

@sntran sntran commented Jan 9, 2017

I was referencing the error in this https://jsfiddle.net/gianlucaguarini/cjcb3oyw/.

I can take a screen shot if you want. I'm on Chrome 55.0.2883.95 (64-bit) on El Capitan.

@GianlucaGuarini
Copy link
Member

@GianlucaGuarini GianlucaGuarini commented Jan 9, 2017

You can use riot-viewbox https://jsfiddle.net/cjcb3oyw/1/

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
3 participants
You can’t perform that action at this time.