theodinproject icon indicating copy to clipboard operation
theodinproject copied to clipboard

Enhancement: add search to Odin Project site

Open ashoofly opened this issue 2 years ago • 5 comments

Complete the following REQUIRED checkboxes:

  • [x] I have thoroughly read and understand The Odin Project Contributing Guide
  • [x] The title of this issue follows the brief description of request format, e.g. Add dark mode to website

The following checkbox is OPTIONAL:

  • [x] I would like to be assigned this issue to work on it

1. Description of the Feature Request:

Adding search to the Odin Project curriculum would help a lot. There are so many useful things in the lessons, but I sometimes forget where I have read it. If I really need to find it, I'll come to Github and search the actual curriculum source ha.

2. Acceptance Criteria:

  • [ ] Add search bar to top-level of Odin Project
  • [ ] Implement search across entire curriculum

3. Additional Information:

ashoofly avatar Jun 09 '22 18:06 ashoofly

Thanks for taking the time to put this together @ashoofly ⭐

There was some discussion about this in the suggestions channel on discord a couple of months ago.

I think this would be a very valuable feature and wouldn't be that difficult to implement if we used https://github.com/Casecommons/pg_search. The hard part is where to put the search bar and the design. Our nav bar is already quite stacked with stuff.

There are some ideas about maybe moving to a side nav which would allow us to accommodate this. But we need to discuss that some more. I'll update this when I have something to share from those conversations.

KevinMulhern avatar Jun 09 '22 18:06 KevinMulhern

Once this feature gets reviewed, I could work on it.

anthonygedeon avatar Jun 15 '22 12:06 anthonygedeon

I've got some more time now, so I'd like to give it a shot once we agree on a design!

ashoofly avatar Jul 06 '22 06:07 ashoofly

Second this. This would be greatly helpful and appreciated. Hope it gets added in the near future.

Shadow5634 avatar Jul 06 '23 06:07 Shadow5634

A suggestion is to move both the Theme and Notifications icons under the profile sub-menu. The new arrangement would look like this:

  • Settings
  • Notifications
  • Theme Toggle
  • Log Out

This approach can help to de-clutter the main UI, consolidating user-centric options in a single, logical location. Users often expect to find these features within their profile or settings menu, and grouping them together can make the interface more intuitive.

This change would free up space on the main navigation, allowing a dedicated Search function represented by a magnifying glass icon. The addition of a universal symbol for search can help improve discoverability of this feature.

HarrisonRClark avatar Jul 28 '23 22:07 HarrisonRClark