Skip to content

adding get_cell function to GridLayout #46

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

Merged
merged 3 commits into from
Aug 21, 2021
Merged

Conversation

FoamyGuy
Copy link
Contributor

@FoamyGuy FoamyGuy commented Aug 6, 2021

resolves #43

Adding a new get_cell() function that allows you to lookup cell content objects by their coordinates within the GridLayout. Added a new example to illustrate this functionality.

@flavio-fernandes
Copy link

LGTM. Also may be nice to include link to the super awesome recording that shows all the steps performed.
https://youtu.be/ED0luF_q3ec

Copy link
Contributor

@rsbohn rsbohn left a comment

Choose a reason for hiding this comment

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

Suggest using grid position in _cell_content_list instead of adding _cell_dictionary.

Copy link
Contributor

@rsbohn rsbohn left a comment

Choose a reason for hiding this comment

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

I like it!

@flavio-fernandes
Copy link

I like it!

Me too. Thanks again for your time going over this code today @FoamyGuy !

@flavio-fernandes
Copy link

/lgtm

@FoamyGuy FoamyGuy merged commit ca714e5 into adafruit:main Aug 21, 2021
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Aug 22, 2021
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.

Provide a way to lookup elements by x,y grid position in GridLayout
3 participants