UI/menu/features/alapalkki/ButtonHighlights#1540
Conversation
elggu
commented
May 15, 2026
elggu
commented
- lisätty napin korostus loppuihin nappeihin
- Avatar painikkeen korostus tällä hetkellä vaan perusympyrä, koska pään outline on muuttuva.
-lisätty suurtuminen painaessa muutamaan nappiin -glow spritet vielä puuttuu
-vielä bugi jossa nappien suuruus ei muutu takaisin normaaliksi ellei toista nappia paina, korjaan asap.
- lisätty napin korostus loppuihin UI nappeihin
| @@ -9,6 +11,8 @@ public class DropDownMenu : MonoBehaviour | |||
| private GameObject _dropDownPanel; | |||
| [SerializeField] | |||
| private bool _dropDownOpen = false; | |||
| [SerializeField] | |||
| private Button _hamburgerButton; | |||
There was a problem hiding this comment.
Tähänhän olisi suoraan voinut hakea PopUpButtonVisualin referenssin, kun se on se mitä haetaan muutenkin.
| private Transform _glowSprite; | ||
| private void Awake() | ||
| { | ||
| _glowSprite = transform.Find("Glow"); |
There was a problem hiding this comment.
Itse pistäisin tämän viittauksen editorin kautta fieldinä tälle, jolloin ei tarvitse lähteä etsimään tätä objektia.
Tämä voi toimia kylläkin fallbackinä, mut yleensä pyrin välttämään Findin käyttöä jos mahdollista.
| @@ -42,6 +42,8 @@ private enum OnlinePlayersView | |||
| [SerializeField] private Button _viewClanPlayersButton; | |||
| [SerializeField] private Button _viewAllPlayersButton; | |||
| [SerializeField] private Button _viewFriendListButton; | |||
| [Header("Popup Button")] | |||
| [SerializeField] private Button _onlinePlayersPopupButton; //for visual selection effect | |||
There was a problem hiding this comment.
Vähän aikaisemmasta kopioituna.
Tähänhän olisi suoraan voinut hakea PopUpButtonVisualin referenssin, kun se on se mitä haetaan muutenkin.
| @@ -21,6 +22,7 @@ public class JukeBoxSoulhomeHandler : MonoBehaviour | |||
| [SerializeField] private GameObject _addMusicInfoPopup; | |||
| [SerializeField] private JukeboxInfoPopupHandler _jukeboxInfoPopupHandler; | |||
| [SerializeField] private PopupController _jukeboxTextPopup; | |||
| [SerializeField] private Button _openJukeboxPopupButton; | |||
There was a problem hiding this comment.
Ja sama mikä aikaisemmissa.
There was a problem hiding this comment.
Noiden kommenttien lisäksi huomasin, että näitä korostuksia ei ollut asetettu muihin yläpalkkityyleihin ja nämä tyylit ovat myös rikkinäisiä tämän takia.
Lisäksi osa komponenteista on asetettu 10-MenuUi sceneen suoraan, kun niiden pitäisi olla niiden yläpalkkien prefabien sisällä.
2 participants
