-
Notifications
You must be signed in to change notification settings - Fork 63
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
Add Reload button to LiveSPICE VST #212
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,6 +3,7 @@ | |
using System.IO; | ||
using System.Windows; | ||
using System.Windows.Controls; | ||
using static System.Windows.Forms.VisualStyles.VisualStyleElement.Window; | ||
|
||
namespace LiveSPICEVst | ||
{ | ||
|
@@ -95,9 +96,22 @@ private void ShowAboutButton_Click(object sender, RoutedEventArgs e) | |
{ | ||
About about = new About() { Owner = Window.GetWindow(this) }; | ||
about.ShowDialog(); | ||
} | ||
} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It looks like you have a mix of tabs and spaces here, which is rendering differently than it did in your editor (presumably). I'm pretty sure we use spaces everywhere, can you reformat using spaces only? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I have no idea how that happened, since I always use spaces. In any case, thanks for noticing. I've fixed the formatting |
||
|
||
private void ReloadCircuitButton_Click(object sender, RoutedEventArgs e) | ||
{ | ||
if (string.IsNullOrEmpty(Plugin.SchematicPath)) | ||
{ | ||
return; | ||
} | ||
|
||
Plugin.LoadSchematic(Plugin.SchematicPath); | ||
|
||
UpdateSchematic(); | ||
} | ||
|
||
|
||
private void ShowCircuitButton_Click(object sender, RoutedEventArgs e) | ||
private void ShowCircuitButton_Click(object sender, RoutedEventArgs e) | ||
{ | ||
if (Plugin.SimulationProcessor.Schematic != null) | ||
{ | ||
|
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.
Is this actually used? May have been an accidental intellisense addition? (That happens to me a lot.)
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.
You are absolutely right. I installed Visual Studio just to open this project and since I haven't used it in well over 5-6 years, I forgot to check the intellisense additions, which is my bad. I removed the unnecessary line now and everything should be as requested. If you have any other comments, let me know!