Skip to content

Commit

Permalink
copyedited comments and documentation
Browse files Browse the repository at this point in the history
  • Loading branch information
EliahKagan committed Sep 17, 2021
1 parent 2157377 commit 02c2c50
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 14 deletions.
6 changes: 3 additions & 3 deletions BUGS.md
Original file line number Diff line number Diff line change
Expand Up @@ -103,9 +103,9 @@ rewritten.

As commented on `PanelSwitcher.TrySwitch`, the way program-driven switching
between LINQPad output panels is implemented is by circumventing encapsulation
on on the `OutputPanel` object (which runs in the query process, but is
supplied by a library, being present as a separate assembly from the query
assembly) and calling the `Activate` method.
on the `OutputPanel` object (which runs in the query process, but is supplied
by a library, being present as a separate assembly from the query assembly) and
calling the `Activate` method.

The comment gives details about why I don”t think any currently available
good means of doing this exist. But as long as I’m doing that, Flood
Expand Down
7 changes: 4 additions & 3 deletions doc/help.html
Original file line number Diff line number Diff line change
Expand Up @@ -909,11 +909,12 @@ <h3 id="platform">
<a class="external"
href="https://www.linqpad.net/"
title="LINQPad website">LINQPad</a>
is a properietary freeware application written by Joseph Albahari.
Flood&rsquo;s charting feature uses the version of
is a proprietary freeware program written by Joseph Albahari (with
paid editions offering more features). Flood&rsquo;s charting
feature uses the version of
<a class="external wide"
href="https://github.com/dotnet/winforms-datavisualization"
title="System.Windows.Forms.DataVisualization on GitHub">System.Windows.Forms.DataVisualization</a>
title="System.Windows.Forms.DataVisualization on GitHub">Sys&shy;tem.Win&shy;dows.Forms.Data&shy;Visualization</a>
that is included in LINQPad 6 (which I believe is
<a class="external wide"
href="https://github.com/albahari/winforms-datavisualization"
Expand Down
16 changes: 8 additions & 8 deletions flood.linq
Original file line number Diff line number Diff line change
Expand Up @@ -434,7 +434,7 @@ internal sealed class PanelSwitcher : Component, IPanelSwitcher {
// PanelManager.GetOutputPanels() returns an array of output panels, and
// writing to Util.SelectedOutputPanelIndex switches panels. When output
// panels are created in such a way as to be listed from left to right in
// the order of creation--such as when they are created sequentally by
// the order of creation--such as when they are created sequentially by
// interacting with LINQPad controls in the Results panel--they are indexed
// in the same order and it is sufficient to add and subtract 1 [since
// Util.SelectedOutputPanelIndex is 0 for the Results panel, which is not
Expand Down Expand Up @@ -557,7 +557,7 @@ internal sealed class PanelSwitcher : Component, IPanelSwitcher {

/// <summary>
/// The main user interface, containing an interactive canvas, an info bar, and
/// expandable/collapsable tips.
/// expandable/collapsible tips.
/// </summary>
internal sealed class MainPanel : TableLayoutPanel {
internal static int DefaultDelayInMilliseconds { get; } = 10;
Expand Down Expand Up @@ -869,7 +869,7 @@ internal sealed class MainPanel : TableLayoutPanel {

var detail = (_jobsCharting, _charting.Checked) switch {
(0, false) =>
"Also, no currenty running fills are charting.",
"Also, no currently running fills are charting.",
(0, true) =>
"But no currently running fills are charting.",
(1, false) =>
Expand Down Expand Up @@ -1137,7 +1137,7 @@ internal sealed class MainPanel : TableLayoutPanel {
// the charting button, since users are likely to accidentally turn
// charting on or off in that situation. This is mostly important when
// the stop button is disabled due to the last fill completing, since
// the user could press Spacebar or Enter just a monent too late (and
// the user could press Spacebar or Enter just a moment too late (and
// not notice that charting was affected). But I think focus shouldn't
// go to the charting button due to successfully using the stop button
// either.
Expand Down Expand Up @@ -1204,7 +1204,7 @@ internal sealed class MainPanel : TableLayoutPanel {
{
if (_jobsCharting <= 0) {
throw new InvalidOperationException(
"Bug: negatvely many jobs are charting?!");
"Bug: negatively many jobs are charting?!");
}

--_jobsCharting;
Expand Down Expand Up @@ -1458,7 +1458,7 @@ internal interface IAlertCookie {
bool IsCurrent { get; }
}

/// <summary>A horizontal bar to show dismissable text-based alerts.</summary>
/// <summary>A horizontal bar to show dismissible text-based alerts.</summary>
internal sealed class AlertBar : TableLayoutPanel {
internal AlertBar(ToolTip toolTip)
{
Expand Down Expand Up @@ -1926,7 +1926,7 @@ internal sealed record
CheckBoxBitmapFilenamePair(string CheckedFilename, string UncheckedFilename);

/// <summary>
/// Background images for the checked and uchecked states of a button-style
/// Background images for the checked and unchecked states of a button-style
/// checkbox.
/// </summary>
/// <remarks>See <see cref="CheckBoxBitmapFilenamePair"/>.</remarks>
Expand Down Expand Up @@ -2472,7 +2472,7 @@ internal sealed class WebView2HelpViewer : HelpViewer {
// (1) Find a better workaround to be used until dependencies
// are bundled, as well as on a future non-bundling branch.
//
// (2) If this workaround is still in place when dependnecies are
// (2) If this workaround is still in place when dependencies are
// bundled, remove it (from all active non-CDN branches).
AdditionalBrowserArguments = "--disable-web-security",
};
Expand Down

0 comments on commit 02c2c50

Please sign in to comment.