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

Progressively enhanced combobox for developer specialties (#820) #868

Open
wants to merge 50 commits into
base: main
Choose a base branch
from

Conversation

MiaSinek
Copy link

@MiaSinek MiaSinek commented May 22, 2023

Demo

Desktop - Javascript enabled

specialties-desktop

Desktop - Javascript disabled

specialties-desktop-js-disabled

Mobile - Javascript enabled

specialties-mobile

Mobile - Javascript disabled

Filtering functionality is not working, because opening the filter sidebar requires Javascript

PR Summary 🌟

This PR is addressing issue #820.

  • Introducing a new dropdown component!
    I created a shiny new component for displaying dropdowns featuring various specialties. Now, both query_component and query_mobile_component make use of it, instead of rendering them separately. 🎉

  • Magical new controller for pills! 💊
    I crafted a powerful controller that takes care of adding and removing specialties, displaying them as pills above the Specialty search field. It'll also ensure the seamless syncing of checkboxes and their states. Because of this, the search URL can be reloaded/bookmarked, and the pill state is restored correctly. 🧙

  • Form autosubmit refactoring for better control! 🎛️
    By enhancing the forms-autosubmit, I enable you to debounce form submissions in other instances. This way, we reduce code duplication and give you more control over submission frequency! 🚀

  • Easier developer filtering experience! 🔍
    Your life just got easier! Now, the update allows you to filter developers based on their specialty. The search results have also received an upgrade, leading to a refreshing new interface. 🥳

Pull request checklist

  • My code contains tests covering the code I modified
  • I linted and tested the project with bin/check
  • I added significant changes and product updates to the changelog

@MiaSinek MiaSinek changed the title 820 progressively enhanced combobox for developer specialties Progressively enhanced combobox for developer specialties (#820) May 22, 2023
@joemasilotti
Copy link
Owner

Hey @MiaSinek - thanks again for your patience on this PR. I'm enabling developer specialties BETA soon and am hoping to address this PR very soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants