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

Fix drawer toggle #3191

Merged
merged 5 commits into from
Jan 6, 2018
Merged

Conversation

gov-ind
Copy link

@gov-ind gov-ind commented Dec 27, 2017

Drawer does not toggle as per this issue. this._drawer.state.drawerShown seems to be available only on iOS?

Test plan (required)

Works on both iOS and Android (Issue didn't exist in iOS in the first place)

Copy link

@GantMan GantMan left a comment

Choose a reason for hiding this comment

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

Always use triple equals, please. If you can fix that and the merge conflict, I think we're good.

@GantMan
Copy link

GantMan commented Jan 5, 2018

I have the code updated --- so just the double equals to a triple (our JavaScript curse) please.

@codecov-io
Copy link

codecov-io commented Jan 5, 2018

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3191   +/-   ##
=======================================
  Coverage   71.88%   71.88%           
=======================================
  Files          51       51           
  Lines        1519     1519           
  Branches      382      382           
=======================================
  Hits         1092     1092           
  Misses        337      337           
  Partials       90       90
Impacted Files Coverage Δ
src/views/Drawer/DrawerView.js 31.7% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7a57c4e...90afcb3. Read the comment docs.

@gov-ind
Copy link
Author

gov-ind commented Jan 6, 2018

Changed double equals to a triple in fix_drawer_toggle branch.

Copy link

@GantMan GantMan left a comment

Choose a reason for hiding this comment

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

Thanks for putting up with me :)

Looks good

@GantMan GantMan merged commit e2e540c into react-navigation:master Jan 6, 2018
tzkoshi pushed a commit to tzkoshi/react-navigation that referenced this pull request Jan 22, 2018
* Fixed drawer toggle

* Fixed drawer toggle

* Update DrawerView.js
sourcecode911 pushed a commit to sourcecode911/react-navigation that referenced this pull request Mar 9, 2020
* Fixed drawer toggle

* Fixed drawer toggle

* Update DrawerView.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants