Inline the navigation destination class

Turns out we really didn't need this. It does add one test for the new side effect of constructor now being less magic (desirable as it's the scope of the create method)

packages/framework/src/Framework/Features/Navigation/NavigationItem.php

@@ -23,7 +23,7 @@
  */
 class NavigationItem implements NavigationElement, Stringable
 {
-    protected NavigationDestination $destination;
+    protected string|Route $destination;
     protected string $label;
     protected int $priority;
 
@@ -40,7 +40,7 @@ class NavigationItem implements NavigationElement, Stringable
      */
     public function __construct(Route|string $destination, string $label, int $priority = NavigationMenu::DEFAULT, ?string $group = null)
     {
-        $this->destination = new NavigationDestination($destination);
+        $this->destination = $destination;
 
         $this->label = $label;
         $this->priority = $priority;
@@ -87,15 +87,15 @@ public function __toString(): string
      */
     public function getPage(): ?HydePage
     {
-        return $this->destination->getRoute()?->getPage();
+        return $this->destination instanceof Route ? $this->destination->getPage() : null;
     }
 
     /**
      * Resolve the destination link of the navigation item.
      */
     public function getLink(): string
     {
-        return $this->destination->getLink();
+        return (string) $this->destination;
     }
 
     /**

packages/framework/tests/Unit/NavigationItemTest.php

@@ -58,12 +58,11 @@ public function testPassingRouteInstanceToConstructorUsesRouteInstance()
         $this->assertEquals($route, (new NavigationItem($route, 'Home'))->getPage()->getRoute());
     }
 
-    public function testPassingRouteKeyToConstructorUsesRouteInstance()
+    public function testPassingRouteKeyToConstructorUsesDestinationAsLink()
     {
-        $route = Routes::get('index');
         $item = new NavigationItem('index', 'Home');
-        $this->assertNotNull($item->getPage());
-        $this->assertSame($route->getPage(), $item->getPage());
+        $this->assertNull($item->getPage());
+        $this->assertSame('index', $item->getLink());
     }
 
     public function testPassingUrlToConstructorSetsRouteToNull()
@@ -209,6 +208,14 @@ public function testCreateWithCustomPriority()
         $this->assertSame(100, NavigationItem::create(Routes::get('index'), 'foo', 100)->getPriority());
     }
 
+    public function testPassingRouteKeyToStaticConstructorUsesRouteInstance()
+    {
+        $route = Routes::get('index');
+        $item = NavigationItem::create('index', 'Home');
+        $this->assertNotNull($item->getPage());
+        $this->assertSame($route->getPage(), $item->getPage());
+    }
+
     public function testRouteBasedNavigationItemDestinationsAreResolvedRelatively()
     {
         Render::swap(Mockery::mock(RenderData::class, [