Search filter options disappear/get bumped to bottom of search results page

Description

When a search is conducted from the main page, some of the results are strangely nested in "boxes" in the search results list. Because the nested boxes take up real estate on the page, it pushes the filter search options to the bottom of the search results page - which, when 100s of results long, is useless. The nested boxes don't seem to have any rhyme or reason, because when clicking on a later page of results, the results don't seem to be nested based on component level. Use "camping" search example for screenshots to report in http://archives.umn.lib.edu for further examples to attached screenshots.

Environment

None

Attachments

4

Activity

Show:

Mark Custer November 6, 2017 at 4:56 PM

Looks like this is the cause: https://github.com/archivesspace/archivesspace/commit/7459536bc78e3017ef4b754a0e3b1369406a0164#diff-089a949d82ba05b235926e101e879d45R90

But not only does that remove the ability to do phrase searches, it also removes the ability to do wildcard searching with the asterisk, etc.

Mark Custer November 6, 2017 at 4:04 PM
Edited

, : for testing, you'd just need to have a scope and content note added to an archival object that's over 500 characters, with something like a <title render="italic"> element wrapped around the entirety of the note.

But I can confirm that this particular issue is fixed in our instance of DEV right now. yay! there are a couple of downsides of stripping the HTML elements outright, though: 1) no formatting / links, but that's fine, in my opinion; 2) it can present spacing problems, which I think could be addressed by updating how those notes are presented. But if there was supposed to be a paragraph break or line break, for example, then those elements are pushed together (e.g. "Summary (English)Film"). it'd be nice if a space could be added in these instances; e.g. "Summary (English) Film". Missing the space is better than breaking whole page and having the facets pushed to the bottom, but it seems like this could be addressed with just a bit more logic inserted into the process.

However, I noticed a new problem in our instance of DEV (version 20171105-1021), which doesn't happen in our instance of TEST (version 2.1.2)... and I just checked 2.2.0 and that's okay, as well (phew!). Putting quotes around your query to initiate a phrase search does not work anymore. Not sure what commit caused this issue, but that will definitely need to be addressed.

Example: if I search for "statistically improbable phrase" in our corpus (without quotes), i get 76 results. if i search for the same query, but with quotes, i get the same 76 results. but i should get 0 results when i apply the quotes! and if i do that search in 2.1.2 (or 2.2), i get 0 results with our corpus, as expected.

Christine Di Bella November 6, 2017 at 3:10 PM

- the fix for this one was merged into master last week but it's hard to test since it only shows itself under very particular conditions. Since you have a dev server running that code, have you been able to note whether the problem record(s) look right in the PUI now?

Mark Custer October 25, 2017 at 1:10 PM

A complete aside, but the following screenshot is one of the best examples of auto-generated ASpace art that I've seen yet:

Mark Custer October 25, 2017 at 1:04 PM

how i wish i would've seen your super helpful note yesterday! in any event, it's also good to arrive at the same conclusion.

Here's my quick fix: Change line https://github.com/archivesspace/archivesspace/blob/master/public/app/views/shared/_result.html.erb#L23 to

<%= strip_tags(truncated_note[0..end_index]) %>

Doing so will remove all HTML from the truncated notes in this view, but it will ensure that the resulting HTML of the overall page is valid, so that the results won't appear nested, and the facets won't be pushed into a completely different section of the page.

Done

Details

Assignee

Reporter

Fix versions

Affects versions

Priority

Harvest Time Tracking

Open Harvest Time Tracking

Created September 14, 2017 at 9:05 PM
Updated November 17, 2017 at 4:37 PM
Resolved November 17, 2017 at 4:37 PM
Harvest Time Tracking