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

Update step_03.ngdoc #6985

Closed
wants to merge 1 commit into from
Closed

Update step_03.ngdoc #6985

wants to merge 1 commit into from

Conversation

yilinglu
Copy link
Contributor

@yilinglu yilinglu commented Apr 3, 2014

Need to remove this single space for the regex to work here.

Need to remove this single space for the regex to work here.
@mary-poppins
Copy link

Thanks for the PR! Please check the items below to help us merge this faster. See the contributing docs for more information.

  • Uses the issue template (#6985)

If you need to make changes to your pull request, you can update the commit with git commit --amend.
Then, update the pull request with git push -f.

Thanks again for your help!

@yilinglu yilinglu added cla: yes and removed cla: no labels Apr 4, 2014
@caitp
Copy link
Contributor

caitp commented Apr 4, 2014

It is weird, but you're correct. @juliemr does getText() trim the text value or something? It's super weird!

@caitp caitp closed this in 8b0b7ca Apr 4, 2014
caitp pushed a commit that referenced this pull request Apr 4, 2014
Need to remove this single space for the regex to work here.

Apparently `getText()` is trimming the text content or something, because there is no good reason
why that space should not be there.

Closes #6985
@juliemr
Copy link
Member

juliemr commented Apr 4, 2014

Sorry for not catching this - you're correct, WebDriver trims whitespace when it gets text values! From the webdriver spec, "All WebDriver implementations must support getting the visible text of a WebElement, with excess whitespace compressed."

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

Successfully merging this pull request may close these issues.

None yet

4 participants