<p>Clicking on the trader chat icon of a trade that is not selected should select that row in the trade list. It does not. This causes confusion as it gives misleading cues as to which trader you are<br>
communicating with. Issue a call to select the row of the button when clicked.</p>
<p>The trader chat view can be opened either by clicking on the chat icon of a trade in the list, or clicking on "OPEN TRADER CHAT" button for the currently displayed trade.  In the latter case, the count of new messages displayed on the chat icon was not cleared even though trader chat was shown.  The solution is to move the scope of the routine that updates trader chat message count from within the table cell handler to the PendingTradesView class so that it can be called whenever<br>
onChat() is invoked (i.e. when the Trader Chat screen is shown).</p>
<p><span class="issue-keyword tooltipped tooltipped-se" aria-label="This pull request closes issue #4172.">Fixes</span> <a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="599998752" data-permission-text="Title is private" data-url="https://github.com/bisq-network/bisq/issues/4172" data-hovercard-type="issue" data-hovercard-url="/bisq-network/bisq/issues/4172/hovercard" href="https://github.com/bisq-network/bisq/issues/4172">#4172</a></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/4195'>https://github.com/bisq-network/bisq/pull/4195</a></p>

<h4>Commit Summary</h4>
<ul>
  <li>Fix an issue whereby the unread chat message count was not cleared properly</li>
  <li>Select current row when user clicks on trader chat icon</li>
</ul>

<h4>File Changes</h4>
<ul>
  <li>
    <strong>M</strong>
    <a href="https://github.com/bisq-network/bisq/pull/4195/files#diff-8bc3a1461865e020675c0665966fbcc9">desktop/src/main/java/bisq/desktop/main/portfolio/pendingtrades/PendingTradesView.java</a>
    (41)
  </li>
</ul>

<h4>Patch Links:</h4>
<ul>
  <li><a href='https://github.com/bisq-network/bisq/pull/4195.patch'>https://github.com/bisq-network/bisq/pull/4195.patch</a></li>
  <li><a href='https://github.com/bisq-network/bisq/pull/4195.diff'>https://github.com/bisq-network/bisq/pull/4195.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/4195">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNVLHRJCMX6XKNK3CD3ROLSXXANCNFSM4MQYDSVQ">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AJFFTNWGH4AWYLK2R5Q4SYDROLSXXA5CNFSM4MQYDSV2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4JBKT7VA.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/4195",
"url": "https://github.com/bisq-network/bisq/pull/4195",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>