Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Search Builder link from search results #26670

Conversation

larssandergreen
Copy link
Contributor

@larssandergreen larssandergreen commented Jun 26, 2023

Overview

Since Search Builder is on the way out, I don't think we need the links to Search Builder from Find Contacts or Advanced Search results. We could replace these with link to SearchKit, but I don't think that's particularly helpful to anyone since it's right there in the menu already.

Also removed the somewhat strange > icon in front of Advanced Search as we are not clicking on it to 'advance to the next thing' as the UI Reference indicates for fa-chevron-right. I think it's there to tell you that the link takes you somewhere, but that's sort of the nature of links.

Before

image
image

After

Bye-bye.
image
image

@civibot
Copy link

civibot bot commented Jun 26, 2023

(Standard links)

@civibot civibot bot added the master label Jun 26, 2023
@demeritcowboy demeritcowboy merged commit 2e35631 into civicrm:master Jun 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants