Skip to content

Commit

Permalink
TopMenu: Move open/close button inside header of menu
Browse files Browse the repository at this point in the history
- Display double chevron icon on button (reference #40)
  • Loading branch information
piotrzarzycki21 committed Nov 9, 2023
1 parent 608abc1 commit a8aa2ac
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -253,13 +253,12 @@
if (this.menuDrawer.isOpen)
{
this.menuDrawer.close();
this.menuCloseIcon.text = MaterialIconType.CHEVRON_RIGHT;
}
else
{
this.menuDrawer.open();
this.menuCloseIcon.text = MaterialIconType.CHEVRON_LEFT;
}
this.menuOpen.visible = !this.menuDrawer.isOpen;
}
private function refreshMenu(changeSource:String):Object
Expand Down Expand Up @@ -413,6 +412,16 @@
</j:IconButton>
<html:P localId="menuTitle" text="Menu"/>
</j:HGroup>
<j:HGroup percentWidth="100" itemsHorizontalAlign="itemsRight">
<j:style>
<js:SimpleCSSStyles padding="1"/>
</j:style>
<j:IconButton click="onCloseMenu(event)" className="topMenuCloseButton">
<j:icon>
<js:MaterialIcon className="mdi mdi-chevron-double-left"/>
</j:icon>
</j:IconButton>
</j:HGroup>
</j:DrawerHeader>
<j:HGroup>
<j:beads>
Expand All @@ -436,10 +445,10 @@
</j:HGroup>
</j:Drawer>
</j:Group>
<j:Group>
<j:IconButton click="onCloseMenu(event)" className="topMenuCloseButton">
<j:Group localId="menuOpen" visible="false" height="50">
<j:IconButton click="onCloseMenu(event)" className="topMenuCloseButton" percentHeight="100">
<j:icon>
<js:MaterialIcon localId="menuCloseIcon" text="{MaterialIconType.CHEVRON_LEFT}"/>
<js:MaterialIcon className="mdi mdi-chevron-double-right"/>
</j:icon>
</j:IconButton>
</j:Group>
Expand Down
2 changes: 1 addition & 1 deletion Super.Human.Portal_Royale/src/resources/app-styles.css
Original file line number Diff line number Diff line change
Expand Up @@ -531,7 +531,7 @@ a:hover {
.jewel.button.topMenuCloseButton {
font-size: 24px;
padding: 0;
min-height: 90px;
height: 100%;
}

.jewel.button.icon.topMenuListItemRendererIcon {
Expand Down

0 comments on commit a8aa2ac

Please sign in to comment.