diff options
author | Kim <1877318+kimsible@users.noreply.github.com> | 2020-07-23 15:09:15 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-07-23 15:09:15 +0200 |
commit | ed5bb517266c80904c44bf44a335f1003f5aa277 (patch) | |
tree | 80671a21fb8c13ab47a65f01b64222a3a1e96a47 /client/src/app/shared/shared-main/misc/top-menu-dropdown.component.ts | |
parent | 345b4a22a8432cfd8c9c9f24634821d58c215b9a (diff) | |
download | PeerTube-ed5bb517266c80904c44bf44a335f1003f5aa277.tar.gz PeerTube-ed5bb517266c80904c44bf44a335f1003f5aa277.tar.zst PeerTube-ed5bb517266c80904c44bf44a335f1003f5aa277.zip |
Improve navigation sub-menu and tabs effects (#2971)
* Improve nav border and colors on active and non-active
* Remove margin-top effect on active nav
* Use opacity / bold instead of color change on nav
* Remove dropdown items label in sub-menu and add a class active
* Position sub-menu to fixed
* Autoclose dropdown sub-menu only on outside click
* Remove open dropdown on hover in sub-menu
* Show reusable h1 for dropdown item in sub-menu
* Put reusable sub-menu h1 styles to mixins
* Add icons to sub-menu dropdown-item h1
* Make all the sub-menu accessible with focus
Co-authored-by: kimsible <kimsible@users.noreply.github.com>
Diffstat (limited to 'client/src/app/shared/shared-main/misc/top-menu-dropdown.component.ts')
-rw-r--r-- | client/src/app/shared/shared-main/misc/top-menu-dropdown.component.ts | 23 |
1 files changed, 0 insertions, 23 deletions
diff --git a/client/src/app/shared/shared-main/misc/top-menu-dropdown.component.ts b/client/src/app/shared/shared-main/misc/top-menu-dropdown.component.ts index 5909db0b5..66f8f7e55 100644 --- a/client/src/app/shared/shared-main/misc/top-menu-dropdown.component.ts +++ b/client/src/app/shared/shared-main/misc/top-menu-dropdown.component.ts | |||
@@ -33,7 +33,6 @@ export class TopMenuDropdownComponent implements OnInit, OnDestroy { | |||
33 | isModalOpened = false | 33 | isModalOpened = false |
34 | currentMenuEntryIndex: number | 34 | currentMenuEntryIndex: number |
35 | 35 | ||
36 | private openedOnHover = false | ||
37 | private routeSub: Subscription | 36 | private routeSub: Subscription |
38 | 37 | ||
39 | constructor ( | 38 | constructor ( |
@@ -68,32 +67,10 @@ export class TopMenuDropdownComponent implements OnInit, OnDestroy { | |||
68 | if (this.routeSub) this.routeSub.unsubscribe() | 67 | if (this.routeSub) this.routeSub.unsubscribe() |
69 | } | 68 | } |
70 | 69 | ||
71 | openDropdownOnHover (dropdown: NgbDropdown) { | ||
72 | this.openedOnHover = true | ||
73 | dropdown.open() | ||
74 | |||
75 | // Menu was closed | ||
76 | dropdown.openChange | ||
77 | .pipe(take(1)) | ||
78 | .subscribe(() => this.openedOnHover = false) | ||
79 | } | ||
80 | |||
81 | dropdownAnchorClicked (dropdown: NgbDropdown) { | 70 | dropdownAnchorClicked (dropdown: NgbDropdown) { |
82 | if (this.openedOnHover) { | ||
83 | this.openedOnHover = false | ||
84 | return | ||
85 | } | ||
86 | |||
87 | return dropdown.toggle() | 71 | return dropdown.toggle() |
88 | } | 72 | } |
89 | 73 | ||
90 | closeDropdownIfHovered (dropdown: NgbDropdown) { | ||
91 | if (this.openedOnHover === false) return | ||
92 | |||
93 | dropdown.close() | ||
94 | this.openedOnHover = false | ||
95 | } | ||
96 | |||
97 | openModal (index: number) { | 74 | openModal (index: number) { |
98 | this.currentMenuEntryIndex = index | 75 | this.currentMenuEntryIndex = index |
99 | this.isModalOpened = true | 76 | this.isModalOpened = true |