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

Implement new design for Toolkit Mobile Menu #2391

Closed
2 tasks done
abuna1985 opened this issue Oct 23, 2021 · 28 comments · Fixed by #3085
Closed
2 tasks done

Implement new design for Toolkit Mobile Menu #2391

abuna1985 opened this issue Oct 23, 2021 · 28 comments · Fixed by #3085
Assignees
Labels
Complexity: Medium P-Feature: Toolkit https://www.hackforla.org/toolkit/ role: front end Tasks for front end developers size: 2pt Can be done in 7-12 hours Status: Updated No blockers and update is ready for review
Milestone

Comments

@abuna1985
Copy link
Member

abuna1985 commented Oct 23, 2021

Overview

We need to implement the Figma design provided by the design team to build our mobile menu for the Toolkit page so we can provide a better user experience

Click here to see current mobile view of the Toolkit page

Screenshot 2021-10-22 213442

Click here to see the Figma design mobile view of the Toolkit page

Screenshot 2021-10-22 214222

Action Items

  • Implement the design for the Toolkit menu in mobile view
  • Complete a pull request with the proposed changes

Resources/Instructions

Figma Design - Toolkit Page
Click below to see source code for text menu in Toolkit

<div class="toolkit-header__text-group">
{% assign category_list = "All, Development, Design, Project Management, Professional Development" | split: ", " %}
{% for category in category_list %}
{% if category == "All" %}
<a href="#{{category | replace: ' ', '+'}}" class="toolkit-header__banner-item selected-category">{{category}}</a>
{% else %}
<a href="#{{category | replace: ' ', '+'}}" class="toolkit-header__banner-item">{{ category }}</a>
{% endif %}
{% endfor %}
</div>

@abuna1985 abuna1985 added role: front end Tasks for front end developers Complexity: Medium P-Feature: Toolkit https://www.hackforla.org/toolkit/ Ready for Prioritization labels Oct 23, 2021
@github-actions

This comment has been minimized.

@ExperimentsInHonesty

This comment has been minimized.

@abuna1985

This comment has been minimized.

@BrianCodes33
Copy link
Contributor

BrianCodes33 commented Jan 28, 2022

Hi @abuna1985 . I'd like to work on this issue. I am seeing the new design on Figma that the team would like to implement on mobile view. Is there a specific breakpoint the design team would like to fire the dropdown (for tablet view as well)? Is it possible to have this dropdown menu items on desktop view to simplify the solution? Similar to the main page program areas...

Screen Shot 2022-01-28 at 8 46 29 AM

@SAUMILDHANKAR
Copy link
Member

@BrianCodes33 Request to please unassign yourself from this issue and move it to Prioritized backlog column. You can pick a good second issue from the prioritized backlog column using this link: https://github.com/hackforla/website/projects/7?card_filter_query=label%3A%22size%3A+good+second+issue%22. Once your good second issue is merged, you can come back and work on this issue. Thanks.

@Georgema20
Copy link
Member

Hi, Been working on this for a while....Need some help. No OH tomorrow so don't know where to go. Guess I will go next OH? Can not find if you guys already have a class to make a div a filter kinda bar.

@github-actions github-actions bot added the Status: Updated No blockers and update is ready for review label Feb 4, 2022
@JessicaLucindaCheng
Copy link
Member

@Georgema20 Sorry that we did not have office hours to answer your question. However, feel free to ask us on Slack when you have questions or if you want to set up a time to meet to discuss your issue outside of our normal meeting times. I'm not sure if we already have a filter component or not, so let me do some research and I'll get back to you within a couple of days.

@JessicaLucindaCheng
Copy link
Member

@Georgema20 In the Design System in Figma here are the filters we have (Note: Some of the SCSS properties in the classes have changed): https://www.figma.com/file/0RRPy1Ph7HafI3qOITg0Mr/Hack-for-LA-Website?node-id=4532%3A444

If you are looking for the current SCSS class definitions and properties, they are here: https://github.com/hackforla/website/blob/063e211e2b6765e631a62af0f211941cef527f71/_sass/components/_project-filter.scss

Hopefully, you can use some of these SCSS classes to help make the filter. Let me know if you have any other questions.

@github-actions github-actions bot added 2 weeks inactive An issue that has not been updated by an assignee for two weeks and removed Status: Updated No blockers and update is ready for review labels Feb 11, 2022
@github-actions
Copy link

@Georgema20

