Skip to content

Commit

Permalink
Merge pull request #63 from CityOfPhiladelphia/search-redesign
Browse files Browse the repository at this point in the history
feedback changes
  • Loading branch information
RitikaDesai98 authored Jul 26, 2024
2 parents 5641570 + 6586236 commit be01c84
Showing 1 changed file with 17 additions and 1 deletion.
18 changes: 17 additions & 1 deletion src/ServicesList.vue
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@
<div class="filter-summary">
<span class="result-summary">
<span v-if="hasResults()">
Showing {{ totalResultsCount }} results out of {{ totalServicesCount }} records <span v-if="checkedItems.length > 0 || options.searchValue.length > 0"> for </span><span v-if="options.searchValue.length > 0"><b><em>"{{ options.searchValue }}"</em></b></span>
Showing {{ totalResultsCount }} results <span v-if="checkedItems.length > 0 || options.searchValue.length > 0"> for </span><span v-if="options.searchValue.length > 0"><b><em>"{{ options.searchValue }}"</em></b></span>
</span>
<span v-else>
No results for <b><em>"{{ options.searchValue }}"</em></b>
Expand All @@ -90,6 +90,18 @@
@click="clearAllFilters()"
>
</span>
<div v-if="!hasResults()" class="helper-text">

Check warning on line 93 in src/ServicesList.vue

View workflow job for this annotation

GitHub Actions / Build files

'class' should be on a new line
There were no results found matching your search. Try adjusting your search settings.
<br>
<br>
Here are some options:
<ul>
<li>Use different or fewer search terms</li>
<li>Check your spelling</li>
<li>Remove or adjust any filters</li>
</ul>
Want to start over? Select Clear all to reset the search settings.
</div>
</div>
<nav
v-if="options.azAnchors && options.azGroup && (language !== 'zh')"
Expand Down Expand Up @@ -538,5 +550,9 @@ export default {
text-decoration: underline;
}
.helper-text{
margin-top: 16px;
}
</style>

0 comments on commit be01c84

Please sign in to comment.