修改底部版权区域，使用 Wrap 组件来实现文字间的间隔，把结束年份动态获取当前年份 #13

houkunlin · 2023-05-16T08:26:42Z

houkunlin commented

2023-05-16 08:26:42 +00:00

First-time contributor

修改底部版权区域，使用 Wrap 组件来实现文字间的间隔，把结束年份动态获取当前年份

houkunlin added 1 commit 2023-05-16 08:26:43 +00:00

feat: 修改底部版权区域，使用 Wrap 组件来实现文字间的间隔，把结束年份动态获取当前年份 29bce26acb

lsr reviewed 2023-05-16 09:04:15 +00:00

src/Footer.tsx

						
				@ -3,3 +3,4 @@

				import { SDKVersion } from './SDKVersion';

				export function Footer() {

				  const year = useMemo(() => new Date().getFullYear(), []);

lsr commented

2023-05-16 09:04:15 +00:00

没有必要过度缓存，这句话跑得挺快的。缓存本身也有开销。

或考虑作为常量放到组件外，这样就不需要缓存了（反正渲染后不会变）

参考文章：
https://jancat.github.io/post/2019/translation-usememo-and-usecallback/

没有必要过度缓存，这句话跑得挺快的。缓存本身也有开销。或考虑作为常量放到组件外，这样就不需要缓存了（反正渲染后不会变）参考文章： https://jancat.github.io/post/2019/translation-usememo-and-usecallback/

lsr marked this conversation as resolved

lsr reviewed 2023-05-16 21:22:13 +00:00

src/Footer.tsx

						
				@ -35,3 +28,1 @@

				          </Link>

				          。

				        </Text>

				        <Wrap>

lsr commented

2023-05-16 21:22:13 +00:00

这个文字改 Wrap 组件控制空格的更改感觉没必要？

反而变得复杂了…？

这个文字改 Wrap 组件控制空格的更改感觉没必要？反而变得复杂了…？

lsr marked this conversation as resolved

lsr changed title from ~~WIP: 修改底部版权区域，使用 Wrap 组件来实现文字间的间隔，把结束年份动态获取当前年份~~ to 修改底部版权区域，使用 Wrap 组件来实现文字间的间隔，把结束年份动态获取当前年份

2023-05-21 13:26:55 +00:00

lsr commented

2023-05-21 13:27:18 +00:00

挑选了部分更改并已合并到主分支。

lsr closed this pull request

2023-05-21 13:30:01 +00:00

houkunlin deleted branch feat/footer

2023-05-22 10:52:08 +00:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No project

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: um/um-react#13