Skip to content

Conversation

@15100399015
Copy link
Collaborator

What is the purpose of the change

Add a scene or page switcher in the lower right corner of the preview page

Related issues/PRs

Related issues: #43

Brief change log

/solidui-web/src/pages/Designer/Preview/index.tsx
/solidui-web/src/pages/Designer/Preview/PreviewPopup.tsx
/solidui-web/src/pages/Designer/Preview/preview.tsx

Checklist

  • I have read the Contributing Guidelines on pull requests.
  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible
  • If this is a code change: I have written unit tests to fully verify the new behavior.

@dlimeng dlimeng requested a review from nutsjian July 6, 2023 15:45
}

export const FeatureBar: React.FC<IPointerProps> = (props) => {
console.log(props);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Since we will use ESLint to check the code, it's better to remove the console.log here

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ok

});
return nodes;
}
console.log(scenePageOptions);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Since we will use ESLint to check the code, it's better to remove the console.log here

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ok

import { ProjectPageViewsResultData } from "@/apis/types/resp";
import SolidViewFactory from "@/views/SolidViewFactory";
import { isNil } from "lodash-es";
import _, { isNil } from "lodash-es";
Copy link
Collaborator

Choose a reason for hiding this comment

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

It's better to use import { isNil } from "lodash-es", when build the project, it can reduce the bundle size.

Copy link
Collaborator

Choose a reason for hiding this comment

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

use isNil(xxx) instead of _.isNil(xxx)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ok

@15100399015
Copy link
Collaborator Author

I have solved the above problem

@15100399015 15100399015 requested a review from nutsjian July 7, 2023 01:17
@nutsjian nutsjian merged commit 81709bc into CloudOrc:dev Jul 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants