Skip to content

Conversation

@mitchcapper
Copy link

actual differences: mitchcapper/RegExpress@ai_assist...more_improvements

Main Features:

  • Biggest add in the new "Inspect" or scope option. Next to each match/group/capture is a magnifier clicking this opens a new tab with all the same settings and the text set to just the value (the match/group or capture text it is next to). Idea being if you have a regex to extract all the movies from a listing you can then scope to the per-movie entry to create a regex quickly to get the details from it
  • When we switched to the more modern UI I think I broke the proper wrap functionality this fixes it. The original also didn't account for \r/\n vs \r\n for width calculations and this takes care of that (also any other specials that take up more than one char were always counted as one for total length calc).
  • The original improvements added ability to limit the engines loaded at startup, this allows specifying more than one
  • Use Environment.Exit to bail out if an existing process was there. Calling Shutdown() still called the mainwindow constructor which in turn resulted in us trying to save session state before we loaded it. This would essentially discard the saved session state. This wasn't noticed because the existing process would just write it again before it exited. Instead Environment.Exit avoids constructing MainWindow at all.

In terms of merging officially
improvements-> ai_assist -> this PR -> html agility

I could likely manually merge these into the improvements branches just not quite as frictionless. If any commits/items are not merged I can rebase as needed.

@mitchcapper
Copy link
Author

Screenshot of new inspect button:
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant