Skip to content

Add drawerGestureLock to DrawerLayoutAndroid #5880

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

Closed
wants to merge 1 commit into from
Closed

Add drawerGestureLock to DrawerLayoutAndroid #5880

wants to merge 1 commit into from

Conversation

avishayil
Copy link

Useful for developers that would like to lock the drawer, closed or
opened (or unlocked by default) until drawer command is executed.

Useful for developers that would like to lock the drawer, closed or
opened (or unlocked by default) until drawer command is executed.
@@ -57,6 +59,7 @@ var DRAWER_STATES = [
* <DrawerLayoutAndroid
* drawerWidth={300}

Choose a reason for hiding this comment

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

no-trailing-spaces: Trailing spaces not allowed.

@facebook-github-bot
Copy link
Contributor

@avishayil updated the pull request.

@facebook-github-bot
Copy link
Contributor

@avishayil updated the pull request.

@satya164
Copy link
Contributor

cc @mkonicek

@@ -105,8 +110,11 @@ public void receiveCommand(
@Override
public @Nullable Map getExportedViewConstants() {
return MapBuilder.of(
"DrawerPosition",
MapBuilder.of("Left", Gravity.START, "Right", Gravity.END));
"DrawerPosition",
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: Please keep the original indentation of 4 spaces.

@mkonicek
Copy link
Contributor

What happens if the drawer is closed and the prop is set to LockedOpen, and vice versa? Should this be documented?

@mkonicek
Copy link
Contributor

Just saw a very similar PR: #5534

@satya164
Copy link
Contributor

Closing since the other PR is now accepted.

@satya164 satya164 closed this Feb 24, 2016
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