Fix incorrect matching of skycoords with pixel coords in catalog results

[rosteen]

This was previously matching the filtered list of N pixel coords in the viewer bounds with the first N skycoords, which led to mismatches between the pixel and sky coords for the results. There's a smarter fix here that involves zipping before filtering both at once rather than doing the check in the loop, this was mostly a quick hack to get the code working properly. If we're worried about performance I can figure out the better way to do this, if not we can merge this for now to fix the bug.

[codecov]

Codecov Report

Attention: Patch coverage is 88.88889% with 1 line in your changes missing coverage. Please review.

Project coverage is 87.55%. Comparing base (d8a70b2) to head (0eecb3b).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
jdaviz/configs/imviz/plugins/catalogs/catalogs.py	88.88%	1 Missing ⚠️

❌ Your patch check has failed because the patch coverage (88.88%) is below the target coverage (90.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3464   +/-   ##
=======================================
  Coverage   87.54%   87.55%           
=======================================
  Files         128      128           
  Lines       19959    19955    -4     
=======================================
- Hits        17474    17471    -3     
+ Misses       2485     2484    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[pllim]

Do we need a regression test?