minor #1665 [Icons] Icon changes (kbond)

This PR was squashed before being merged into the 2.x branch.

Discussion
----------

[Icons] `Icon` changes

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| Issues        | n/a
| License       | MIT

1. Removed `Serializable` from `Icon` (this isn't needed unless I'm mistaken)
2. I moved `Icon` up a level (this is a personal preference). `@smnandre` did you have plans for more classes in the `Svg` namespace?

Commits
-------

b8930e4 [Icons] `Icon` changes

Author: kbond

src/Icons/src/Svg/Icon.php renamed to src/Icons/src/Icon.php

@@ -9,14 +9,14 @@
  * file that was distributed with this source code.
  */
 
-namespace Symfony\UX\Icons\Svg;
+namespace Symfony\UX\Icons;
 
 /**
  * @author Simon André <[email protected]>
  *
  * @internal
  */
-final class Icon implements \Stringable, \Serializable
+final class Icon implements \Stringable
 {
     /**
      * Transforms a valid icon ID into an icon name.
@@ -193,16 +193,6 @@ public function __toString(): string
         return $this->toHtml();
     }
 
-    public function serialize(): string
-    {
-        return serialize([$this->innerSvg, $this->attributes]);
-    }
-
-    public function unserialize(string $data): void
-    {
-        [$this->innerSvg, $this->attributes] = unserialize($data);
-    }
-
     public function __serialize(): array
     {
         return [$this->innerSvg, $this->attributes];

src/Icons/src/IconCacheWarmer.php

@@ -13,7 +13,6 @@
 
 use Symfony\UX\Icons\Exception\IconNotFoundException;
 use Symfony\UX\Icons\Registry\CacheIconRegistry;
-use Symfony\UX\Icons\Svg\Icon;
 use Symfony\UX\Icons\Twig\IconFinder;
 
 /**

src/Icons/src/IconRegistryInterface.php

@@ -12,7 +12,6 @@
 namespace Symfony\UX\Icons;
 
 use Symfony\UX\Icons\Exception\IconNotFoundException;
-use Symfony\UX\Icons\Svg\Icon;
 
 /**
  * @author Kevin Bond <[email protected]>

src/Icons/src/Iconify.php

@@ -17,7 +17,6 @@
 use Symfony\Contracts\Cache\CacheInterface;
 use Symfony\Contracts\HttpClient\HttpClientInterface;
 use Symfony\UX\Icons\Exception\IconNotFoundException;
-use Symfony\UX\Icons\Svg\Icon;
 
 /**
  * @author Kevin Bond <[email protected]>

src/Icons/src/Registry/CacheIconRegistry.php

@@ -13,8 +13,8 @@
 
 use Symfony\Contracts\Cache\CacheInterface;
 use Symfony\UX\Icons\Exception\IconNotFoundException;
+use Symfony\UX\Icons\Icon;
 use Symfony\UX\Icons\IconRegistryInterface;
-use Symfony\UX\Icons\Svg\Icon;
 
 /**
  * @author Kevin Bond <[email protected]>

src/Icons/src/Registry/ChainIconRegistry.php

@@ -12,8 +12,8 @@
 namespace Symfony\UX\Icons\Registry;
 
 use Symfony\UX\Icons\Exception\IconNotFoundException;
+use Symfony\UX\Icons\Icon;
 use Symfony\UX\Icons\IconRegistryInterface;
-use Symfony\UX\Icons\Svg\Icon;
 
 /**
  * @author Kevin Bond <[email protected]>

src/Icons/src/Registry/IconifyOnDemandRegistry.php

@@ -12,9 +12,9 @@
 namespace Symfony\UX\Icons\Registry;
 
 use Symfony\UX\Icons\Exception\IconNotFoundException;
+use Symfony\UX\Icons\Icon;
 use Symfony\UX\Icons\Iconify;
 use Symfony\UX\Icons\IconRegistryInterface;
-use Symfony\UX\Icons\Svg\Icon;
 
 /**
  * @author Kevin Bond <[email protected]>

src/Icons/src/Registry/LocalSvgIconRegistry.php

@@ -13,8 +13,8 @@
 
 use Symfony\Component\Filesystem\Filesystem;
 use Symfony\UX\Icons\Exception\IconNotFoundException;
+use Symfony\UX\Icons\Icon;
 use Symfony\UX\Icons\IconRegistryInterface;
-use Symfony\UX\Icons\Svg\Icon;
 
 /**
  * @author Kevin Bond <[email protected]>

src/Icons/tests/Unit/IconRendererTest.php

@@ -13,9 +13,9 @@
 
 use PHPUnit\Framework\TestCase;
 use Symfony\UX\Icons\Exception\IconNotFoundException;
+use Symfony\UX\Icons\Icon;
 use Symfony\UX\Icons\IconRegistryInterface;
 use Symfony\UX\Icons\IconRenderer;
-use Symfony\UX\Icons\Svg\Icon;
 use Symfony\UX\Icons\Tests\Util\InMemoryIconRegistry;
 
 /**

src/Icons/tests/Unit/Svg/IconTest.php renamed to src/Icons/tests/Unit/IconTest.php

@@ -9,10 +9,10 @@
  * file that was distributed with this source code.
  */
 
-namespace Symfony\UX\Icons\Tests\Unit\Svg;
+namespace Symfony\UX\Icons\Tests\Unit;
 
 use PHPUnit\Framework\TestCase;
-use Symfony\UX\Icons\Svg\Icon;
+use Symfony\UX\Icons\Icon;
 
 final class IconTest extends TestCase
 {
@@ -275,4 +275,11 @@ public static function provideWithAttributesTestCases(): iterable
             ['foo' => 'foobar', 'baz' => 'qux'],
         ];
     }
+
+    public function testSerialize(): void
+    {
+        $icon = new Icon('foo', ['bar' => 'baz']);
+
+        $this->assertEquals($icon, unserialize(serialize($icon)));
+    }
 }

src/Icons/tests/Unit/Registry/InMemoryIconRegistryTest.php

@@ -13,7 +13,7 @@
 
 use PHPUnit\Framework\TestCase;
 use Symfony\UX\Icons\Exception\IconNotFoundException;
-use Symfony\UX\Icons\Svg\Icon;
+use Symfony\UX\Icons\Icon;
 use Symfony\UX\Icons\Tests\Util\InMemoryIconRegistry;
 
 /**

src/Icons/tests/Unit/Registry/LocalSvgIconRegistryTest.php

@@ -12,8 +12,8 @@
 namespace Symfony\UX\Icons\Tests\Unit\Registry;
 
 use PHPUnit\Framework\TestCase;
+use Symfony\UX\Icons\Icon;
 use Symfony\UX\Icons\Registry\LocalSvgIconRegistry;
-use Symfony\UX\Icons\Svg\Icon;
 
 /**
  * @author Kevin Bond <[email protected]>

src/Icons/tests/Util/InMemoryIconRegistry.php

@@ -12,8 +12,8 @@
 namespace Symfony\UX\Icons\Tests\Util;
 
 use Symfony\UX\Icons\Exception\IconNotFoundException;
+use Symfony\UX\Icons\Icon;
 use Symfony\UX\Icons\IconRegistryInterface;
-use Symfony\UX\Icons\Svg\Icon;
 
 /**
  * @author Simon André <[email protected]>