Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feature: menu group - default expand option #1389

Merged
merged 16 commits into from
Oct 27, 2022
Merged

feature: menu group - default expand option #1389

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
5a3f70e
feature(menuGroup): add prop for menuGropu that allow component to be…
Mar 28, 2021
8146485
feat(menuGroup): add expanded menuGroup example to showcase
Mar 28, 2021
a8dc6b3
fix(menuGroup): resolve comments
Mar 29, 2021
41f21c4
fix: typo fix
Mar 30, 2021
8cde36f
remove: remove dead code
Mar 30, 2021
46334a3
update: change initialExpand logic
Mar 31, 2021
1d0e67c
fix: typescript fix
Mar 31, 2021
41756c1
fix: fire event only when component was mounted
Apr 1, 2021
c89675f
Merge branch 'master' into feature/menuGroup/defaultExpand
malashkevich Apr 12, 2021
76f83b2
Merge branch 'master' into feature/menuGroup/defaultExpand
malashkevich Apr 20, 2021
168eb30
Merge branch 'master' into feature/menuGroup/defaultExpand
whitestranger7 Jun 1, 2021
45bdfc3
Merge branch 'master' into feature/menuGroup/defaultExpand
whitestranger7 Jun 15, 2021
4b013ee
Merge branch 'master' into feature/menuGroup/defaultExpand
malashkevich Sep 23, 2021
3d4af14
feat: adjusted initially expanded behaviour implementation
greenfrvr Oct 21, 2022
df31c87
chore: removed redundant code
greenfrvr Oct 21, 2022
1749f91
chore: code cleanup
greenfrvr Oct 21, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 11 additions & 1 deletion src/components/ui/menu/menuGroup.component.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,13 @@ const POSITION_OUTSCREEN: Point = Point.outscreen();
*/
export class MenuGroup extends React.Component<MenuGroupProps, State> {

private initialExpanded: boolean;

constructor(props) {
super(props);
this.initialExpanded = this.props.initialExpanded || false;
}

public state: State = {
submenuHeight: 1,
};
Expand Down Expand Up @@ -114,9 +121,10 @@ export class MenuGroup extends React.Component<MenuGroupProps, State> {

private onSubmenuMeasure = (frame: Frame): void => {
this.setState({ submenuHeight: frame.size.height });
whitestranger7 marked this conversation as resolved.
Show resolved Hide resolved
if (this.props.initialExpanded) {
if (this.initialExpanded) {
greenfrvr marked this conversation as resolved.
Show resolved Hide resolved
const expandValue: number = this.expandAnimationValue > 0 ? 0 : this.state.submenuHeight;
this.createExpandAnimation(expandValue, 0).start();
this.initialExpanded = false
}
};

Expand Down Expand Up @@ -181,6 +189,8 @@ export class MenuGroup extends React.Component<MenuGroupProps, State> {
public render(): React.ReactNode {
const { children, ...itemProps } = this.props;

console.log(this.initialExpanded);

return (
<React.Fragment>
<MenuItem
Expand Down