Skip to content
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

[DRAFT] [DNM] Suggestions for #10591 #1

Open
wants to merge 4 commits into
base: remove_disable_run
Choose a base branch
from

Conversation

aparajit-pratap
Copy link

[DRAFT] Suggestions for DynamoDS#10591

@aparajit-pratap aparajit-pratap changed the title [DRAFT] Suggestions for #10591 [DRAFT] [DNM] Suggestions for #10591 Apr 26, 2020
@@ -137,7 +137,7 @@ private void OnViewModelMessageLogged(ILogMessage msg)

private void AddToSidebar(bool displayDefaultContent)
{
// verify the browser window has been initialised
// verify the browser window has been initialized
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ignore

@@ -21,8 +21,6 @@
src: url("fonts/ArtifaktElement-Regular.woff") format("woff");
color: white;
background-color: #353535;
overflow-x: hidden;
overflow-y: hidden;
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ignore

@@ -21,8 +21,6 @@
src: url("fonts/ArtifaktElement-Regular.woff") format("woff");
color: white;
background-color: #353535;
overflow-x: hidden;
overflow-y: hidden;
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ignore

@@ -14,8 +14,6 @@
src: url("fonts/ArtifaktElement-Regular.woff") format("woff");
background-color: #353535;
padding: 1%;
overflow-x: hidden;
overflow-y: hidden;
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ignore

@@ -307,7 +307,7 @@ public struct InfoBubbleDataPacket
public Point TopLeft;
public Point BotRight;
public string Text;
public Uri Link;
internal Uri Link;
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ignore

mmisol pushed a commit that referenced this pull request May 27, 2020
* Add Engine property to PythonNodeBase

* Add engine selector dropdown to python editor

* Add notification API to nodemodel

* add warnings to resources

* Add IronPython dialog on open saved workspace

* Add Engine label on node

* make LogNotification internal

* python engine enum

* Python migration extension DRAFT (#1)

* Python migration extension DRAFT

* update extension

* small cleanup

* move dialog and notification strings to extension resource

* Update PythonMigrationViewExtension.cs

* added python engine label to From String Node

* add engine selector to python script from string node in context menu

* Update EngineLabel.xaml

* comment updates

* Update Dynamo.All.sln

* Update PythonMigrationViewExtension.cs

* Update PythonMigrationViewExtension.cs

* comment updates

* Update Resources.Designer.cs

* update csproj + small cleanup

* added more information btn to IronPythonDialog

* comment updates

* tests added

* tests

* comment updates

* Updates dialog, notification and documentationbrowser text

* update tests

* fix dialog box and small clean up

* warning html update

* updates to warning messages

* Update PythonStringNode.cs

Co-authored-by: Michael Dewberry <[email protected]>
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.

2 participants