Skip to content

Commit

Permalink
UX: fix button alignment, remove dead CSS (#56)
Browse files Browse the repository at this point in the history
  • Loading branch information
awesomerobot authored May 13, 2024
1 parent 9173272 commit 2333c26
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 23 deletions.
23 changes: 4 additions & 19 deletions common/common.scss
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,7 @@ $max-width: 600px;

.search-menu-container {
width: 100%;
min-width: 0;
}
}

Expand All @@ -59,16 +60,11 @@ $max-width: 600px;
.browser-search-tip {
display: none;
}
.search-input input#search-term[type="text"] {
margin: 0;
width: 100%;
}

.search-input {
flex: 1 1 auto;
margin: 0;
.searching {
display: flex;
#search-term {
min-width: 0;
flex: 1 1;
}
}

Expand Down Expand Up @@ -124,13 +120,6 @@ $max-width: 600px;
}
}

// hide the search icon when a search context is selected
// eg when searching in a topic
.search-input .search-context + .search-icon:not(.has-search-button-text),
.search-input .search-context ~ .search-icon:not(.has-search-button-text) {
display: none;
}

.results {
box-sizing: border-box;
background: var(--secondary);
Expand All @@ -156,10 +145,6 @@ $max-width: 600px;
}
}

.searching a.show-advanced-search {
display: none;
}

.search-link .d-icon {
color: var(--primary-medium);
}
Expand Down
4 changes: 0 additions & 4 deletions mobile/mobile.scss
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,3 @@
.search-menu .search-input input#search-term {
width: 100%;
}

.custom-search-banner-wrap .search-icon {
top: 0.15em;
}

0 comments on commit 2333c26

Please sign in to comment.