@Mit-Patwa Pushed a couple of cleanup commits (was easier than leaving comments!) and I also noticed a bug with the Discover menus - on main the shadow is white but here the shadow is black. Could you check?

@binary-koan sorry for the late reply ,I was a little caught up.
I'm not able to see this black shadow on my branch,it seems to work fine?

@binary-koan _a assigning this to you since you reviewed the previous PRs, let me know if you can't though.
@Mit-Patwa can you fix the conflicts in the mean time?

@binary-koan I was able to get the black shadow, this happens when I clicked on one of the discover categories for the first time, this was happening because of aria-[expanded=true]:shadow!(see here )since aria-expanded value is false when clicked for the first time.
Solution: use aria-[current=true]:hover:shadow! instead.

@binary-koan can we merge?

Sorry for the wait, will hopefully get to this tomorrow

Okay! I’m just a little excited since it’s my first merge :)

Could you check if the failing tests are related?

@binary-koan a bunch of tests are failing because of changed formatting, (No visual changes)
one of the failing tests :

Failure/Error: expect(page).to have_radio_button("First Tier", text: "$3 $1.47 a month")
  expected to find visible radio_button "First Tier" with text "$3 $1.47 a month" but there were no matches. Also found "$3\n$1.47 a month\nFirst Tier", which matched the selector but not all filters.

What's the best approach? just modifying the test cases to expect:
expect(page).to have_radio_button("First Tier", text: "$3 \n$1.47 a month")
instead of
expect(page).to have_radio_button("First Tier", text: "$3 $1.47 a month")

The normalize_ws option is probably better, but that's fine if there are cases it doesn't work

@binary-koan could you help me out with the test cases,when I try to run the failing tests locally, I get a 502 bad gateway error before the testing even starts , because of which I get false test failures.What's the solution?

@binary-koan could you trigger CI the tests should pass now, also I noticed you changed some of the tailwind utilities in the cleanup commits , did they work fine?

Added a couple of minor fixes (realised my cleanup did break the small pill styles, my bad ...), will go ahead and merge now. Thanks again for the contribution!

Awesome!!!Thanks for my first merge,what's the process for claiming the bounty?