- Issue created by @pdureau
- 🇫🇷France pdureau Paris
Jean: "Packs are the icon files, as provided by upstream. Sets are the UI artefact we are defining and manipulating".
- Assigned to mogtofu33
- 🇫🇷France pdureau Paris
I am nitpicking, but while we renaming "icon set" to "icon pack" in the filenames, is it possible to also remove the "UI" prefix?
- Drupal\ui_icons\Plugin\UiIconsetManagerInterface >> Drupal\ui_icons\Plugin\IconpackManagerInterface
- Drupal\ui_icons\Plugin\UiIconsetManager >> Drupal\ui_icons\Plugin\IconpackManager
- Drupal\ui_icons\PluginForm\UiIconsetExtractorForm >> Drupal\ui_icons\PluginForm\IconpackExtractorForm
Because we are already in the
ui_icons
namespace, no need to repeat it in filename, and it seems Drupal Core doesn't do such a thing.(I will create another ticket (alpha3? later?) after this one to remove "UiIcon" in other filenames.)
- First commit to issue fork.
- Status changed to Needs work
3 months ago 11:17am 13 August 2024 - 🇫🇷France mogtofu33
What about advance on this task for a global renaming and usage of Icon instead of UiIcons.
- Merge request !4Issue #3466907 by mogtofu33, pdureau: [1.0.0-alpha2] Rename Icon packs and UIIcon → (Merged) created by mogtofu33
-
mogtofu33 →
committed fb312cc9 on 1.0.x
Issue #3466907 by mogtofu33, pdureau: [1.0.0-alpha2] Rename Icon packs...
-
mogtofu33 →
committed fb312cc9 on 1.0.x
- Status changed to Fixed
3 months ago 1:17pm 13 August 2024 - 🇫🇷France mogtofu33
Fast merging as there is way too much changes to isolate testing.
- Issue was unassigned.
- Status changed to Fixed
3 months ago 9:03am 26 August 2024