Please add update using the below template (even if you have a pull request). Afterwards, remove the 'To Update !' label and add the 'Status: Updated' label.

  1. Progress: "What is the current status of your project? What have you completed and what is left to do?"
  2. Blockers: "Difficulties or errors encountered."
  3. Availability: "How much time will you have this week to work on this issue?"
  4. ETA: "When do you expect this issue to be completed?"
  5. Pictures: "Add any pictures of the visual changes made to the site so far."

If you need help, be sure to either: 1) place your issue in the developer meeting discussion column and ask for help at your next meeting, 2) put a "Status: Help Wanted" label on your issue and pull request, or 3) put up a request for assistance on the #hfla-site channel.

You are receiving this comment because your last comment was before Monday, February 7, 2022 at 11:17 PM PST.

@JessicaLucindaCheng JessicaLucindaCheng added To Update ! No update has been provided and removed 2 weeks inactive An issue that has not been updated by an assignee for two weeks labels Feb 12, 2022
@github-actions github-actions bot added 2 weeks inactive An issue that has not been updated by an assignee for two weeks and removed To Update ! No update has been provided labels Feb 18, 2022
@github-actions
Copy link

@Georgema20

Please add update using the below template (even if you have a pull request). Afterwards, remove the 'To Update !' label and add the 'Status: Updated' label.

  1. Progress: "What is the current status of your project? What have you completed and what is left to do?"
  2. Blockers: "Difficulties or errors encountered."
  3. Availability: "How much time will you have this week to work on this issue?"
  4. ETA: "When do you expect this issue to be completed?"
  5. Pictures: "Add any pictures of the visual changes made to the site so far."

If you need help, be sure to either: 1) place your issue in the developer meeting discussion column and ask for help at your next meeting, 2) put a "Status: Help Wanted" label on your issue and pull request, or 3) put up a request for assistance on the #hfla-site channel.

You are receiving this comment because your last comment was before Monday, February 14, 2022 at 11:18 PM PST.

@JessicaLucindaCheng
Copy link
Member

JessicaLucindaCheng commented Feb 19, 2022

@Georgema20 Did you need help or have questions? Also, please provide a weekly progress update in a comment below that includes the following:

1. Progress: "What is the current status of your project? What have you completed and what is left to do?"
2. Blockers: "Difficulties or errors encountered."
3. Availability: "How much time will you have this week to work on this issue?"
4. ETA: "When do you expect this issue to be completed?"
5. Pictures: "Add any pictures of the visual changes made to the site so far."

@JessicaLucindaCheng
Copy link
Member

Left a Slack message on Fri, Feb 25, 2022.

@github-actions
Copy link

github-actions bot commented Mar 4, 2022

@Georgema20

Please add update using the below template (even if you have a pull request). Afterwards, remove the 'To Update !' label and add the 'Status: Updated' label.

  1. Progress: "What is the current status of your project? What have you completed and what is left to do?"
  2. Blockers: "Difficulties or errors encountered."
  3. Availability: "How much time will you have this week to work on this issue?"
  4. ETA: "When do you expect this issue to be completed?"
  5. Pictures: "Add any pictures of the visual changes made to the site so far."

If you need help, be sure to either: 1) place your issue in the developer meeting discussion column and ask for help at your next meeting, 2) put a "Status: Help Wanted" label on your issue and pull request, or 3) put up a request for assistance on the #hfla-site channel.

You are receiving this comment because your last comment was before Monday, February 28, 2022 at 11:18 PM PST.

@SAUMILDHANKAR
Copy link
Member

@Georgema20 Moving the issue back to the Prioritized Backlog column due to inactivity. Please feel free to pick it back up when available. Will leave a slack message about the same. Thanks.

@SAUMILDHANKAR SAUMILDHANKAR removed the 2 weeks inactive An issue that has not been updated by an assignee for two weeks label Mar 5, 2022
@JessicaLucindaCheng JessicaLucindaCheng added the size: 2pt Can be done in 7-12 hours label Mar 8, 2022
@kathrynsilvaconway kathrynsilvaconway self-assigned this Apr 14, 2022
@kathrynsilvaconway
Copy link
Member

ETA: End of the day on 4/19
Availability : 6 hours

@github-actions github-actions bot added the Status: Updated No blockers and update is ready for review label Apr 15, 2022
@kathrynsilvaconway
Copy link
Member

  1. Progress: Got theGained better overall understanding of how we organize CSS and HTML. basic style of the dropdown menu in place. Nav bar appears above 480px, dropdown appears for everythin below. Feeling confident I can finish it this week.
  2. Blockers: Struggling to understand how html and CSS are organized on the site in general. I can't tell if there are existing classes I should be using or not, and I am not sure that I am organizing my CSS correctly. Will consult team leads on this.
  3. Availability: 6 Hours
  4. ETA: End of the Day on 4/26

