Skip to content

UI/hamburger menu popup/refactor/new popup design#1646

Open
Emilierald wants to merge 3 commits into
mainfrom
UI/HamburgerMenuPopup/Refactor/NewPopupDesign
Open

UI/hamburger menu popup/refactor/new popup design#1646
Emilierald wants to merge 3 commits into
mainfrom
UI/HamburgerMenuPopup/Refactor/NewPopupDesign

Conversation

@Emilierald

Copy link
Copy Markdown
Contributor

Added the tutorial button to the hamburger menu. Updated the look of the story pop-up.

renewed the look of the story pop-up
Added the tutorial button to the hamburger menu.

@BillTheBeast BillTheBeast left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Muutaman korjauksen tekisin tähän:

  1. Eikös tuo DropdownButton voisi käyttää tuota DropDownPanelia tuon sisään rakennetun rakenteen sijaan, tämä mahdollistaa sen helpomman muokkaamisen myöhemmin kun ei tarvitse itse napin sijainnista välittää niin paljoa. (Ja se Paneli prefabi on olemassa muutenkin.)

  2. DropdownButtonin hohto objekti on väärässä paikkaa, korjaappas se.

  3. Tuo Tarina popup ei pitäisi varmaankaan olla osana itse yläpalkkia. Varsiinkaan kun se aiheuttaa ongelmia tuon sijainti arvojen ja koon suhteen. Siirrä se UIOverlayn FullScreenPopups objektin alle niin toi sijoittelu etenkin sen tausta painikkeen suhteen korjaantuu. (Sen sisällä pitäisi olla ne Popupit, jotka syystä tai toisesta tarvitsevat koko ikkunan koon.)

Lisäksi voisit periaatteessa myös käydä kopioimassa OnlinePlayerPanel skriptistä sen ikkunan piilotuskoodin (LateUpdaten ja CheckIfPanel metodin sisällä) ja pistää sen tähän käyttöön, tällöin se popupin ulkopuolelle painallus myös osuu siihen mihin olet painamassa. Eikä blokkaa turhaan painalluksia (etenkin ylä- ja alapalkin kohdalla relevantti.)

Dropdown menu is using the DropDownMenuPanel prefab, the glow is placed correctly and the menu is longer. Story popup moved under FullPagePopups, made the way the popup registers inputs and closes better.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Tarkistuksessa

Development

Successfully merging this pull request may close these issues.

2 participants