<p>Replace faulty cell update logic, which uses a <code>ChangeListener<Scene></code>, added in July 2017 (#73f21399) to keep the price column in the offer book table up to date, as it appears to occasionally result in blank cells. Also it seems only the prices, not the volumes, were being kept in sync with the market price feed.</p>
<p>Make the price and volume cells stateless and keep them in sync with the market feed by adding it as a dependency of each <code>OfferBookListItem</code> Observable generated by the cell value factory, instead of directly attaching listeners to it. In this way, <code>TableCell::updateItem</code> will be called by the framework whenever the price/volume needs updating.</p>
<p>(This does have the disadvantage that if the price feed is unavailable, causing <code>Offer::getPrice</code> to return null, then the cells will reflect that immediately instead of showing any old, stale values, but that is necessary for the UI to behave consistently anyway.)</p>
<p>--</p>
<p>This is to fix issues like the following, that I have occasionally observed in the Sell Bitcoin offer book, filtered for BSQ (though never in the Buy Bitcoin offer book or for other counter currencies for some reason):</p>

<hr>

<h4>You can view, comment on, or merge this pull request online at:</h4>
<p>  <a href='https://github.com/bisq-network/bisq/pull/4420'>https://github.com/bisq-network/bisq/pull/4420</a></p>

<h4>Commit Summary</h4>
<ul>
  <li>Rename misleading OfferBookChartViewModel method name</li>
  <li>Fix intermittent blank price cells in offer book view</li>
</ul>

<h4>File Changes</h4>
<ul>
  <li>
    <strong>M</strong>
    <a href="https://github.com/bisq-network/bisq/pull/4420/files#diff-6e51b713c030d7a2fc6b28b197744823">desktop/src/main/java/bisq/desktop/main/market/offerbook/OfferBookChartViewModel.java</a>
    (27)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/bisq-network/bisq/pull/4420/files#diff-c32e3ea8a6f08e076e6164033b29e2ae">desktop/src/main/java/bisq/desktop/main/offer/offerbook/OfferBookView.java</a>
    (74)
  </li>
</ul>

<h4>Patch Links:</h4>
<ul>
  <li><a href='https://github.com/bisq-network/bisq/pull/4420.patch'>https://github.com/bisq-network/bisq/pull/4420.patch</a></li>
  <li><a href='https://github.com/bisq-network/bisq/pull/4420.diff'>https://github.com/bisq-network/bisq/pull/4420.diff</a></li>
</ul>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/bisq-network/bisq/pull/4420">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNXK3MSMNCAYC5TTKMLSBUBSXANCNFSM4QF6DDSQ">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AJFFTNRWXSZW3LBBGTNNS7DSBUBSXA5CNFSM4QF6DDS2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4KFQGJUQ.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/bisq-network/bisq/pull/4420",
"url": "https://github.com/bisq-network/bisq/pull/4420",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>