Confusing //console.log comments in code base

[baterson]

Hello @techniq I've been exploring the code base and find some debug comments like this:

// console.log('clicked:menuItems', e.target, menuEl);

svelte-ux/packages/svelte-ux/src/lib/components/Menu.svelte

Line 51 in f419435

// console.log('clicked:menuItems', e.target, menuEl);

If someone just forgot to remove it, I would be happy to contribute to remove similar comments :)

[techniq]

Hey @baterson 👋. There is definitely some cleanup that can be done with console.log.

One thing I plan to do is leverage Logger which lets you opt into these messages by component/level at runtime via localStorage.logger = '...'. I think I left many of these as breadcrumbs to add (see issue). If you would want to start that process of using Logger that would be awesome!

Using SelectField as a reference should help, but you basically just create an instance for the component...

const logger = new Logger('Menu');

then replace console.log() with logger.debug(), logger.info(), logger.error(), etc.

[techniq]

Closed via #452