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

Meals/Pantry icons don't match order of filter buttons #927

Closed
fancyham opened this issue Apr 2, 2021 · 7 comments
Closed

Meals/Pantry icons don't match order of filter buttons #927

fancyham opened this issue Apr 2, 2021 · 7 comments
Assignees
Labels
good first issue Good for newcomers Hacktoberfest P-Feature: Map Food seeker map view Release Note: Enhancement Shows on Release Notes under "Enhancements" Release Note: Tech Debt Shows on Release Notes under "Technical Debt" Role: Design UI/UX User interface / user experience design

Comments

@fancyham
Copy link
Collaborator

fancyham commented Apr 2, 2021

The icons on the map / lists don't match up with the filter buttons.

On the filters, Pantry is on the left, Meals is on the right.

On the List and Map, the icons have Meals on the left, Pantry on the right.

The mismatched order make it harder to visually and intuitively relate buttons to items on map.

2021-04-01 Icons don't match filter order

This will also be a problem if we colorize the filter buttons to match the icons.

Some possible solutions:

Simple: Swap the filter buttons so 'Meals' is to the left of 'Pantry'.
2021-04-01 Swap filter buttons

More difficult: Swap sides on icons.

Consider:
Which is higher priority to our target users? Pantries or meals?

@fancyham fancyham changed the title Icons showing meal program/pantry don't match order of filter buttons Meals/Pantry icons don't match order of filter buttons Apr 2, 2021
@ktjnyc ktjnyc added Role: Design UI/UX User interface / user experience design good first issue Good for newcomers labels Apr 2, 2021
@entrotech
Copy link
Member

Good observation. We have a lot more pantries in the database, but that doesn't necessarily mean that they are more important to users. It is easier to swap the position of the fliter buttons than to create new icons for the markers and list items - but that shouldn't be the dirving factor on what we choose to do.

@entrotech entrotech mentioned this issue Apr 8, 2021
11 tasks
@dking1342 dking1342 self-assigned this Jun 23, 2021
@dking1342
Copy link
Contributor

buttons have been swapped and awaiting merge

Screen Shot 2021-06-22 at 10 44 25 PM

@fancyham
Copy link
Collaborator Author

fancyham commented Jun 23, 2021 via email

@dking1342
Copy link
Contributor

It looks like the js file with the svg is located in the client/src/images folder. I believe it is labelled as the splitPantryMealIcon.

Screen Shot 2021-06-22 at 11 15 32 PM

@gigicobos gigicobos added the P-Feature: Map Food seeker map view label Jul 22, 2021
@gigicobos gigicobos assigned fancyham and unassigned dking1342 and fancyham Jul 23, 2021
@sameergautam sameergautam added the Missing: Role The role is missing from this issue label Jul 29, 2021
@gigicobos gigicobos removed the Missing: Role The role is missing from this issue label Jul 29, 2021
@fancyham fancyham self-assigned this Jul 30, 2021
@fancyham
Copy link
Collaborator Author

fancyham commented Aug 6, 2021

@dking1342 I created new 'mixed' icons where I swapped the left and right sides and put them back into the Google Drive where the original files are located (client/src/images). They're named locator.svg and main.svg

I don't know how to add them to the .js files or make a pull request yet — would you mind putting the svg data into the correct files?

I think that should solve this issue?

https://drive.google.com/drive/folders/1jb3yV4B5Isd8oelQ2DTYUvXhm9e6t5fy

@fancyham fancyham assigned dking1342 and unassigned fancyham Aug 6, 2021
@dking1342
Copy link
Contributor

I have swapped out the svg paths to reflect the changes. However, there is an issue with data which prevents the icons, correct icons being shown. I'll have to work on fixing this issue before a pull request can be made.

@dking1342
Copy link
Contributor

@entrotech fixed the issue with the data not showing. The icons not show the proper image and the filter buttons have been ordered to match the icon images. It is now ready to be sent for a pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers Hacktoberfest P-Feature: Map Food seeker map view Release Note: Enhancement Shows on Release Notes under "Enhancements" Release Note: Tech Debt Shows on Release Notes under "Technical Debt" Role: Design UI/UX User interface / user experience design
Projects
Archived in project
Development

No branches or pull requests

6 participants