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

BREAKING: Drawer redesign #377

Merged

Conversation

gawrysiak
Copy link
Collaborator

@gawrysiak gawrysiak commented May 16, 2018

Drawer redesign for 2.0. Changes:

  • changed the active state styling
  • colors are now theme based, not fixed
  • theme based roundness
  • removed the "color" prop - it can be now controller with the primaryColor (see the expo app example)
  • added snapshot tests

screen shot 2018-05-16 at 23 59 54

screen shot 2018-05-17 at 00 00 02

Motivation

Fixes #358.

Test plan

Snapshot tests added. Run yarn test.

@callstack-bot
Copy link

Hey @gawrysiak, thank you for your pull request 🤗. The documentation from this branch can be viewed here.

Copy link
Member

@satya164 satya164 left a comment

Choose a reason for hiding this comment

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

Looks great!

@satya164 satya164 merged commit 6b3ef91 into callstack:material-next May 20, 2018
@gawrysiak gawrysiak deleted the breaking/drawer-redesign branch May 20, 2018 17:03
satya164 pushed a commit that referenced this pull request May 24, 2018
Drawer redesign for 2.0. Changes:
- changed the active state styling
- colors are now theme based, not fixed
- theme based roundness
- removed the "color" prop - it can be now controller with the primaryColor (see [the expo app example](https://github.com/callstack/react-native-paper/pull/377/files#diff-cf8d6e3323746c5c486de08cb0c333c3R59))
- added snapshot tests

<img width="248" alt="screen shot 2018-05-16 at 23 59 54" src="https://user-images.githubusercontent.com/7827311/40146708-e58bf92c-5966-11e8-8916-e76cb42e2f86.png">
<img width="248" alt="screen shot 2018-05-17 at 00 00 02" src="https://user-images.githubusercontent.com/7827311/40146727-f26662a4-5966-11e8-9cee-dc01e3cd5599.png">

Fixes #358.

Snapshot tests added. Run `yarn test`.
satya164 pushed a commit that referenced this pull request May 25, 2018
Drawer redesign for 2.0. Changes:
- changed the active state styling
- colors are now theme based, not fixed
- theme based roundness
- removed the "color" prop - it can be now controller with the primaryColor (see [the expo app example](https://github.com/callstack/react-native-paper/pull/377/files#diff-cf8d6e3323746c5c486de08cb0c333c3R59))
- added snapshot tests

<img width="248" alt="screen shot 2018-05-16 at 23 59 54" src="https://user-images.githubusercontent.com/7827311/40146708-e58bf92c-5966-11e8-8916-e76cb42e2f86.png">
<img width="248" alt="screen shot 2018-05-17 at 00 00 02" src="https://user-images.githubusercontent.com/7827311/40146727-f26662a4-5966-11e8-9cee-dc01e3cd5599.png">

Fixes #358.

Snapshot tests added. Run `yarn test`.
satya164 pushed a commit that referenced this pull request May 25, 2018
Drawer redesign for 2.0. Changes:
- changed the active state styling
- colors are now theme based, not fixed
- theme based roundness
- removed the "color" prop - it can be now controller with the primaryColor (see [the expo app example](https://github.com/callstack/react-native-paper/pull/377/files#diff-cf8d6e3323746c5c486de08cb0c333c3R59))
- added snapshot tests

<img width="248" alt="screen shot 2018-05-16 at 23 59 54" src="https://user-images.githubusercontent.com/7827311/40146708-e58bf92c-5966-11e8-8916-e76cb42e2f86.png">
<img width="248" alt="screen shot 2018-05-17 at 00 00 02" src="https://user-images.githubusercontent.com/7827311/40146727-f26662a4-5966-11e8-9cee-dc01e3cd5599.png">

Fixes #358.

Snapshot tests added. Run `yarn test`.
satya164 pushed a commit that referenced this pull request May 25, 2018
Drawer redesign for 2.0. Changes:
- changed the active state styling
- colors are now theme based, not fixed
- theme based roundness
- removed the "color" prop - it can be now controller with the primaryColor (see [the expo app example](https://github.com/callstack/react-native-paper/pull/377/files#diff-cf8d6e3323746c5c486de08cb0c333c3R59))
- added snapshot tests

<img width="248" alt="screen shot 2018-05-16 at 23 59 54" src="https://user-images.githubusercontent.com/7827311/40146708-e58bf92c-5966-11e8-8916-e76cb42e2f86.png">
<img width="248" alt="screen shot 2018-05-17 at 00 00 02" src="https://user-images.githubusercontent.com/7827311/40146727-f26662a4-5966-11e8-9cee-dc01e3cd5599.png">

Fixes #358.

Snapshot tests added. Run `yarn test`.
satya164 pushed a commit that referenced this pull request May 26, 2018
Drawer redesign for 2.0. Changes:
- changed the active state styling
- colors are now theme based, not fixed
- theme based roundness
- removed the "color" prop - it can be now controller with the primaryColor (see [the expo app example](https://github.com/callstack/react-native-paper/pull/377/files#diff-cf8d6e3323746c5c486de08cb0c333c3R59))
- added snapshot tests

<img width="248" alt="screen shot 2018-05-16 at 23 59 54" src="https://user-images.githubusercontent.com/7827311/40146708-e58bf92c-5966-11e8-8916-e76cb42e2f86.png">
<img width="248" alt="screen shot 2018-05-17 at 00 00 02" src="https://user-images.githubusercontent.com/7827311/40146727-f26662a4-5966-11e8-9cee-dc01e3cd5599.png">

Fixes #358.

Snapshot tests added. Run `yarn test`.
eriveltonelias pushed a commit to eriveltonelias/react-native-paper that referenced this pull request Sep 21, 2018
Drawer redesign for 2.0. Changes:
- changed the active state styling
- colors are now theme based, not fixed
- theme based roundness
- removed the "color" prop - it can be now controller with the primaryColor (see [the expo app example](https://github.com/callstack/react-native-paper/pull/377/files#diff-cf8d6e3323746c5c486de08cb0c333c3R59))
- added snapshot tests

<img width="248" alt="screen shot 2018-05-16 at 23 59 54" src="https://user-images.githubusercontent.com/7827311/40146708-e58bf92c-5966-11e8-8916-e76cb42e2f86.png">
<img width="248" alt="screen shot 2018-05-17 at 00 00 02" src="https://user-images.githubusercontent.com/7827311/40146727-f26662a4-5966-11e8-9cee-dc01e3cd5599.png">

Fixes callstack#358.

Snapshot tests added. Run `yarn test`.
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.

3 participants