SynchronizedView with TView parameters only
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 35/100
Research direction
Review the SynchronizedViewChangedEventArgs<T, TView> example in the issue and compare it with the described View.cs and .uxml separation. Determine whether a SynchronizedView exposing only TView fits the existing API, then document the decision and any required API changes.
Written by the indexing model from the issue text.
Description
I have never worked with mvvm targeted frameworks. I implement mvvm in Unity and learn pattern from articles. So I might be misunderstanding the pattern. Isn't SynchronizedViewChangedEventArgs<T, TView> violates encapsulation "view doesn't know about data" in that example?
var model = new ObservableList<SomeData>();
var vm = model.CreateView(data => CreateView(someData)); //generates view for model data
var view = new ScrollView(); //view has field vm
vm.ObserveAdd().Subscribe(evt => view.Add(evt.Value.View)); //view observes vm and have acces to the evt.Value.Value (model data)
Currently, I use a View.cs file alongside .uxml to just bind uxml to the code. So I put data related logic in the vm and appearance related logic in the view (drag&drop ScrollView element). This is why view wants to look at vm.SynchronizedView, but access only TView without T. Does it make sense to create TView only SynchronizedView?
- Dominant language
- C#
- Stars
- 1k
- Forks
- 73
- Avg merge
- 1d 14h
- Merged PRs (30d)
- 3
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from Cysharp/ObservableCollections
-
Difficulty 4/5 1-2 days Newbie friendliness 48/100
Cysharp/ObservableCollections#142 ·
-
Difficulty 4/5 1-2 days Newbie friendliness 54/100
Cysharp/ObservableCollections#141 ·
-
ObserveCountChanged: a count change during Subscribe is silently dropped — the handler is attached before countPrev is initializedPossibly taken @armadavalor claimed this 8 days ago. Open
Difficulty 3/5 1-2 days Newbie friendliness 76/100
Cysharp/ObservableCollections#132 ·
-
Difficulty 5/5 Over a week Newbie friendliness 30/100
Cysharp/ObservableCollections#131 ·
-
Difficulty 4/5 3-5 days Newbie friendliness 48/100
Cysharp/ObservableCollections#128 ·
All issues in Cysharp/ObservableCollections
Similar issues
-
bug needs-triage
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
microsoft/microsoft-ui-xaml#12515 · 1 comment · 1 reaction ·
Maintainers usually reply within 2 days
-
[v5-preview5] `func new` displays incorrect command in error message when project is not initializedOpenNeeds: Triage (Functions) v5 v5-preview5
Difficulty 1/5 Under an hour Newbie friendliness 85/100
Azure/azure-functions-core-tools#5838 ·
Maintainers usually reply within 2 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
Maintainers usually reply within 1 day
-
customer-reported needs-team-triage question server-Azure.Mcp tools-Pricing
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
microsoft/mcp#4087 · 1 comment ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100