-
Notifications
You must be signed in to change notification settings - Fork 204
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Clickable code search results line numbers. #1579
base: master
Are you sure you want to change the base?
Clickable code search results line numbers. #1579
Conversation
54a7131
to
2029263
Compare
Download the artifacts for this pull request here: GUI:
CLI: |
2029263
to
424d807
Compare
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
got no thoughts right now on how to get rid of the tuple, so fine by me with the exception of grammar.
the whole search thing needs to be rewritten in the future anyway. so lets not stall this feature for that.
@@ -298,25 +305,25 @@ private async void UserControl_DataContextChanged(object sender, DependencyPrope | |||
CurrentDecompiled = null; | |||
CurrentDisassembled = null; | |||
|
|||
if (MainWindow.CodeEditorDecompile != Unstated) //if opened from the code search results "link" | |||
if (EditorTab != CodeEditorTab.Unknown) // if opened from the code search results "link" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please use Uppercase If
in comment.
int totalLineCount = resultsDict.Select(x => x.Value.Count).Sum(); | ||
bool tooManyLines = totalLineCount > 10000; | ||
if (tooManyLines) | ||
mainWindow.ShowWarning($"There are too many code lines to display ({totalLineCount}), so there would be no clickable line numbers."); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mainWindow.ShowWarning($"There are too many code lines to display ({totalLineCount}), so there would be no clickable line numbers."); | |
mainWindow.ShowWarning($"There are too many code lines to display ({totalLineCount}), so no clickable line numbers are shown."); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, what if "...so the line numbers won't be clickable."?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"won't" (will not) is future. you want present though.
Whether you use "so the line numbers aren't clickable" or "so no clickable line numbers are shown" doesnt really matter for me.
Description
CodeEditorDecompile
) and enum (CodeEditorMode
) out from theMainWindow
, and also renamed them.overwrite
argument forTab.SaveTabContentState()
.Fixes #1580.