<p></p>
<p>After spending more time with them, I'm really liking the new charts. Very useful, and I see now how the time slider is critically important to being able to drill down and get detailed information.</p>
<p>A couple additional, hopefully minor feedback issues:</p>
<ul class="contains-task-list">
<li class="task-list-item"><input type="checkbox" id="" disabled="" class="task-list-item-checkbox"> The state of the timeline slider is not persisted between view changes. So if I select a date range in <code>DAO > Facts and Figures > Dashboard</code> on one or both charts, and then switch to a different view, say <code>Market > Trades</code> and then switch back to <code>DAO > Facts and Figures > Dashboard</code>, my timeline slider gets reset to the default range.</li>
<li class="task-list-item"><input type="checkbox" id="" disabled="" class="task-list-item-checkbox"> The BSQ/USD toggle should be labeled USD/BSQ, because what's actually being charted is "the number of USD per 1 BSQ", not the other way around. As it reads right now, it's confusing. I realize this isn't the only place in the UI that we use this reverse labeling, but perhaps we could at least do it correctly here and then change the other uses later.</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/5156#issuecomment-779019073">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNS6U6VK64QIRIT4473S7DF3DANCNFSM4XEGEIFA">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AJFFTNUZNSZVYT26EKDGJ73S7DF3DA5CNFSM4XEGEIFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFZXOGQI.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/5156#issuecomment-779019073",
"url": "https://github.com/bisq-network/bisq/pull/5156#issuecomment-779019073",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>