search

cityofaustin / atd-moped

A comprehensive mobility project tracking platform for Austin, Texas.
https://mobility.austin.gov/moped/
4 stars 3 forks source link

[Enhancement] Show active advanced project list filters as chips #1363

Closed chiaberry closed 4 months ago

chiaberry commented 4 months ago

Associated issues

Closes https://github.com/cityofaustin/atd-data-tech/issues/15287

Testing

URL to test: https://deploy-preview-1363--atd-moped-main.netlify.app/moped/projects?filters=%5B%7B%22field%22%3A%22project_name%22%2C%22operator%22%3A%22string_contains_case_insensitive%22%2C%22value%22%3A%22parent%22%7D%2C%7B%22field%22%3A%22current_phase%22%2C%22operator%22%3A%22string_equals_case_insensitive%22%2C%22value%22%3A%22Potential%22%7D%5D&isOr=true

Steps to test:

Add different filters, try some is blanks and some contains with long text.

Screenshot 2024-06-18 at 11 59 07 AM

Ship list

mddilley commented 4 months ago

@chiaberry This looks so great! Could you take a look at how the chips wrap to the next line when there are a lot of filters? I think we used Grid in the Tags section of the project summary to get them to spill over neatly.

Screenshot 2024-06-18 at 10 33 22 AM
mddilley commented 4 months ago

@frankhereford could you help me reproduce the bug that you mentioned above? I tried out https://deploy-preview-1363--atd-moped-main.netlify.app/moped/projects?filters=%5B%7B%22field%22%3A%22type_name%22%2C%22operator%22%3A%22string_is_not_null%22%2C%22value%22%3Anull%7D%5D&isOr=false, and I'm seeing the filter apply. I could have misunderstood the bug. Thank you for helping track down the remaining bugs with the filters!!

mddilley commented 4 months ago

Merging this to continue testing - thanks y'all!