Skip to content

Inserter - have the RecyclerView update and re-draw item in onChange#111

Merged
mzorz merged 1 commit intofeature/inserterfrom
fix/block-height-not-refreshed
Aug 14, 2018
Merged

Inserter - have the RecyclerView update and re-draw item in onChange#111
mzorz merged 1 commit intofeature/inserterfrom
fix/block-height-not-refreshed

Conversation

@mzorz
Copy link
Copy Markdown
Contributor

@mzorz mzorz commented Aug 14, 2018

After the change introduced in #108, the first time the RecyclerView renders its items these will all have the same height, rendering some of the blocks "cut".

BEFORE
screen shot 2018-08-14 at 11 21 29

AFTER
screen shot 2018-08-14 at 11 22 09

This PR addresses this by making sure to set these items dirty, signaling the RecyclerView to re-draw them as onChange is received.

@mzorz mzorz requested a review from hypest August 14, 2018 14:23
Copy link
Copy Markdown
Contributor

@hypest hypest left a comment

Choose a reason for hiding this comment

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

LGTM!

@hypest
Copy link
Copy Markdown
Contributor

hypest commented Aug 14, 2018

The flow error in this PR seems to be present in feature/inserter anyway so, to keep this PR the small one it is, let's fix the error in one of the follow up Inserter PRs @mzorz , wdyt?

@mzorz
Copy link
Copy Markdown
Contributor Author

mzorz commented Aug 14, 2018

The flow error in this PR seems to be present in feature/inserter anyway so, to keep this PR the small one it is, let's fix the error in one of the follow up Inserter PRs @mzorz , wdyt?

Agreed! Will try and address shortly 🙇

@mzorz mzorz merged commit 7b7429b into feature/inserter Aug 14, 2018
@mzorz mzorz deleted the fix/block-height-not-refreshed branch August 14, 2018 15:30
hypest added a commit that referenced this pull request Jan 18, 2019
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.

2 participants