https://dev.arvados.org/https://dev.arvados.org/favicon.ico?15576888422015-03-08T01:44:57ZArvadosArvados - Bug #5420: [Workbench] Checkbox selections in collections page is wrong when using back buttonhttps://dev.arvados.org/issues/5420?journal_id=220542015-03-08T01:44:57ZRadhika Chippadaradhika@curoverse.com
<ul><li><strong>Description</strong> updated (<a title="View differences" href="/journals/22054/diff?detail_id=21219">diff</a>)</li><li><strong>Category</strong> set to <i>Workbench</i></li></ul> Arvados - Bug #5420: [Workbench] Checkbox selections in collections page is wrong when using back buttonhttps://dev.arvados.org/issues/5420?journal_id=222932015-03-11T20:26:04ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Target version</strong> changed from <i>Bug Triage</i> to <i>2015-04-01 sprint</i></li></ul> Arvados - Bug #5420: [Workbench] Checkbox selections in collections page is wrong when using back buttonhttps://dev.arvados.org/issues/5420?journal_id=222942015-03-11T20:26:53ZTom Cleggtom@curii.com
<ul></ul><p>Hopefully we just need to add a unique <code>id</code> to each checkbox, so the browser doesn't have to assume the "first two checkboxes" mean the same thing after reloading a page.</p> Arvados - Bug #5420: [Workbench] Checkbox selections in collections page is wrong when using back buttonhttps://dev.arvados.org/issues/5420?journal_id=225762015-03-20T20:09:15ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Assigned To</strong> set to <i>Radhika Chippada</i></li></ul> Arvados - Bug #5420: [Workbench] Checkbox selections in collections page is wrong when using back buttonhttps://dev.arvados.org/issues/5420?journal_id=227132015-03-25T15:07:37ZRadhika Chippadaradhika@curoverse.com
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>In Progress</i></li></ul> Arvados - Bug #5420: [Workbench] Checkbox selections in collections page is wrong when using back buttonhttps://dev.arvados.org/issues/5420?journal_id=227322015-03-25T17:14:56ZRadhika Chippadaradhika@curoverse.com
<ul></ul><p>Implementation notes:</p>
<ul>
<li>Adding id to checkboxes resolved the issue in /collections page (verified before and after using Chrome)</li>
</ul>
<ul>
<li>Adding id alone did not help with the collection#show page. I also needed to added autocomplete="off" for the checkboxes as well as for the search box in this page. Here is the explanation: using Safari (and probably Chrome), access <a class="external" href="https://workbench.4xphq.arvadosapi.com/collections/4xphq-4zz18-gbv4fbtq6sdvzpd">https://workbench.4xphq.arvadosapi.com/collections/4xphq-4zz18-gbv4fbtq6sdvzpd</a> . You will see 3 files in this collection. Type “md5sum” in the search box. You will see two files. Select them and click on Selection -> create new collection… . Once you are in the new collection page, hit back button. You will now see all three files (all the files in this collection); however, the search box still has the text “md5sum” in it and the two md5sum files are selected. That is, the collection page is reloaded, but Safari is caching selections. I hence needed to add autocomplete off for the search box as well as for the checkboxes in this page.</li>
</ul>
<ul>
<li>Verified using manual testing using Chrome in my VM.</li>
</ul> Arvados - Bug #5420: [Workbench] Checkbox selections in collections page is wrong when using back buttonhttps://dev.arvados.org/issues/5420?journal_id=229422015-03-30T15:44:47ZBrett Smithbrett.smith@curii.com
<ul></ul><p>Radhika Chippada wrote:</p>
<blockquote>
<p>Implementation notes:</p>
<ul>
<li>Adding id to checkboxes resolved the issue in /collections page (verified before and after using Chrome)</li>
</ul>
</blockquote>
<p>This part looks good, thanks.</p>
<blockquote>
<ul>
<li>Adding id alone did not help with the collection#show page. I also needed to added autocomplete="off" for the checkboxes as well as for the search box in this page. Here is the explanation: using Safari (and probably Chrome), access <a class="external" href="https://workbench.4xphq.arvadosapi.com/collections/4xphq-4zz18-gbv4fbtq6sdvzpd">https://workbench.4xphq.arvadosapi.com/collections/4xphq-4zz18-gbv4fbtq6sdvzpd</a> . You will see 3 files in this collection. Type “md5sum” in the search box. You will see two files. Select them and click on Selection -> create new collection… . Once you are in the new collection page, hit back button. You will now see all three files (all the files in this collection); however, the search box still has the text “md5sum” in it and the two md5sum files are selected. That is, the collection page is reloaded, but Safari is caching selections. I hence needed to add autocomplete off for the search box as well as for the checkboxes in this page.</li>
</ul>
</blockquote>
<p>I'm not so sure the page is being reloaded. It seems like another possibility is that our filtering JavaScript isn't being triggered when the browser loads the page with text already in the search box. Turning autocomplete off seems like a net UX loss to me: it seems likely that users will want to repeat searches, and browsers should be able to help them with that.</p>
<p>I also don't think it's a bug if the browser remembers what checkboxes are selected, as long as the selections refer to the same objects as users browse back and forth. The "before" behavior you describe sounds like it's okay except for the fact that the search filter isn't being applied. I think it would be nicer to fix that issue in the JavaScript, rather than artificially limit the form to prevent it from happening.</p>
<p>Thanks.</p> Arvados - Bug #5420: [Workbench] Checkbox selections in collections page is wrong when using back buttonhttps://dev.arvados.org/issues/5420?journal_id=230352015-04-01T14:50:18ZRadhika Chippadaradhika@curoverse.com
<ul></ul><p>Brett said:</p>
<blockquote>
<p>I'm not so sure the page is being reloaded.</p>
</blockquote>
<p>I verified by looking at workbench and api server logs that the page is indeed reloaded.</p>
<blockquote>
<p>It seems like another possibility is that our filtering JavaScript isn't being triggered when the browser loads the page with text already in the search box. Turning autocomplete off seems like a net UX loss to me: it seems likely that users will want to repeat searches, and browsers should be able to help them with that.</p>
</blockquote>
<p>I reverted the search (as well as the checkbox) autocomplete off and created a separate issue <a class="issue tracker-1 status-3 priority-4 priority-default closed parent" title="Bug: [Workbench] Collection#show page search filter is neither cleared nor applied when back button is... (Resolved)" href="https://dev.arvados.org/issues/5620">#5620</a> to address this specific issue preferably to trigger search filtering on using back button.</p>
<blockquote>
<p>I also don't think it's a bug if the browser remembers what checkboxes are selected, as long as the selections refer to the same objects as users browse back and forth. The "before" behavior you describe sounds like it's okay except for the fact that the search filter isn't being applied. I think it would be nicer to fix that issue in the JavaScript, rather than artificially limit the form to prevent it from happening.</p>
</blockquote>
<p>The checkbox selections are correct (since no new files added etc). Since we are not clearing search box text + they are correct selections, it does not seem appropriate to clear selections alone. Thus, I reverted this update as well.</p> Arvados - Bug #5420: [Workbench] Checkbox selections in collections page is wrong when using back buttonhttps://dev.arvados.org/issues/5420?journal_id=230422015-04-01T15:15:09ZRadhika Chippadaradhika@curoverse.com
<ul><li><strong>Status</strong> changed from <i>In Progress</i> to <i>Resolved</i></li></ul><p>Applied in changeset arvados|commit:5781ff2dc48f95b7554259e0210272e7de2d883b.</p>