search

abey79 / vsvg

Fast and portable tools for plotter users
http://whisk.rs
MIT License
107 stars 12 forks source link

Add "Inspect" section to display debug values #132

Closed afternoon2 closed 5 months ago

afternoon2 commented 7 months ago
afternoon2 commented 7 months ago

As a user, I'd still opt for a rename possibility (inspect sounds less alarming, but still in a "checking if everything's fine" manner rather than presenting program's valid features). And if not via the runner's method, then maybe with ctx.inspect_label(label: &str)?

abey79 commented 7 months ago

As a user, I'd still opt for a rename possibility (inspect sounds less alarming, but still in a "checking if everything's fine" manner rather than presenting program's valid features). And if not via the runner's method, then maybe with ctx.inspect_label(label: &str)?

This really feels like API clutter, regardless of where it lives. What's the use case here? What problem would that solve?

afternoon2 commented 7 months ago

just a customization option, but I won't fight for it

afternoon2 commented 7 months ago

ah, forgot to add a label for the PR, sorry 🙏🏻

abey79 commented 7 months ago

ah, forgot to add a label for the PR, sorry 🙏🏻

No prob, I'm not even sure they are settable by external contributors 🤷🏻

abey79 commented 5 months ago

Sorry for the humongous delay! 🙏🏻