diff --git a/Code/Framework/AzToolsFramework/AzToolsFramework/ViewportSelection/EditorTransformComponentSelection.cpp b/Code/Framework/AzToolsFramework/AzToolsFramework/ViewportSelection/EditorTransformComponentSelection.cpp index 433602e6d8..a558e5895d 100644 --- a/Code/Framework/AzToolsFramework/AzToolsFramework/ViewportSelection/EditorTransformComponentSelection.cpp +++ b/Code/Framework/AzToolsFramework/AzToolsFramework/ViewportSelection/EditorTransformComponentSelection.cpp @@ -441,7 +441,7 @@ namespace AzToolsFramework clusterId); } - static void SetViewportUiClusterVisible(ViewportUi::ClusterId clusterId, bool visible) + static void SetViewportUiClusterVisible(const ViewportUi::ClusterId clusterId, const bool visible) { ViewportUi::ViewportUiRequestBus::Event( ViewportUi::DefaultViewportId, @@ -449,7 +449,7 @@ namespace AzToolsFramework clusterId, visible); } - static void SetViewportUiClusterActiveButton(ViewportUi::ClusterId clusterId, ViewportUi::ButtonId buttonId) + static void SetViewportUiClusterActiveButton(const ViewportUi::ClusterId clusterId, const ViewportUi::ButtonId buttonId) { ViewportUi::ViewportUiRequestBus::Event( ViewportUi::DefaultViewportId, @@ -457,7 +457,7 @@ namespace AzToolsFramework clusterId, buttonId); } - static ViewportUi::ButtonId RegisterClusterButton(ViewportUi::ClusterId clusterId, const char* iconName) + static ViewportUi::ButtonId RegisterClusterButton(const ViewportUi::ClusterId clusterId, const char* iconName) { ViewportUi::ButtonId buttonId; ViewportUi::ViewportUiRequestBus::EventResult( diff --git a/Code/Framework/AzToolsFramework/AzToolsFramework/ViewportUi/Button.cpp b/Code/Framework/AzToolsFramework/AzToolsFramework/ViewportUi/Button.cpp index 701344db18..7ef6d96223 100644 --- a/Code/Framework/AzToolsFramework/AzToolsFramework/ViewportUi/Button.cpp +++ b/Code/Framework/AzToolsFramework/AzToolsFramework/ViewportUi/Button.cpp @@ -13,7 +13,6 @@ #include "AzToolsFramework_precompiled.h" #include -#include namespace AzToolsFramework::ViewportUi::Internal { @@ -22,4 +21,11 @@ namespace AzToolsFramework::ViewportUi::Internal , m_buttonId(buttonId) { } + + Button::Button(AZStd::string icon, AZStd::string name, ButtonId buttonId) + : m_icon(AZStd::move(icon)) + , m_name(AZStd::move(name)) + , m_buttonId(buttonId) + { + } } // namespace AzToolsFramework::ViewportUi::Internal diff --git a/Code/Framework/AzToolsFramework/AzToolsFramework/ViewportUi/Button.h b/Code/Framework/AzToolsFramework/AzToolsFramework/ViewportUi/Button.h index 8ff98abc5d..cd4d548510 100644 --- a/Code/Framework/AzToolsFramework/AzToolsFramework/ViewportUi/Button.h +++ b/Code/Framework/AzToolsFramework/AzToolsFramework/ViewportUi/Button.h @@ -12,7 +12,6 @@ #pragma once -#include #include namespace AzToolsFramework::ViewportUi::Internal @@ -27,10 +26,12 @@ namespace AzToolsFramework::ViewportUi::Internal Deselected }; - explicit Button(AZStd::string icon, ButtonId buttonId); + Button(AZStd::string icon, ButtonId buttonId); + Button(AZStd::string icon, AZStd::string name, ButtonId buttonId); ~Button() = default; AZStd::string m_icon; //!< The icon for this button, string path to an image. + AZStd::string m_name; //!< The name displayed as a label next to the button's icon. State m_state = State::Deselected; ButtonId m_buttonId; }; diff --git a/Code/Framework/AzToolsFramework/AzToolsFramework/ViewportUi/Cluster.cpp b/Code/Framework/AzToolsFramework/AzToolsFramework/ViewportUi/ButtonGroup.cpp similarity index 66% rename from Code/Framework/AzToolsFramework/AzToolsFramework/ViewportUi/Cluster.cpp rename to Code/Framework/AzToolsFramework/AzToolsFramework/ViewportUi/ButtonGroup.cpp index 2de6836aa5..3aeb14ed46 100644 --- a/Code/Framework/AzToolsFramework/AzToolsFramework/ViewportUi/Cluster.cpp +++ b/Code/Framework/AzToolsFramework/AzToolsFramework/ViewportUi/ButtonGroup.cpp @@ -11,37 +11,27 @@ */ #include -#include +#include namespace AzToolsFramework::ViewportUi::Internal { - Cluster::Cluster() + ButtonGroup::ButtonGroup() : m_buttons() , m_buttonTriggeredEvent() { } - void Cluster::SetViewportUiElementId(const ViewportUiElementId id) + void ButtonGroup::SetViewportUiElementId(const ViewportUiElementId id) { m_viewportUiId = id; } - ViewportUiElementId Cluster::GetViewportUiElementId() const + ViewportUiElementId ButtonGroup::GetViewportUiElementId() const { return m_viewportUiId; } - void Cluster::SetClusterId(const ClusterId clusterId) - { - m_clusterId = clusterId; - } - - ClusterId Cluster::GetClusterId() const - { - return m_clusterId; - } - - void Cluster::SetHighlightedButton(ButtonId buttonId) + void ButtonGroup::SetHighlightedButton(ButtonId buttonId) { if (auto buttonEntry = m_buttons.find(buttonId); buttonEntry != m_buttons.end()) { @@ -53,15 +43,22 @@ namespace AzToolsFramework::ViewportUi::Internal } } - ButtonId Cluster::AddButton(const AZStd::string& icon) + ButtonId ButtonGroup::AddButton(const AZStd::string& icon, const AZStd::string& name) { auto buttonId = ButtonId(m_buttons.size() + 1); - m_buttons.insert({buttonId, AZStd::make_unique