remove todos

Author: mortoray

Samples/UIStructure/Navigation/AppMenu.ux

@@ -1,8 +1,6 @@
 <DockPanel Color="#DDD" ux:Class="AppMenu">
-	<!-- TODO: Remove this, depends on removing that JavaScript part -->
 	<Router ux:Dependency="router"/>
 
-	<!-- TODO: Remove need for this JavaScript, do routing with a UX action -->
 	<JavaScript>
 		exports.gotoOne = function() {
 			router.goto( "home", {}, "flights", {}, "arrivals" )
@@ -27,7 +25,6 @@
 	<Panel ux:Class="AppMenuItem" HitTestMode="LocalBounds">
 		<string ux:Property="Icon"/>
 		<string ux:Property="Label"/>
-		<!-- TODO: Remove use of `object` here, introduce an appropriate type -->
 		<object ux:Property="Handler"/>
 
 		<Text Alignment="CenterLeft" Width="32" Margin="4,0" Value="{Property this.Icon}"

Samples/UIStructure/Navigation/MainView.ux

@@ -4,7 +4,6 @@
 	<Router ux:Name="router"/>
 
 	<ClientPanel>
-		<!-- TODO: Provide dismiss in UX, or way to set a value (`Active`) to `null` -->
 		<JavaScript>
 			exports.dismissMenu = function() {
 				edge.dismiss()

Samples/UIStructure/Navigation/Pages/FlightsPage.ux

@@ -19,8 +19,6 @@
 		</PageIndicator>
 	</TitleAreaContent>
 
-	<!-- TODO: Navigator wasn't working here with the NavigateTo above -->
-	<!-- 	<Navigator ux:Name="flightNav" DefaultPath="arrivals"> -->
 	<PageControl Interaction="None" ux:Name="flightNav">
 		<Page Color="#EEF" Title="Arrivals" ux:Name="arrivals">
 			<Text Alignment="Center" Value="Listing of arrivals"/>