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

Afform - compose layouts with multiple SearchKit displays #22887

Merged
merged 3 commits into from
Mar 17, 2022

Conversation

colemanw
Copy link
Member

@colemanw colemanw commented Mar 4, 2022

Overview

Allows multiple search displays on an afform, which permits composing a dashboard-like layout.

Before

Afform GUI could only handle one search display per form.

After

image

@civibot
Copy link

civibot bot commented Mar 4, 2022

(Standard links)

@civibot civibot bot added the master label Mar 4, 2022
@colemanw colemanw changed the title Afform layout Afform - compose layouts with multiple SearchKit displays Mar 4, 2022
@MegaphoneJon
Copy link
Contributor

If the search filters from one display can be made to affect both displays at once, this opens up Search Kit to replace other tools in a number of common use cases. e.g. many CiviReports can be replicated in Search Kit except they can't display totals at the bottom, which this could fix.

@eileenmcnaughton
Copy link
Contributor

I was able to get 2 searches on a page - I couldn't figure out how to put them side by side. As @MegaphoneJon notes the ability to filter more than one at once would be a killer feature - at the moment I can only add search level filters not page level filters.

From a WMF pov I suspect the request to be able to add report dashlets into the mix will come up pretty quickly since not everything can be moved over as yet. Also being able to add afform forms will come up a bit slower (e.g on my dashboard on the site I use I have an afform to create new organisations + a search to find contacts so I can search in the search & then use the form if not found)

Anyway you can present to David et al & see what feedback you get - that's my initial thoughts

@eileenmcnaughton
Copy link
Contributor

Another interesting use case I just hit - I'd love to add created_date to the relationship browse but not sure everyone wants it - so then I think maybe I could replace the tab with search kit - but the tab has 2 displays on it (active & inactive)

@colemanw colemanw force-pushed the afformLayout branch 2 times, most recently from 9d0d1fe to 07a5eac Compare March 13, 2022 23:01
@colemanw
Copy link
Member Author

Those are good suggestions but they shouldn't block merging this. It would be good to get this one merged @eileenmcnaughton so I can finish up the project for WMF.

@eileenmcnaughton
Copy link
Contributor

Looks like this progress is good - more to come

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants