Ajout du bouton by andre726 · Pull Request #26 · uwp-squad/livecoding-uwp

andre726 · 2017-03-30T14:52:47Z

No description provided.

Odonno · 2017-03-31T10:18:38Z

LIvecoding-uwp/Livecoding/ViewModels/ChatList.cs

+
+namespace Livecoding.UWP.ViewModels
+{
+    class ChatList : ListViewItem


Pourquoi cette classe ? Si elle ne fait rien de précis, elle ne doit pas exister. De plus, si c'est pour gérer l'affichage d'une ListView pour le chat, tu peux le faire sans difficulté dans la View et le ViewModel.

c'est dans quelle fichier que je dois gérer l'affichage de la liste?

Tu dois modifier 2 fichiers principalement : la StreamPage et le StreamViewModel. En gros, la vue et la logique de la vue.

Dans le ViewModel, il faut par exemple que tu ajoutes la propriété suivante :

private bool _showChat; public bool ShowChat { get { return _showChat; } set { _showChat = value; RaisePropertyChanged(); } }

Et après dans la vue, tu fais le binding.

Odonno · 2017-03-31T10:20:57Z

LIvecoding-uwp/Livecoding/Views/StreamPage.xaml

                            </DataTemplate>
                        </lvcontrols:LivecodingMediaTransportControls.HeaderTemplate>
                    </lvcontrols:LivecodingMediaTransportControls>
+                    <AppBarButton x:Name="ChatButton"


Il me semble que cet AppBarButton doit être intégré au contrôle LivecodingMediaTransportControls. Normalement, le template de ce contrôle est dans App.xaml.cs.

tu peut détaillé?

Odonno · 2017-03-31T10:21:13Z

LIvecoding-uwp/Livecoding/Views/StreamPage.xaml

+                                          Icon="Contact"
+                                          Style="{StaticResource AppBarButtonStyle}"
+                                          MediaTransportControlsHelper.DropoutOrder="5"
+                                          VerticalAlignment="Center"


Il manque la balise de fermeture />

Odonno · 2017-03-31T10:21:51Z

LIvecoding-uwp/Livecoding/Views/StreamPage.xaml

+            </MediaPlayerElement>
+            <ListViewItem x:Name="ChatList" IsEnabled="False" Visibility="Collapsed"/>
+
+            </MediaPlayerElement.TransportControls>


Déjà présent. A supprimer.

Odonno · 2017-03-31T10:22:03Z

LIvecoding-uwp/Livecoding/Views/StreamPage.xaml

+
                </MediaPlayerElement.TransportControls>
+
+            </MediaPlayerElement>


Déjà présent. A supprimer.

Odonno · 2017-03-31T10:22:29Z

LIvecoding-uwp/Livecoding/Views/StreamPage.xaml

                </MediaPlayerElement.TransportControls>
+
+            </MediaPlayerElement>
+            <ListViewItem x:Name="ChatList" IsEnabled="False" Visibility="Collapsed"/>


A déplacer en dehors du contrôle MediaPlayerElement.

Odonno · 2017-03-31T10:24:16Z

LIvecoding-uwp/Livecoding/Views/StreamPage.xaml

                </MediaPlayerElement.TransportControls>
+
+            </MediaPlayerElement>
+            <ListViewItem x:Name="ChatList" IsEnabled="False" Visibility="Collapsed"/>


Je te conseille d'utiliser le contrôle ListView plutôt que ListViewItem Et aussi, pourquoi utiliser la propriété IsEnabled, la propriété Visibility est amplement suffisante.

Odonno · 2017-04-03T11:12:16Z

LIvecoding-uwp/Livecoding/Views/StreamPage.xaml

+            </MediaPlayerElement>
+
        </Grid>
+        <ListView x:Name="ChatList" Visibility="Collapsed"  />


A renommer en ChatListView.

Odonno · 2017-04-03T11:13:17Z

LIvecoding-uwp/Livecoding/Views/StreamPage.xaml

+            </MediaPlayerElement>
+
        </Grid>
+        <ListView x:Name="ChatList" Visibility="Collapsed"  />


Pour mettre à jour le fait qu'elle soit visible : Visibility={Binding Path=ShowChat}

Odonno · 2017-04-03T11:13:57Z

LIvecoding-uwp/Livecoding/Views/StreamPage.xaml

+                                          DataContext="{Binding ShowChat, Source = {StaticRessource Locator}}"
+                                          MediaTransportControlsHelper.DropoutOrder="5"
+                                          VerticalAlignment="Center"
+                                          Onclick="ChatHide_Show">


Inutile de mettre un event OnClick, il faudra passer par une Command.

quelle est la différence?

Si tu passes par des events comme OnClick par exemple, tu vas devoir gérer la logique dans le code-behind, ce qui est à éviter dans 99% des cas.

Si tu passes par une Command, tu vas pouvoir manipuler la logique dans le ViewModel. Tu peux prendre exemple sur ce ViewModel https://github.com/uwp-squad/livecoding-uwp/blob/master/LIvecoding-uwp/Livecoding/ViewModels/LivestreamsViewModel.cs#L50

Le bouton devrait se situer à proximité des autres boutons ici https://github.com/uwp-squad/livecoding-uwp/blob/master/LIvecoding-uwp/Livecoding/App.xaml#L728

Et dans l'immédiat, tu peux faire un binding direct sur la commande de ton ViewModel pour voir ce que ça donne. Je vois pas d'autre solution simple pour le moment.

Odonno · 2017-04-03T11:15:04Z

LIvecoding-uwp/Livecoding/Views/StreamPage.xaml

                            <DataTemplate>
                                <usercontrols:StreamHeaderControl DataContext="{Binding Stream, Source={StaticResource Locator}}" />
+
+                                <controls:AppBarButton x:Name="ChatButton"


Idem que la fois précédente, à placer dans App.xaml.cs

mais c'est du xaml ici, pas du c#

Je me suis trompé, c'est dans App.xaml. Dans le DataTemplate du control.

ok, je vais chercher pour voir ou le mettre :)

Odonno · 2017-04-03T11:15:21Z

LIvecoding-uwp/Livecoding/Views/StreamPage.xaml

+                                <controls:AppBarButton x:Name="ChatButton"
+                                          Icon="Contact"
+                                          Style="{StaticResource AppBarButtonStyle}"
+                                          DataContext="{Binding ShowChat, Source = {StaticRessource Locator}}"


Pas besoin de DataContext.

Odonno · 2017-04-03T11:17:06Z

LIvecoding-uwp/Livecoding/ViewModels/StreamViewModel.cs


        #region Properties
+        private ListView _chatlist;
+        public ListView ChatList


A remplacer par public ObservableCollection<ChatMessage> Messages { get; } = new ObservableCollection<ChatMessage>();

dispositions des element dans appxaml, depuis bcp d'erreur

Odonno · 2017-04-20T22:20:11Z

LIvecoding-uwp/Livecoding/App.xaml

                                            </Storyboard>
                                        </VisualState>
                                    </VisualStateGroup>
+


Suppression de la ligne en trop

Odonno · 2017-04-20T22:20:19Z

LIvecoding-uwp/Livecoding/App.xaml

                                                            </AppBarButton.Icon>
                                                        </AppBarButton>
+                                                        <AppBarButton x:Name="ChatButton"
+


Suppression de la ligne en trop

Odonno · 2017-04-20T22:20:47Z

LIvecoding-uwp/Livecoding/App.xaml

+                                                        <AppBarButton x:Name="ChatButton"
+
+                                                                      MediaTransportControlsHelper.DropoutOrder ="5"
+                                                                      Visibility="Collapsed"


