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

[Tabs] Expand configuration in MDCTabBarView example. #8498

Merged
merged 1 commit into from
Sep 25, 2019
Merged

[Tabs] Expand configuration in MDCTabBarView example. #8498

merged 1 commit into from
Sep 25, 2019

Conversation

romoore
Copy link
Contributor

@romoore romoore commented Sep 25, 2019

Adds configuration options for preferredLayoutStyle and toggling
contentInset.

Fewer Tabs (Portrait) More Tabs (Landscape)
Simulator Screen Shot - iPhone 7 - 2019-09-25 at 09 12 25 Simulator Screen Shot - iPhone 7 - 2019-09-25 at 09 15 47

Helps exercise #8236

Adds configuration options for `preferredLayoutStyle` and toggling
`contentInset`.

Helps exercise #8236
@material-automation
Copy link

material-automation commented Sep 25, 2019

bazel detected changes to the following targets:

//components/AppBar:SwiftExamples
//components/Tabs:Bundle
//components/Tabs:BundleFiles
//components/Tabs:ColorThemer
//components/Tabs:FontThemer
//components/Tabs:ObjcExamples
//components/Tabs:SwiftExamples
//components/Tabs:TabBarView
//components/Tabs:Tabs
//components/Tabs:TabsExamplesAssets
//components/Tabs:Theming
//components/Tabs:TypographyThemer
//components/Tabs:autobot
//components/Tabs:kokoro
//components/Tabs:private
//components/Tabs:privateTabBarView
//components/Tabs:snapshot_test_lib
//components/Tabs:snapshot_tests
//components/Tabs:snapshot_tests.swift_runtime_linkopts
//components/Tabs:snapshot_tests_test_binary
//components/Tabs:snapshot_tests_test_bundle
//components/Tabs:unit_test_sources
//components/Tabs:unit_tests
//components/Tabs:unit_tests_IPAD_PRO_12_9_IN_9_3
//components/Tabs:unit_tests_IPAD_PRO_12_9_IN_9_3.swift_runtime_linkopts
//components/Tabs:unit_tests_IPAD_PRO_12_9_IN_9_3_test_binary
//components/Tabs:unit_tests_IPAD_PRO_12_9_IN_9_3_test_bundle
//components/Tabs:unit_tests_IPHONE_7_PLUS_IN_10_3
//components/Tabs:unit_tests_IPHONE_7_PLUS_IN_10_3.swift_runtime_linkopts
//components/Tabs:unit_tests_IPHONE_7_PLUS_IN_10_3_test_binary
//components/Tabs:unit_tests_IPHONE_7_PLUS_IN_10_3_test_bundle
//components/Tabs:unit_tests_environment
//components/Tabs:unit_tests_environment.swift_runtime_linkopts
//components/Tabs:unit_tests_environment_test_binary
//components/Tabs:unit_tests_environment_test_bundle

Copy link
Contributor

@jverkoey jverkoey left a comment

Choose a reason for hiding this comment

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

Nice!

@romoore romoore merged commit 1ce82c6 into material-components:develop Sep 25, 2019
@romoore romoore deleted the issue-8236-example branch September 25, 2019 14:57
romoore pushed a commit that referenced this pull request Sep 25, 2019
Some icon images were not being tested. Adds those missing tests.

Found while working on #8498
romoore pushed a commit that referenced this pull request Sep 25, 2019
Icons were not fully integrated into the catalog project and the relevant
scripts were out of date. This PR addressed the following issues:

*   The Icons source files had been hand-modified and did not match the
    script outputs.
*   The Icons unit tests were not included in the Catalog or Dragons
    projects.

It addresses these issues through the following:

*   Updates `scripts/sync_icons.sh` to:
    -   Match the copyright header format used by the project.
    -   Run clang-format on each generated source file.
    -   Add Icons unit tests to the MaterialComponents podspec.
    -   Remove duplicate import of `MaterialIcons.h` in the source
        implementation files.
*   Adds the Icons unit test targets to the Catalog and Dragons Xcode schemes.

Discovered while working on #8498
ggdiez pushed a commit to solidgear/material-components-ios that referenced this pull request Sep 27, 2019
…nents#8498)

Adds configuration options for `preferredLayoutStyle` and toggling
`contentInset`.

|Fewer Tabs (Portrait)|More Tabs (Landscape)|
|---|---|
|![Simulator Screen Shot - iPhone 7 - 2019-09-25 at 09 12 25](https://user-images.githubusercontent.com/1753199/65605352-f3260180-df76-11e9-8fcd-2cd68ec30fa3.png)|![Simulator Screen Shot - iPhone 7 - 2019-09-25 at 09 15 47](https://user-images.githubusercontent.com/1753199/65605434-181a7480-df77-11e9-993d-1a9c4e7a68e0.png)|


Helps exercise #8236
ggdiez pushed a commit to solidgear/material-components-ios that referenced this pull request Sep 27, 2019
Some icon images were not being tested. Adds those missing tests.

Found while working on material-components#8498
ggdiez pushed a commit to solidgear/material-components-ios that referenced this pull request Sep 27, 2019
)

Icons were not fully integrated into the catalog project and the relevant
scripts were out of date. This PR addressed the following issues:

*   The Icons source files had been hand-modified and did not match the
    script outputs.
*   The Icons unit tests were not included in the Catalog or Dragons
    projects.

It addresses these issues through the following:

*   Updates `scripts/sync_icons.sh` to:
    -   Match the copyright header format used by the project.
    -   Run clang-format on each generated source file.
    -   Add Icons unit tests to the MaterialComponents podspec.
    -   Remove duplicate import of `MaterialIcons.h` in the source
        implementation files.
*   Adds the Icons unit test targets to the Catalog and Dragons Xcode schemes.

Discovered while working on material-components#8498
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants