Skip to content

feat(Shared): Migrate Image#882

Closed
kevinvangelder wants to merge 35 commits intomicrosoft:masterfrom
infinitered:shared-project-image
Closed

feat(Shared): Migrate Image#882
kevinvangelder wants to merge 35 commits intomicrosoft:masterfrom
infinitered:shared-project-image

Conversation

@kevinvangelder
Copy link
Contributor

No description provided.

pre10der89 and others added 30 commits October 31, 2016 09:49
* Fixed some crashes when starting due to differences between Net46 and UWP

* Converted Net46 Dialogs to be shown as modeless to ensure execution can continue (e.g. progress dialog)
…sign data file/classes

* Added ScrollViewer to RedBoxDialog
…tures since KeyDown events are not reaching RootView without

  making the RootView Focusable and explictly setting focus on it...

* Styled the Developer Options dialog...

* Fixed crash when saving application settings via the developer options dialog
…nsured the Focus was applied to it by the ReactPage OnCreate

* Restore support for keyboard accelerators to bring up dev options, changed dev option accelerators to match other React Native Clients.

* Fix issue in getting the correct bundle path

* Added support to copy the ChakraCore DLL to the Playground Net46 output directory
var waitHandle = new AutoResetEvent(false);
var json = default(JObject);
bool websocketFailedCalled = false;

Copy link
Contributor

Choose a reason for hiding this comment

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

Take a look at a bunch of the comments from the ScrollView review, same stuff needs to be cleaned up here.

@rozele
Copy link
Contributor

rozele commented Nov 16, 2016

🕐

@@ -1,4 +1,4 @@
using Newtonsoft.Json.Linq;
using Newtonsoft.Json.Linq;
Copy link
Contributor Author

@kevinvangelder kevinvangelder Nov 17, 2016

Choose a reason for hiding this comment

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

¯\_(ツ)_/¯

@kevinvangelder
Copy link
Contributor Author

Replaced by #895.

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.

3 participants