Skip to content

Commit

Permalink
update styling for VerticalFullPageMap viewAllResults button (#1084)
Browse files Browse the repository at this point in the history
Due to a layout change in vertical full page map template to [Fix CollapsibleFilters tab order](51008f6), the styling for the top level `Answers-contentWrap` class, `pointer-events: none;`, got applied to the view results button, making it unclickable. Even though other templates, like vertical grid, also had the same layout changes, this style is only use in VerticalFullPageMap.scss.
This pr added a styling `pointer-events: all;` targeting Answers-viewResultsButton classname specifically when it's use in mobile view in VerticalFullPageMap template.

J=TECHOP-5941
TEST=manual

spin up test site and open up page `locations_full_page_map_with_filters` (which is a vertical-full-page-map template). Bring screen size to mobile width, and click the filters button. See that the view results button is now clickable and return me to the results view.
As a sanity check, check the the view results button in `people` page (which is a vertical-grid template) works as expected still.
  • Loading branch information
yen-tt committed Jul 22, 2022
1 parent 2e22199 commit 98a5b85
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions static/scss/answers/interactive-map/VerticalFullPageMap.scss
Original file line number Diff line number Diff line change
Expand Up @@ -641,6 +641,11 @@
width: 100%;
position: absolute;
}

@include bplte($mobile-break-point-max)
{
pointer-events: all;
}
}

&.CollapsibleFilters .Answers-filtersWrapper {
Expand Down

0 comments on commit 98a5b85

Please sign in to comment.