Skip to content

Commit

Permalink
Fix overflow-x
Browse files Browse the repository at this point in the history
  • Loading branch information
stevenjoezhang committed May 6, 2024
1 parent e9fe72b commit c437189
Show file tree
Hide file tree
Showing 5 changed files with 5 additions and 18 deletions.
2 changes: 1 addition & 1 deletion layout/_partials/search/algolia-search.njk
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
<i class="fa fa-times-circle"></i>
</span>
</div>
<div class="search-result-container no-result">
<div class="search-result-container">
<div class="search-result-icon">
<i class="fab fa-algolia fa-5x"></i>
</div>
Expand Down
2 changes: 1 addition & 1 deletion layout/_partials/search/localsearch.njk
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
<i class="fa fa-times-circle"></i>
</span>
</div>
<div class="search-result-container no-result">
<div class="search-result-container">
<div class="search-result-icon">
<i class="fa fa-spinner fa-pulse fa-5x"></i>
</div>
Expand Down
13 changes: 3 additions & 10 deletions source/css/_common/components/third-party/search.styl
Original file line number Diff line number Diff line change
Expand Up @@ -79,11 +79,14 @@ if (hexo-config('local_search.enable') or hexo-config('algolia_search.enable'))
}

.search-result-container {
display: flex;
flex-direction: column;
height: calc(100% - 55px);
overflow: auto;
padding: 5px 25px;

hr {
flex-shrink: 0;
margin: 5px 0 10px;

&:first-child {
Expand All @@ -105,22 +108,12 @@ if (hexo-config('local_search.enable') or hexo-config('algolia_search.enable'))
border-bottom: 1px dashed $grey-light;
padding: 5px 0;
}
}

.search-popup {
.search-input-container {
flex-grow: 1;
padding: 2px;
}

.no-result {
display: flex;
}

.search-result-list {
width: 100%;
}

.search-result-icon {
color: $grey-light;
margin: auto;
Expand Down
3 changes: 0 additions & 3 deletions source/js/third-party/search/algolia-search.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@ document.addEventListener('DOMContentLoaded', () => {
hitsPerPage : hits.per_page || 10
});
if (data.nbHits === 0) {
container.classList.add('no-result');
container.innerHTML = '<div class="search-result-icon"><i class="far fa-frown fa-5x"></i></div>';
} else {
const stats = CONFIG.i18n.hits_time
Expand All @@ -47,7 +46,6 @@ document.addEventListener('DOMContentLoaded', () => {
pagination += '</nav>';
}

container.classList.remove('no-result');
container.innerHTML = `<div class="search-stats">
<span>${stats}</span>
<img src="${CONFIG.images}/logo-algolia-nebula-blue-full.svg" alt="Algolia">
Expand All @@ -68,7 +66,6 @@ document.addEventListener('DOMContentLoaded', () => {
const inputEventFunction = async() => {
const searchText = input.value.trim();
if (searchText === '') {
container.classList.add('no-result');
container.innerHTML = '<div class="search-result-icon"><i class="fab fa-algolia fa-5x"></i></div>';
return;
}
Expand Down
3 changes: 0 additions & 3 deletions source/js/third-party/search/local-search.js
Original file line number Diff line number Diff line change
Expand Up @@ -25,10 +25,8 @@ document.addEventListener('DOMContentLoaded', () => {
resultItems = localSearch.getResultItems(keywords);
}
if (keywords.length === 1 && keywords[0] === '') {
container.classList.add('no-result');
container.innerHTML = '<div class="search-result-icon"><i class="fa fa-search fa-5x"></i></div>';
} else if (resultItems.length === 0) {
container.classList.add('no-result');
container.innerHTML = '<div class="search-result-icon"><i class="far fa-frown fa-5x"></i></div>';
} else {
resultItems.sort((left, right) => {
Expand All @@ -41,7 +39,6 @@ document.addEventListener('DOMContentLoaded', () => {
});
const stats = CONFIG.i18n.hits.replace('${hits}', resultItems.length);

container.classList.remove('no-result');
container.innerHTML = `<div class="search-stats">${stats}</div>
<hr>
<ul class="search-result-list">${resultItems.map(result => result.item).join('')}</ul>`;
Expand Down

0 comments on commit c437189

Please sign in to comment.