SketchUp / testup-2

TestUp 2 for SketchUp - A GUI wrapper for running Minitest in SketchUp
MIT License
26 stars 37 forks source link

Reload referencing missing icon #196

Open Eneroth3 opened 3 years ago

Eneroth3 commented 3 years ago

The reload command references a "arrow_refresh.png" icon. This file however appears to be missing.

It seems these lines adding the icon could just be removed, as this command is primarily added to a menu (only added to toolbar if extension is in debug mode). An image file could also be added.

thomthom commented 3 years ago

Ah, this is a debug command.

https://github.com/SketchUp/testup-2/blob/e3891c9131dc425a794743046b540117419a0a6c/src/testup/ui.rb#L105-L116

It's always added to the Debug menu:

https://github.com/SketchUp/testup-2/blob/e3891c9131dc425a794743046b540117419a0a6c/src/testup/ui.rb#L170

And conditionally added to the toolbar:

https://github.com/SketchUp/testup-2/blob/e3891c9131dc425a794743046b540117419a0a6c/src/testup/ui.rb#L189-L192