paranext / paranext-core

Electron client, extension host, and C# library for Paranext
https://paranext.github.io/paranext-core/
MIT License
14 stars 2 forks source link

Fixed top bar spacing, catch and explain data provider errors as a quick solution #848

Closed tjcouch-sil closed 2 months ago

tjcouch-sil commented 2 months ago

I noticed we had a bunch of errors in the console from menu data problems, so I investigated and made some quick tweaks to make them (and hopefully all other data provider subscriber errors) a bit less cryptic and scary. It would probably be best to do something better for this long-term, but it seems that giving more information is probably about the only easy-ish thing to do right now since it's all event-based and we currently offer no function callbacks for error handling in events. This isn't quite as "fail fast" as we usually are, but the errors are largely out of the hands of the developer down this low. Hopefully these try/catches will at least help somewhat.

Also there was a small spacing issue on the BCV control that is also fixed.

image


This change is Reviewable