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.

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

搜索页样式调整 #909

Merged
merged 4 commits into from
Nov 11, 2024
Merged

搜索页样式调整 #909

merged 4 commits into from
Nov 11, 2024

Conversation

ywmoyue
Copy link
Owner

@ywmoyue ywmoyue commented Nov 7, 2024

Cherry pick from #898

感谢 @ProJend

  • Cherrypick
  • 响应式处理

@ywmoyue ywmoyue self-assigned this Nov 7, 2024
@ywmoyue ywmoyue added this to the 待发布-4.7.7 milestone Nov 7, 2024
@ProJend
Copy link

ProJend commented Nov 7, 2024

这个不能单独摘出来,会破坏缓存策略,试用十月更新最新的commit

@ProJend
Copy link

ProJend commented Nov 7, 2024

我一刀切缓存跟个commit有很大关系

@ywmoyue
Copy link
Owner Author

ywmoyue commented Nov 7, 2024

这个不能单独摘出来,会破坏缓存策略,试用十月更新最新的commit

CompositionTarget.Rendered这个事件有问题吗,我看不出其他地方有什么问题

@ywmoyue
Copy link
Owner Author

ywmoyue commented Nov 7, 2024

CompositionTarget.Rendered这个事件有问题吗,我看不出其他地方有什么问题

似乎也没问题,这个事件看你写的逻辑也只会触发一次,还是说关闭页面会导致一些对象不释放?

@ywmoyue
Copy link
Owner Author

ywmoyue commented Nov 7, 2024

测了几遍没发现问题,查看内存也没有看到哪个搜索页中对象没释放的,我用一段时间看看

@ProJend
Copy link

ProJend commented Nov 7, 2024

测了几遍没发现问题,查看内存也没有看到哪个搜索页中对象没释放的,我用一段时间看看

和内存无关,这个commit是半成品,最新的commit才是优化好的搜索页

@ProJend
Copy link

ProJend commented Nov 7, 2024

现在来改就有点重复了

@ywmoyue
Copy link
Owner Author

ywmoyue commented Nov 7, 2024

@ProJend 我构建并运行了 #898 的最新代码,没有发现搜索页的样式较这个PR有什么特别的变化,功能上 6cd40cd 这个提交的搜索页不再固定加载默认视频页面的功能没有了

我暂时只发现这一段 #898 的最新提交代码与这个PR的搜索页的区别,明天我再详细比较两个提交的搜索页代码。

image

另外,你没有说明这个提交为什么会破坏缓存策略

@ProJend
Copy link

ProJend commented Nov 7, 2024

没什么,我好几天没动过代码了,可能搞混了,d801c26才是破坏的提交

@ProJend
Copy link

ProJend commented Nov 7, 2024

奇怪,我测试的时候明明是NavigationCacheMode = NavigationCacheMode.Required,怎么上传就变成NavigationCacheMode = NavigationCacheMode.Disabled

@ProJend
Copy link

ProJend commented Nov 7, 2024

@ywmoyue 这是一个综合性的问题
我第一次修改搜索页d801c26的时候发现NavigatingCacheMode不起作用,索性全删了NavigatingCacheMode
第二次修改搜索页6cd40cd的时候注意到了是页面导航OnNavigatedTo的问题,每次返回加载页面都会首选视频内容,我修正了这个问题,但是忘了把NavigatingCacheMode加回去。
第三次修改搜索页1ad08ba的时候发现有了缓存后的搜索页有几率乱跑内容,最终我在初始化SearchVM时改回了首选视频内容。

总的来说我修改了搜索页里返回上一页的逻辑,让它不再首选视频内容,并加上了缓存,其他的没有变化

@ProJend
Copy link

ProJend commented Nov 7, 2024

很失败的试水,还是了解代码未够啊,先优化点UI好了

@ProJend
Copy link

ProJend commented Nov 8, 2024

需要继续更新的话,这个才是搜索页的正确样式

SearchVM searchVM;

public SearchPage()
{
    NavigationCacheMode = NavigationCacheMode.Required;
    searchVM = new SearchVM(_pivotIndex = 0, _comboIndex = 0);
    this.InitializeComponent();
}

@ywmoyue
Copy link
Owner Author

ywmoyue commented Nov 8, 2024

需要继续更新的话,这个才是搜索页的正确样式

SearchVM searchVM;

public SearchPage()
{
    NavigationCacheMode = NavigationCacheMode.Required;
    searchVM = new SearchVM(_pivotIndex = 0, _comboIndex = 0);
    this.InitializeComponent();
}

好的,非常感谢

@ywmoyue ywmoyue merged commit 3d39917 into dev Nov 11, 2024
@ywmoyue ywmoyue mentioned this pull request Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants