Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

fix($location): set baseHref in mock browser to / - #8889

Closed
tbosch wants to merge 1 commit into
angular:masterfrom
tbosch:locationmock
Closed

tbosch wants to merge 1 commit into
angular:masterfrom
tbosch:locationmock

Conversation

@tbosch

@tbosch tbosch commented Sep 2, 2014

Copy link
Copy Markdown
Contributor

Set the default value for the base tag in the mock browser to /,
as we now always require a base tag to be present for html5 mode.

Fixes #8866.

Set the default value for the base tag in the mock browser to `/`,
as we now always require a base tag to be present for html5 mode.

Fixes angular#8866.
@tbosch

tbosch commented Sep 2, 2014

Copy link
Copy Markdown
Contributor Author

@jeffbcross Could you review this?

@jeffbcross jeffbcross self-assigned this Sep 2, 2014
@jeffbcross

Copy link
Copy Markdown
Contributor

@tbosch sure

@jeffbcross jeffbcross added this to the 1.3.0-rc.1 milestone Sep 2, 2014
@jeffbcross

Copy link
Copy Markdown
Contributor

LGTM if Travis is happy

@tbosch tbosch closed this Sep 2, 2014
@tbosch
tbosch deleted the locationmock branch September 2, 2014 16:54
tbosch added a commit that referenced this pull request Sep 2, 2014
Set the default value for the base tag in the mock browser to `/`,
as we now always require a base tag to be present for html5 mode.

Fixes #8866
Closes #8889
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Angular.js 1.3-rc0 HTML5 Location and base tags

2 participants