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

ER: Filters on project page #4530

Closed
4 of 5 tasks
JessicaLucindaCheng opened this issue Apr 17, 2023 · 12 comments
Closed
4 of 5 tasks

ER: Filters on project page #4530

JessicaLucindaCheng opened this issue Apr 17, 2023 · 12 comments
Labels
Bug Something isn't working Complexity: Large Complexity: See issue making label See the Issue Making label to understand the issue writing difficulty level ER Emergent Request Issue Making: Level 2 Make issue(s) from an ER or Epic P-Feature: Home page https://www.hackforla.org/ P-Feature: Projects page https://www.hackforla.org/projects/ role: front end Tasks for front end developers size: 1pt Can be done in 4-6 hours

Comments

@JessicaLucindaCheng
Copy link
Member

JessicaLucindaCheng commented Apr 17, 2023

Emergent Requirement - Problem

On the current homepage and Projects page, I can't see all the filter categories unless I do one of the following:

  • Scroll all the way through all the projects
  • Or collapse the Program Area filter in order to see the Status filter category or be able to see the View all in the Languages/Technologies filter category

To see the problem I am seeing, view the https://www.hackforla.org/projects/ page using Developer Tools and inspect it at 1024 x 659 and look at the filters on the left side while scrolling down the page.

Click here to see the problem with not being able to see all the filters while scrolling

Issue you discovered this emergent requirement in

  • Not applicable

Date discovered

2023-Apr-17

Did you have to do something temporarily

  • YES
  • NO

Scroll all the way through the projects or collapse a filter category

Who was involved

@JessicaLucindaCheng

What happens if this is not addressed

It's not user friendly

Resources

Recommended Action Items

  • Discuss with Stakeholder Rep (Bonnie) potential solutions
  • Make a new issue
  • Discuss with team

Potential solutions [draft]

@JessicaLucindaCheng JessicaLucindaCheng added Bug Something isn't working role: back end/devOps Tasks for back-end developers Complexity: Medium P-Feature: Projects page https://www.hackforla.org/projects/ size: 1pt Can be done in 4-6 hours Ready for Prioritization ready for dev lead Issues that tech leads or merge team members need to follow up on Issue Making: Level 2 Make issue(s) from an ER or Epic and removed Ready for Prioritization labels Apr 17, 2023
@ExperimentsInHonesty
Copy link
Member

ExperimentsInHonesty commented Apr 25, 2023

We should make a new issue to fix it so that it operates the same way as REI and Amazon (pined to the top)

Also filter should be pinned to content
image
image

@ExperimentsInHonesty ExperimentsInHonesty added role: front end Tasks for front end developers and removed role: back end/devOps Tasks for back-end developers labels Apr 25, 2023
@JessicaLucindaCheng JessicaLucindaCheng added ready for dev lead Issues that tech leads or merge team members need to follow up on and removed ready for dev lead Issues that tech leads or merge team members need to follow up on labels Apr 25, 2023
@chrismenke45 chrismenke45 self-assigned this Apr 26, 2023
@github-actions

This comment was marked as outdated.

@chrismenke45 chrismenke45 removed their assignment Apr 26, 2023
@Adastros Adastros self-assigned this May 2, 2023
@github-actions

This comment was marked as outdated.

@Adastros

This comment was marked as outdated.

@Adastros
Copy link
Member

Adastros commented May 2, 2023

@Adastros

This comment was marked as outdated.

@ExperimentsInHonesty ExperimentsInHonesty added the Complexity: See issue making label See the Issue Making label to understand the issue writing difficulty level label Oct 10, 2023
@Anahisv23
Copy link
Member

Hi @JessicaLucindaCheng @Adastros, I'm working on issue #4593. My peers and I had a question regarding the issue. The issue states that the search bar needs to be pinned on viewports width above 767px (desktop view). Is pinning the search bar for viewports below 767px (mobile view) for the homepage and project page something we want to address in this issue or in another issue? Thank you!

@ExperimentsInHonesty
Copy link
Member

ExperimentsInHonesty commented Jun 18, 2024

Hi @JessicaLucindaCheng @Adastros, I'm working on issue #4593. My peers and I had a question regarding the issue. The issue states that the search bar needs to be pinned on viewports width above 767px (desktop view). Is pinning the search bar for viewports below 767px (mobile view) for the homepage and project page something we want to address in this issue or in another issue? Thank you!

@Anahisv23 Good question. We are going to address the mobile separately in another issue

@ExperimentsInHonesty
Copy link
Member

ExperimentsInHonesty commented Jun 18, 2024

Notes for person making new ER

  • We need to make a new issue to answer how we want the filter to behave when it's in mobile

How we want it to behave

  • It will freeze the filter and search at the top as you scroll down.
  • The applied filters will only be visible when you open the filter hamburger menu
  • see REI.com website

What we need in order to make the new ER

  • GIF of mobile of current site behavior
  • Screenshot of REI
  • revised viewport width below 767px

Once this comment has been made into an ER, then reassign Adastros to this issue, and close it.

@ExperimentsInHonesty ExperimentsInHonesty added Draft Issue is still in the process of being created ready for merge team needs a senior review either to do some re writing or to approve it for ready for prioritization and removed ready for dev lead Issues that tech leads or merge team members need to follow up on labels Jun 18, 2024
@Anahisv23
Copy link
Member

Hi @JessicaLucindaCheng @Adastros, I'm working on issue #4593. My peers and I had a question regarding the issue. The issue states that the search bar needs to be pinned on viewports width above 767px (desktop view). Is pinning the search bar for viewports below 767px (mobile view) for the homepage and project page something we want to address in this issue or in another issue? Thank you!

@Anahisv23 Good question. We are going to address the mobile separately in another issue

Gotcha. Thank you for the clarification!

@ExperimentsInHonesty

This comment was marked as outdated.

@ExperimentsInHonesty
Copy link
Member

@ExperimentsInHonesty ExperimentsInHonesty removed Draft Issue is still in the process of being created ready for merge team needs a senior review either to do some re writing or to approve it for ready for prioritization labels Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Complexity: Large Complexity: See issue making label See the Issue Making label to understand the issue writing difficulty level ER Emergent Request Issue Making: Level 2 Make issue(s) from an ER or Epic P-Feature: Home page https://www.hackforla.org/ P-Feature: Projects page https://www.hackforla.org/projects/ role: front end Tasks for front end developers size: 1pt Can be done in 4-6 hours
Projects
Development

No branches or pull requests

6 participants