X Tutup
Skip to content

fix(test): "integration tests svg should support svg elements" fails …#5000

Closed
marclaval wants to merge 1 commit intoangular:masterfrom
marclaval:issue4987
Closed

fix(test): "integration tests svg should support svg elements" fails …#5000
marclaval wants to merge 1 commit intoangular:masterfrom
marclaval:issue4987

Conversation

@marclaval
Copy link
Copy Markdown
Contributor

…in non-Chrome browsers

Fixes #4987

If a Dart guru has a better idea to test the namespaceURI of the attribute, please let me know.

@pkozlowski-opensource
Copy link
Copy Markdown
Member

I'm definitively not a "Dart guru" but I would rather use if(isDart) instead of try / catch

@marclaval marclaval added the action: review The PR is still awaiting reviews from at least one requested reviewer label Oct 29, 2015
@marclaval
Copy link
Copy Markdown
Contributor Author

Indeed, done

@vicb vicb added pr_state: LGTM action: merge The PR is ready for merge by the caretaker labels Oct 29, 2015
@mary-poppins mary-poppins removed action: merge The PR is ready for merge by the caretaker labels Oct 29, 2015
@mary-poppins
Copy link
Copy Markdown

Merging PR #5000 on behalf of @vicb to branch presubmit-vicb-pr-5000.

1 similar comment
@mary-poppins
Copy link
Copy Markdown

Merging PR #5000 on behalf of @vicb to branch presubmit-vicb-pr-5000.

@marclaval marclaval closed this in c4964e7 Oct 29, 2015
@marclaval marclaval deleted the issue4987 branch December 11, 2017 10:07
@angular-automatic-lock-bot
Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: review The PR is still awaiting reviews from at least one requested reviewer cla: yes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update tests for FF, Safari, and IE to account for renderer changes in SVG

6 participants

X Tutup