Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue 99 - Collapsible filter/legend #241

Closed
wants to merge 9 commits into from

Conversation

amaxama
Copy link
Contributor

@amaxama amaxama commented Sep 9, 2020

What

Add collapsability to top filters and toggle legend on map when filters are collapsed

Why

Issue #99

Ensure the following interactions work as expected. Please test using a mobile form factor.

  • Tapping on a marker on the map displays information about the marker in a popup.
  • Tapping the "Show list of locations" button replaces the map view with a list view.
  • Tapping an item in the list replaces the list view with a map view, and navigates the map to the tapped item on the map.
  • Tapping one of the ✅'s in the legend filters items on the map and in the list.
  • Changing the language to spanish changes things on the page.
  • Clicking the Help/Info button opens and closes a menu with information.
  • Clicking the X Close button on the Help/Info screen closes the modal.

Check the app in the following web browsers:

  • Chrome
  • Firefox
  • Edge
  • Safari

@amaxama amaxama added the Status: DON'T MERGE YET Don't merge yet. Please :) label Sep 9, 2020
@amaxama amaxama linked an issue Sep 9, 2020 that may be closed by this pull request
@@ -20,6 +20,7 @@
--primary-700: hsl(var(--primary), 35%);
--primary-800: hsl(var(--primary), 25%);
--primary-900: hsl(var(--primary), 15%);
--primary-blue: hsl(var(--primary), 10%);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the original dark blue used for the title header - open to a better variable name if there are suggestions!

Comment on lines +65 to +82
<div class="action-group">
<button data-translation-id="show_list_button" id="locations-toggle-button" class="location-list-toggle toggle-button" aria-label="Show list of locations">
Show list of locations
</button>
<button data-translation-id="help_info" id="help-info-toggle-button" class="toggle-button" aria-label="Toggle help info section">
Help/Info
</button>
<button data-translation-id="refresh_button" id='refresh-page-button' class='toggle-button' aria-label="refresh page data">
<img class="refresh-image" alt="${lang_name}" src="/images/refresh.png">
<span class="refresh-text" data-translation-id="refresh">
Refresh
</span>
</button>
<div class='counter'>
<script type="text/javascript" src="//counter.websiteout.net/js/7/0/41000/0" ></script>
</div>
</div>
</div>
Copy link
Contributor Author

@amaxama amaxama Sep 10, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Most of this is indentation changes

}

/* component: title */
.title {
padding: 10px 15px 0px 15px;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moved all legend and filter related items to filter.css

@@ -52,7 +52,7 @@ a.txt-white:hover {

.content {
z-index: 1;
width: 33.3333%;
width: 36%;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mad just a little wider so title fits on one line more often

@@ -22,6 +22,10 @@
"filter_by_both": "ለ ሁለቱም ክፍት ነው ",
"filter_by_closed": "ለጊዜው ዝግ ነው ",
"filter_by_unknown": "ሁኔታው አይታወቅም ",
"receiving": "receiving",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All of these are just placeholders for now!

@amaxama
Copy link
Contributor Author

amaxama commented Sep 10, 2020

Here's an example of what it looks like (animation is slow cause my computer is laggy, it should be a bit more smooth than that):
filter-legend-toggle

@mc-funk
Copy link
Contributor

mc-funk commented Sep 10, 2020

So exciting!
I’m wondering if there might be.a better placement for the toggle caret — on mobile for me I am struggling to tap the caret Instead of the checkbox it’s next to. Maybe it would be worth nudging down the font of TCMAP header and getting them on the same line, and make it a little bigger to boot?

@amaxama
Copy link
Contributor Author

amaxama commented Sep 11, 2020

@mc-funk Yeah... I played around with a few places to put it..I had originally thought of putting it in line with the title but I actually kind of want to put a kebab menu there for overflow actions like "refresh"....I could try putting it over to the right though and see how that goes. Regardless, I just tried it on mobile and the touch area that picks up tap input is definitely not big enough so I'm going to increase that for sure. I'll try that first and see if that makes a difference for you and then I can try it over to the right if it's sill not great

@amaxama amaxama closed this Sep 12, 2020
@amaxama amaxama deleted the issue-99/maxam/collapse-filter branch September 12, 2020 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: DON'T MERGE YET Don't merge yet. Please :)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants