mrlan
/
EnglishPal
4
0
Fork 11
Code Issues Pull Requests 22 Projects Releases Wiki Activity

Bug521-LiYuFeng #67

Closed
liyufeng wants to merge 1 commits from Bug521-LiYuFeng into Alpha-snapshot20240618
pull from: Bug521-LiYuFeng
merge into: mrlan:Alpha-snapshot20240618
Conversation 6 Commits 1 Files Changed 2
liyufeng commented 2022-12-10 22:31:59 +08:00
There is no content yet.
liyufeng added 1 commit 2022-12-10 22:31:59 +08:00
liyufeng changed title from Now words order can change dynamically. to Bug521-LiYuFeng 2022-12-10 22:33:00 +08:00
liyufeng commented 2022-12-10 22:33:38 +08:00
Poster

Now words order can change dynamically.

Now words order can change dynamically.
👍 1
liyufeng requested review from mrlan 2022-12-10 22:40:07 +08:00
mrlan commented 2022-12-11 15:48:49 +08:00
Owner

@liyufeng

Thanks

我部署了，可以正确运行。小问题：

  • 生词表居中了，期望不居中。

  • 生词一多（比如500个），重新排序速度明显变慢，可以快一点吗？不需要所有单词都按照词频重新排一遍序，只要把词频增加或者减少的那个单词放到正确的位置即可。

-蓝珲

@liyufeng Thanks 我部署了，可以正确运行。小问题： - 生词表居中了，期望不居中。 - 生词一多（比如500个），重新排序速度明显变慢，可以快一点吗？不需要所有单词都按照词频重新排一遍序，只要把词频增加或者减少的那个单词放到正确的位置即可。 -蓝珲
liyufeng commented 2022-12-12 20:08:21 +08:00
Poster

@liyufeng

Thanks

我部署了，可以正确运行。小问题：

  • 生词表居中了，期望不居中。

  • 生词一多（比如500个），重新排序速度明显变慢，可以快一点吗？不需要所有单词都按照词频重新排一遍序，只要把词频增加或者减少的那个单词放到正确的位置即可。

-蓝珲

蓝老师：

我们改动的内容中不包括生词表居中的功能，可能是您在合并其他分支的时候出现的问题。（具体可以看我的看板中的效果演示视频，是没有生词表居中功能的。）

至于单词数量基数大所导致的排序速度慢的问题，确实需要改进一下，我们暂时还没有想好。

> @liyufeng > > Thanks > > 我部署了，可以正确运行。小问题： > > - 生词表居中了，期望不居中。 > > - 生词一多（比如500个），重新排序速度明显变慢，可以快一点吗？不需要所有单词都按照词频重新排一遍序，只要把词频增加或者减少的那个单词放到正确的位置即可。 > > -蓝珲 蓝老师： 我们改动的内容中不包括生词表居中的功能，可能是您在合并其他分支的时候出现的问题。（具体可以看我的看板中的效果演示视频，是没有生词表居中功能的。） 至于单词数量基数大所导致的排序速度慢的问题，确实需要改进一下，我们暂时还没有想好。
🎉 1
mrlan commented 2022-12-13 10:29:15 +08:00
Owner

@liyufeng

蓝老师，我们改动的内容中不包括生词表居中的功能，可能是您在合并其他分支的时候出现的问题。（具体可以看我的看板中的效果演示视频，是没有生词表居中功能的。）

最近合并的是 Bug505-ZhangYiteng 分支。我部署了该分支，生词表不会居中。

-蓝珲

@liyufeng > 蓝老师，我们改动的内容中不包括生词表居中的功能，可能是您在合并其他分支的时候出现的问题。（具体可以看我的看板中的效果演示视频，是没有生词表居中功能的。） 最近合并的是 [Bug505-ZhangYiteng](http://121.4.94.30:3000/mrlan/EnglishPal/pulls?q=&type=all&sort=&state=closed&labels=&milestone=0&assignee=0) 分支。我部署了该分支，生词表不会居中。 -蓝珲
liyufeng commented 2022-12-15 15:51:42 +08:00
Poster

@liyufeng

蓝老师，我们改动的内容中不包括生词表居中的功能，可能是您在合并其他分支的时候出现的问题。（具体可以看我的看板中的效果演示视频，是没有生词表居中功能的。）

最近合并的是 Bug505-ZhangYiteng 分支。我部署了该分支，生词表不会居中。

-蓝珲

蓝老师，关于居中显示的问题，我们已经找到了原因，稍后会重新进行改进。
原因：我们在生词表展示部分设置的类为 container ,而这个类的style已经被你全局修饰了，所以才会有居中的效果。

> @liyufeng > > > 蓝老师，我们改动的内容中不包括生词表居中的功能，可能是您在合并其他分支的时候出现的问题。（具体可以看我的看板中的效果演示视频，是没有生词表居中功能的。） > > > 最近合并的是 [Bug505-ZhangYiteng](http://121.4.94.30:3000/mrlan/EnglishPal/pulls?q=&type=all&sort=&state=closed&labels=&milestone=0&assignee=0) 分支。我部署了该分支，生词表不会居中。 > > > > -蓝珲 > > 蓝老师，关于居中显示的问题，我们已经找到了原因，稍后会重新进行改进。 原因：我们在生词表展示部分设置的类为 container ,而这个类的style已经被你全局修饰了，所以才会有居中的效果。
liyufeng commented 2022-12-15 15:55:45 +08:00
Poster

@liyufeng

蓝老师，我们改动的内容中不包括生词表居中的功能，可能是您在合并其他分支的时候出现的问题。（具体可以看我的看板中的效果演示视频，是没有生词表居中功能的。）

最近合并的是 Bug505-ZhangYiteng 分支。我部署了该分支，生词表不会居中。

-蓝珲

蓝老师，关于居中显示的问题，我们已经找到了原因，稍后会重新进行改进。
原因：我们在生词表展示部分设置的类为 container ,而这个类的style已经被你全局修饰了，所以才会有居中的效果。

只需要把那个类的名称改成别的即可。例如：改为 class = 'words_container'
还要同步地将js中的对应 'container' -> 'words_container'

> > @liyufeng > > > > > 蓝老师，我们改动的内容中不包括生词表居中的功能，可能是您在合并其他分支的时候出现的问题。（具体可以看我的看板中的效果演示视频，是没有生词表居中功能的。） > > > > > > 最近合并的是 [Bug505-ZhangYiteng](http://121.4.94.30:3000/mrlan/EnglishPal/pulls?q=&type=all&sort=&state=closed&labels=&milestone=0&assignee=0) 分支。我部署了该分支，生词表不会居中。 > > > > > > > > -蓝珲 > > > > > > 蓝老师，关于居中显示的问题，我们已经找到了原因，稍后会重新进行改进。 > 原因：我们在生词表展示部分设置的类为 container ,而这个类的style已经被你全局修饰了，所以才会有居中的效果。 只需要把那个类的名称改成别的即可。例如：改为 class = 'words_container' 还要同步地将js中的对应 'container' -> 'words_container'
mrlan changed title from Bug521-LiYuFeng to Bug521-LiYuFeng 2024-08-30 08:00:16 +08:00
mrlan changed target branch from master to Alpha-snapshot20240618 2024-08-30 08:00:16 +08:00
mrlan closed this pull request 2024-08-30 08:01:41 +08:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
mrlan
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
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: mrlan/EnglishPal#67
There is no content yet.
Delete Branch "Bug521-LiYuFeng"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?