Skip to content

Comments

Change doc for SectionedCardList#431

Open
djhuistra wants to merge 1 commit intogabrielemariotti:masterfrom
djhuistra:master
Open

Change doc for SectionedCardList#431
djhuistra wants to merge 1 commit intogabrielemariotti:masterfrom
djhuistra:master

Conversation

@djhuistra
Copy link

The documentation for using SectionedCardList was confusing for me.

It uses "GplayCardSection" and "MyCardSection" in the examples before it introduces (how to) extend the default "CardSection".

At first I believed "GplayCardSection" was part of the library and was confused as to why the example did not work.

By using the default "CardSection" in the examples, the code is directly runnable, which I believe is preferred :)

The documentation for using SectionedCardList was confusing for me.

It used "GplayCardSection"  and "MyCardSection" in the examples before it introduces (how to) extend the "CardSection".

At first I believed "GplayCardSection" was part of the library and was confused as to why the example did not work.

By using the default "CardSection" in the examples, the code is directly runnable, which I believe is preferred :)
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.

1 participant