Le bouton est censé être toujours visible, non ? Ligne à supprimer.

Odonno · 2017-04-20T22:23:20Z

LIvecoding-uwp/Livecoding/App.xaml

+
+                                                                      MediaTransportControlsHelper.DropoutOrder ="5"
+                                                                      Visibility="Collapsed"
+                                                                      Command="ChatVisibilityChange">


Voilà comment on écrit une commande généralement :

Command="{Binding ShowChatCommand}"

Dans notre cas, je pense plutôt qu'il faut l'appeler ToggleChatCommand.

Odonno · 2017-04-20T22:24:15Z

LIvecoding-uwp/Livecoding/App.xaml

+                                                                <FontIcon Glyph="comment"/>
+                                                            </AppBarButton.Icon>
+                                                        </AppBarButton>
+


Suppression des lignes en trop

Odonno · 2017-04-20T22:28:48Z

LIvecoding-uwp/Livecoding/Views/StreamPage.xaml

                        <lvcontrols:LivecodingMediaTransportControls.HeaderTemplate>
                            <DataTemplate>
                                <usercontrols:StreamHeaderControl DataContext="{Binding Stream, Source={StaticResource Locator}}" />
+


Suppression des lignes en trop

Odonno · 2017-04-20T22:28:51Z

LIvecoding-uwp/Livecoding/Views/StreamPage.xaml

                            </DataTemplate>
                        </lvcontrols:LivecodingMediaTransportControls.HeaderTemplate>
                    </lvcontrols:LivecodingMediaTransportControls>
+


Suppression des lignes en trop

Odonno · 2017-04-20T22:28:54Z

LIvecoding-uwp/Livecoding/Views/StreamPage.xaml

+
                </MediaPlayerElement.TransportControls>
+
+


Suppression des lignes en trop

Odonno · 2017-04-20T22:28:57Z

LIvecoding-uwp/Livecoding/Views/StreamPage.xaml

            </MediaPlayerElement>
+
+
+


Suppression des lignes en trop

Odonno · 2017-04-20T22:29:56Z

LIvecoding-uwp/Livecoding/App.xaml

+                                                                      Visibility="Collapsed"
+                                                                      Command="ChatVisibilityChange">
+                                                            <AppBarButton.Icon>
+                                                                <FontIcon Glyph="comment"/>


Il me semble qu'il y a une majuscule pour le nom d'une glyph. Donc ce serait Comment plutôt.

Odonno · 2017-04-20T22:32:01Z

Pour ce qui est de la commande, il faudra la créer dans le ViewModel. Regarde des exemples si cela peut t'aider. Comme par ici : https://github.com/uwp-squad/livecoding-uwp/blob/master/LIvecoding-uwp/Livecoding/ViewModels/LivestreamsViewModel.cs#L37

Ajout du bouton

4bb8269

Odonno requested changes Mar 31, 2017

View reviewed changes

André added 2 commits April 3, 2017 12:39

modifications 1

353ff85

modif2

1d0d245

Odonno requested changes Apr 3, 2017

View reviewed changes

Odonno reviewed Apr 3, 2017

View reviewed changes

Modifications4

8b6b530

dispositions des element dans appxaml, depuis bcp d'erreur

Odonno requested changes Apr 20, 2017

View reviewed changes

Odonno force-pushed the master branch from 7249312 to 2109dcf Compare May 18, 2017 21:19


		</MediaPlayerElement.TransportControls>

		</MediaPlayerElement>

Conversation

andre726 commented Mar 30, 2017

Uh oh!

Choose a reason for hiding this comment

Uh oh!

andre726 Apr 2, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Odonno Apr 3, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Odonno Apr 5, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Odonno commented Apr 20, 2017

Uh oh!

Reviewers

andre726 Apr 2, 2017 •

edited

Loading

Odonno Apr 3, 2017 •

edited

Loading

Odonno Apr 5, 2017 •

edited

Loading