open-log-viewer icon indicating copy to clipboard operation
open-log-viewer copied to clipboard

Published 20 hours ago • verified publisher icon tmoreno

Highlighting via regex goes beyond match + cannot copy/paste on macOS

Open working-name opened this issue 4 years ago • 9 comments

Hi there,

It looks like regex matching is sort of like a start spot for highlighting to continue indefinitely until a next regex is matched. I personally find it more useful if highlighting limits to the full match instead.

Screen Shot 2020-09-15 at 3 02 09 PM

https://regex101.com/r/C8Zn1H/1/ - that's what the debug regex should help mark as green. https://regex101.com/r/C8Zn1H/2 - that's the error regex

As a secondary issue: I cannot copy/paste from log viewer at all, not even from the regular expression fields on 10.14.4 using latest linked dmg version 1.4.1.

working-name avatar Sep 15 '20 19:09 working-name

Hi Alan,

I've made the highlight feature work in that way because, for me, it's easier to read the logs files. I could add a checkbox to enable or disable this feature.

According to copy/paste, I can't test macOS devices, I usually develop on Windows and I'm using linux virtual machines for some testing, so it's hard for me to test on macOS but I will try.

Thanks for your feedback!

tmoreno avatar Dec 20 '20 21:12 tmoreno

Hi Alan,

I've just created a new branch where I've added the copy/paste support for macOS,

https://github.com/tmoreno/open-log-viewer/tree/feature/enable-copy-paste-on-macos

Could you build the app and check if the short cuts works correctly?

Thanks!

tmoreno avatar Dec 29 '20 17:12 tmoreno

@tmoreno I have tested that feature branch on my macOS Big Sur v11.1 and copying still does not work. To note, I am using CMD + C as I did not see a contextual menu or File navigation for copying.

styledev avatar Jan 27 '21 17:01 styledev

@styledev and @tmoreno,

I git cloned the repo, built and ran. At first, I didn't see the Edit menu either. Nor did copy/paste work. Then, I realized I didn't checkout the proper branch. Once I did git checkout enable-copy-paste-on-macos, and then rebuilt, copy/paste worked (macOS 11.3).

jhult avatar Apr 09 '21 02:04 jhult

@tmoreno, any chance you could merge this and release a new build?

jhult avatar May 14 '21 09:05 jhult

Hi @jhult @styledev @working-name

I've just published a new version with the macOS copy/paste shortcut fixed.

https://github.com/tmoreno/open-log-viewer/releases/tag/1.5.1

Enjoy!!

tmoreno avatar May 14 '21 20:05 tmoreno

@tmoreno awesome! Thank you so much, I can copy paste everywhere now. Using 10.15.7.

Sorry to be a nag but would it be possible to make the current match-everything-til-next-regex behavior optional? It's ok if not, it's still a HUGE help going through large logfiles that contain mostly useless things. Thank you, once again.

working-name avatar May 16 '21 16:05 working-name

@tmoreno, thanks! 1.5.1 works great with copy/paste on macOS 11.4 Beta.

jhult avatar May 17 '21 14:05 jhult

@tmoreno awesome! Thank you so much, I can copy paste everywhere now. Using 10.15.7.

Sorry to be a nag but would it be possible to make the current match-everything-til-next-regex behavior optional? It's ok if not, it's still a HUGE help going through large logfiles that contain mostly useless things. Thank you, once again.

Hi @working-name!

Currently, I'm working in other side project and I don't have enough free time 😅

I have planned to do this, and other features, as soon as possible.

Thanks for your feedback.

tmoreno avatar May 19 '21 14:05 tmoreno