-
Notifications
You must be signed in to change notification settings - Fork 121
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
some help on upgrading to 1.0.0 #39
Comments
the rendering issue is related to high dpi of my screen, it was set to 200% after set to 100% then it displays correctly. However i cant use the screen anymore without getting a headheack. maybe its possible to configure neutronium to support hidpi? cefsharp/CefSharp#1803 |
Hello @jmecosta , could you open a dedicated issue for dpi support please? Regarding the migration I assume that you read the migration guide.
Do you mean that the data first show-up correctly but are not updated when you change the property value and send an INotifyPropertyChanged event?
Check if this will not solve your problem, |
@David-Desmaisons ive read the upgrade guide, the parts that were relevant to my case :) and then i got a exception regarding the wpf dispatcher. And most of my code was already calling the dispatch method. im not so sure about that data is actually read for the viewmodel when it starts since it that case it would display nothing. And its displaying the dummy data i have by default in the javascript file. i will attach my code in a bit |
the relevatn files are packagemanager.html and js/package-manager.js |
Could you open the Vue devtool and check the App data? |
@David-Desmaisons thanks i think its related with section ViewModel binding my app was not done from template, but its not knockout either. So i have the viewmodel set to the object as the documentation says but i dont know how to readjust the binding to point to new object. should be quite simple :) |
@jmecosta , you are correct: now you have this addicional ViewModel property. You should rewrite your binding taking in consideration this addicional property. |
Hello @jmecosta any news on this? Thanks. |
@David-Desmaisons i dont understand how to connect data still :( commands are fired propertly from javascript, but that is easy to understnad by just calling ViewModel.Command. But for example for a grid data:
were i have in mixin:
How do i connect this to viewmodel? ive tried :data="ViewModel.Packages" but that did not work either |
In fact, with our mixin, after injection of C# properties, application data looks like: ├── ViewModel
├── Packages
├── gridColumns
├── InvalidReason
├── Versions
└── searchQuery I guess you want something like that: └── ViewModel
├── Packages
├── gridColumns
├── InvalidReason
├── Versions
└── searchQuery You can handle this in different ways:
Either way should work |
all good, thanks for the help.
was what i was missing. |
@David-Desmaisons hi
i now tried upgrading my application to new version, and i can no longer update the ui from the c# side. one of the things ive updated was i think related with vue 2.0 (https://vuejs.org/v2/guide/components.html#data-Must-Be-a-Function) and also window.glueHelper.setOption({ mixins: [localMixin] });
my debug log does not show anything relevant:
The thread 0x6b10 has exited with code 0 (0x0).
Neutronium - Browser Log: Message: Neutronium: ViewModel injected, source: , line number: 11183, page: file:///C:/prod/buildallextension/BuildDrop/vs17/packagemanager.html
Maybe unrelated issue, and likely caused by my laptop computer resolutioon but i can get rendering properly done. It was also in previous version. Also the example show similar issue. Have you come accross this:
thanks
The text was updated successfully, but these errors were encountered: