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

Feature/#229 wait no element #249

Merged
merged 5 commits into from
Mar 7, 2019
Merged

Feature/#229 wait no element #249

merged 5 commits into from
Mar 7, 2019

Conversation

ziflex
Copy link
Member

@ziflex ziflex commented Mar 5, 2019

Closes #229

Added

  • WAIT_NO_ELEMENT
  • WAIT_NO_CLASS
  • WAIT_NO_CLASS_ALL

@ziflex ziflex added area/stdlib Standard library issue area/drivers HTML drivers labels Mar 5, 2019
@ziflex ziflex requested a review from 3timeslazy March 5, 2019 22:39
@codecov
Copy link

codecov bot commented Mar 5, 2019

Codecov Report

Merging #249 into master will not change coverage.
The diff coverage is 0%.

@@          Coverage Diff           @@
##           master    #249   +/-   ##
======================================
  Coverage    60.1%   60.1%           
======================================
  Files         179     179           
  Lines        5296    5296           
======================================
  Hits         3184    3184           
  Misses       1763    1763           
  Partials      349     349

}
)

const (
WaitEventPresence = 0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add comments for WaitEventPresenceand WaitEventAbsence pls :)

}
}

if found == values.False {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add comment explaining why values.True returns here.

@ziflex ziflex merged commit d0caef8 into master Mar 7, 2019
@ziflex ziflex deleted the feature/#229-WAIT_NO_ELEMENT branch March 7, 2019 02:52
3timeslazy pushed a commit to 3timeslazy/ferret that referenced this pull request Apr 10, 2019
* Added possibility to wait for an element or a class absence
3timeslazy pushed a commit to 3timeslazy/ferret that referenced this pull request Apr 10, 2019
* Added possibility to wait for an element or a class absence
3timeslazy pushed a commit to 3timeslazy/ferret that referenced this pull request Apr 10, 2019
* Added possibility to wait for an element or a class absence
3timeslazy pushed a commit to 3timeslazy/ferret that referenced this pull request Apr 10, 2019
* Added possibility to wait for an element or a class absence
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/drivers HTML drivers area/stdlib Standard library issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants