Think Aloud feedback: remove Advanced Search link

bcgov / MFIN-Data-Catalogue

The Finance Data Catalogue enables users to discover data holdings at the BC Ministry of Finance and offers information and functionality that benefits consumers of data for business purposes. The product is built using Drupal and adheres to the Government of BC's Core Administrative and Descriptive etadata Standard.

Other

6 stars 0 forks source link

Think Aloud feedback: remove Advanced Search link #508

Open NicoledeGreef opened 4 months ago

NicoledeGreef commented 4 months ago

More than one interviewee reported that the "Advanced Search" link is confusing. It should either be relabeled or removed.

Since clicking it has the exact same result as a search without having supplied one or more search terms, we will remove the link.

[x] remove Advanced Search from the Search screen
[ ] consider adding a "Browse entire catalogue" link that does the same thing as an empty search but present it elsewhere to the user in case they get hung up and want to go into the entire catalogue and use the facets and add search terms (hold this bullet item for now pending a UX Consult)

danhgov commented 3 months ago

I got this working.

For some reason, I got this error on Test but not Prod, when clicking the link for Structure... Pages:

This should probably be a ticket of its own. (...And now there is: #533.)

On prod, it works, and I was able to use it to get the following links for on Test... all of which work, except for the "Structure... Pages" one.

You have to

get to Structure... Pages...
click to edit Home
choose the "Home" Variant
Choose "Content"
Edit the "Bcbb Search API Block"
Open the Search Configuration dropdown
Deselect the "Show advanced search link" checkbox
Click to "Update Block"
Click to "Update and Save" the 'Home' page setup.

This is currently done on Test but not on Dev or Prod. It is also in a PR, to make the change permanent on all 3 servers. @NicoledeGreef, let me know if you'd like me to make this change manually also on Prod, so we don't have to wait for the PR to be merged and deployed.

NicoledeGreef commented 3 months ago

Visually, the Advanced search link is gone as anticipated - thanks.

danhgov commented 2 months ago

@NicoledeGreef, can I go ahead and merge this?

NicoledeGreef commented 2 months ago

Looking at dev, test, and prod, the Advanced Search link is appearing. I had reviewed it somewhere at some point and it was gone. Wondering where I saw this as noted in my previous comment.

danhgov commented 2 months ago

@NicoledeGreef, the changes I made in the UI to demonstrate this to you on dev/test must have been wiped out by a subsequent deploy.

But the code to make it permanent is in a PR, listed above. With your go-ahead, I'll merge it in, and it should fix it on dev/test, and also on prod if you're ready to deploy it there, too.

NicoledeGreef commented 2 months ago

Thanks - I wondered if that might be the case. Yes, please merge.

danhgov commented 2 months ago

@NicoledeGreef, this is now merged, and is on Dev/Test.

NicoledeGreef commented 2 months ago

Thanks @danhgov!

chrislaick commented 2 months ago

Deployed to PROD.