@github-actions github-actions bot removed the Status: Updated No blockers and update is ready for review label Apr 29, 2022
@github-actions
Copy link

@kathrynsilvaconway

Please add update using the below template (even if you have a pull request). Afterwards, remove the 'To Update !' label and add the 'Status: Updated' label.

  1. Progress: "What is the current status of your project? What have you completed and what is left to do?"
  2. Blockers: "Difficulties or errors encountered."
  3. Availability: "How much time will you have this week to work on this issue?"
  4. ETA: "When do you expect this issue to be completed?"
  5. Pictures: "Add any pictures of the visual changes made to the site so far."

If you need help, be sure to either: 1) place your issue in the developer meeting discussion column and ask for help at your next meeting, 2) put a "Status: Help Wanted" label on your issue and pull request, or 3) put up a request for assistance on the #hfla-site channel.

You are receiving this comment because your last comment was before Tuesday, April 26, 2022 at 12:21 AM PST.

@github-actions github-actions bot added the 2 weeks inactive An issue that has not been updated by an assignee for two weeks label Apr 29, 2022
@JessicaLucindaCheng JessicaLucindaCheng added To Update ! No update has been provided and removed 2 weeks inactive An issue that has not been updated by an assignee for two weeks labels May 1, 2022
@github-actions github-actions bot added Status: Updated No blockers and update is ready for review and removed To Update ! No update has been provided labels May 6, 2022
@ExperimentsInHonesty
Copy link
Member

ExperimentsInHonesty commented May 28, 2022

@kathrynsilvaconway I noticed the menu carrots are different than our other menus. Has this received design sign off?

Toolkit

EB8D98B2-3DD5-46CA-A0B0-A7DDFC4D6B6B

Projects page

2C754979-29B5-405D-915C-1BD8AF8908D7

@JessicaLucindaCheng
Copy link
Member

JessicaLucindaCheng commented May 29, 2022

Leaving a note here to show you what I see on my Galaxy Note 9 using Firefox:

Galaxy Note 9 using Firefox screenshot

Screenshot_20220528-193610_Firefox.jpg

@ExperimentsInHonesty
Copy link
Member

@kathrynsilvaconway I have reopened this issue. It's actually larger than originally thought, but let's have you make a new issue for the outstanding problem.

Here is the problem.

You created a new dropdown filter css instead of using one that already existed. Its not surprising that you didn't use one that already existed, because it's in a specific SCSS file related to another page. The solution should have been to examine the elements folder, see there was no SCSS file for filter, identify which pages that have filters (projects page, wins page). Get the classes from those pages SCSS file and figure out how to make a new SCSS file that will serve both those pages and the new one you built.

@kathrynsilvaconway
Copy link
Member

@kathrynsilvaconway I have reopened this issue. It's actually larger than originally thought, but let's have you make a new issue for the outstanding problem.

Here is the problem.

You created a new dropdown filter css instead of using one that already existed. Its not surprising that you didn't use one that already existed, because it's in a specific SCSS file related to another page. The solution should have been to examine the elements folder, see there was no SCSS file for filter, identify which pages that have filters (projects page, wins page). Get the classes from those pages SCSS file and figure out how to make a new SCSS file that will serve both those pages and the new one you built.

Hi Bonnie, I think I was way off base with this issue from the start. I thought they justed wanted me to take the toolkit navbar and put it in a dropdown menu. It didn't seem like it had any relationship to the filter menus on the other pages. I will at how the other files in the elements folder are built and start writing an issue this coming week.

@ExperimentsInHonesty
Copy link
Member

@kathrynsilvaconway please put a link to the new issue here, once you create it. Then you can close this issue 😀

@kathrynsilvaconway
Copy link
Member

kathrynsilvaconway commented Jun 21, 2022

The elements in this issue will be reworked in the future to make the dropdown more consistent with other filter menus on the site. This menu should ultimately use the same classes used by the filters on the wins and projects pages. I have created a new issue to move all of classes for the projects filter over to the elements folder. This will allow future developers to access them more easily:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Complexity: Medium P-Feature: Toolkit https://www.hackforla.org/toolkit/ role: front end Tasks for front end developers size: 2pt Can be done in 7-12 hours Status: Updated No blockers and update is ready for review
Projects
Development

Successfully merging a pull request may close this issue.

7 participants