从媒体网格中删除无限滚动行为

[阿费西娅]

将此从#40330.

此问题旨在讨论从媒体网格中删除无限滚动的技术实现细节。有关无限滚动行为的一般讨论，请转到主问题#40330.

[阿费西娅]

50105.diff（如果有）​是用于初始测试的概念验证。

仍需完成：

• 添加/删除附件时更新附件总数
• 更多测试

如果能在不同的安装上运行扩展测试，并在媒体库中存储不同数量的附件，那就太棒了。我建议重点关注：

• 一般行为
• 是否应增加40个附件的默认“每页”
• 按类型和日期筛选，在筛选时要记住存在bug请参见示例#50025.

测试建议：

• 按钮/微调器等的样式是临时的，请暂时忽略它
• 通过浏览器的开发工具调节网络：这有助于观察UI行为，例如微调器可见性
• 测试是否启用了媒体垃圾桶（通过设置define（'MEDIA_TRASH'，true）；在您的wp-config.php文件）
• 测试过滤器add_filter（'media_library_infinite_srolling'，'__return_true'）；恢复无限滚动行为
• 测试媒体库
• 使用Classic Editor测试媒体模式（添加媒体，特色图像）
• 用古腾堡测试媒体模式（图片、画廊、特色图片…）
• 可选：使用插件测试Media模式，插件实现自定义帧以上传/选择附件
• 检查查询缓存集合数组

无障碍：
现在，单击“加载更多”将重点放在第一个添加的附件上。我认为最好探索其他选择，因为最好避免意外转移焦点。

问题：

1
该修补程序更改用于检索附件的WP_Query返回的数据。以前，响应只包含附件数组。现在，它有两个项目：

• 附件数组
• 总附件数，即：发现_支柱

稍后，对响应进行分析，以便将附件数组传递给视图，并为总附件数设置一个新属性。基本上，响应和缓存查询使用不同的格式。在这方面一个有趣的问题是：当媒体库定义滚动过滤器还原无限滚动？媒体网格无论如何都能工作，但我想应该探讨一下潜在的向后兼容性问题？

2
对于任何熟悉媒体浏览历史的人来说，一个更有趣的问题是，各种“compat”方法意味着什么？这部分对我来说有点晦涩难懂。比如：

• 附件.保存Compat
• AJAX调用保存附件-压缩和相关wp_ajax保存附件compat
• 这个附件_字段_编辑事物（I认为这用于旧的上传方法？）

[谢尔盖·比尤科夫]

相关：#50143

[阿费西娅]

相关：#50410这是此票据的拦截器，至少对于与媒体项计数/总数相关的文本部分。

[阿费西娅]

50105.2.差异​刷新修补程序并：

• 添加“跳转到第一个新项目”按钮
• 按钮只对键盘用户显示在焦点上
• 使用更多每磅18英寸

[audrasjb公司]

在我看来真的很好！
正在添加需求-开发说明和需要测试关键字。

[阿费西娅]

请参阅上面的屏幕截图。要澄清行为：

• 只有当有更多项目要加载时，“加载更多”按钮才可见
• 只有在提取项目时，微调器才可见
• 添加新项目后，“跳转到第一个新项目”按钮仅对键盘用户可见

待办事项[已编辑]：

1. 避免在“加载更多”按钮消失时失去焦点，因为没有要加载的新项目
2. 媒体模式中的滚动问题
3. 添加/删除附件时更新总附件数：另一个选项可能是在添加/删除了附件后隐藏计数
4. 考虑是否应该增加40个附件的默认“每页”
5. 如何处理此修补程序更明显的其他长期存在的问题，例如：筛选附件时，请参阅#50025（过滤器不显示新上传）

[阿费西娅]

以下是阻止此提议更改的现有票据列表：
#50410,#50025,#46127、和#47215.

注意：这些都是Media视图中存在的错误行为，它们将永远存在。

[戴维斯鲍姆瓦尔德]

@afercia家属的门票仍然开放。目前，这仍然是5.5中的可行选项吗？

[阿费西娅]

@戴维鲍姆瓦尔德我认为这张票拍不到5.5英镑。我还没有下注，因为我想让组件维护人员知道这张票。

[德斯罗什]

由于这已经在媒体上进行了深入讨论，可以肯定地说，媒体团队已经意识到这一点，只是还没有时间回过头来。到目前为止，由于缺乏运动，我们要去踢平底船。

[阿费西娅]

在今天的易访问性bug-scrub中讨论了这个问题：考虑到Beta 1的周期很快就要到来，并且考虑到这个问题需要更大的可见性和大量的时间进行深入测试，团队同意将其投入到未来的版本中。

[蚂蚁党]

如前所述：https://core.trac.wordpress.org/ticket/50025#评论：13

#50025是一个障碍，所以我们应该在这张票之前优先考虑它，但两者似乎都在5.7的范围内。移动到里程碑以保持可见性。：）

[乔多尔森]

50105.2.diff的更新版本

[乔多尔森]

修补程序已刷新。这与Andrea正在进行的最后阶段没有变化；我认为有几个方面可以改进https://core.trac.wordpress.org/ticket/50105#评论：17还有待完成。

注意：“跳转到第一个新项目”仅在向前移动时按tab键顺序，而在从媒体项目侧边栏向后移动时不按tab顺序。然而，它只有在加载了新项之后才有意义，因此，它在使用后是否应该继续存在是一个合理的问题。

[hellofrom托尼亚]

在准备测试擦洗计划时，请为测试人员提供更多信息：

• 重现问题的步骤是什么？
• 是否有任何测试依赖项，例如插件或脚本？
• 应用修补程序后的预期行为是什么？

[ricjcs公司]

实现该功能后，最好能够打开或关闭它。
有些用户更喜欢无限滚动。

[亚当西尔弗斯坦]

50105.5.差异​针对主干刷新。

Trac票：https://core.trac.wordpress.org/ticket/50105

[亚当西尔弗斯坦]

@ricjcs公司

实现该功能后，最好能够打开或关闭它。
有些用户更喜欢无限滚动。

这可以通过使用过滤器的补丁实现(媒体库定义滚动); 可以使用一行代码或一个迷你插件重新启用此功能。

​https://user-images.githubusercontent.com/2676022/107101313-24058000-67d4-11eb-9685-d23965e3563a.mp4

[亚当西尔弗斯坦]

这很接近；不过，票子还需要一些工作才能完全准备好，请看https://core.trac.wordpress.org/ticket/50105#评论：37

由于我们的测试版本是5.7，所以我们需要尽快实现这一点，或者将测试版本设置为5.8。

[亚当西尔弗斯坦]

避免在“加载更多”按钮消失时失去焦点，因为没有要加载的新项目

想要的行为是什么？焦点应该设置为最后一个媒体项目还是“跳转到第一个新项目”按钮？

媒体模式中的滚动问题

不确定这些是什么，还有更多细节吗？

添加/删除附件时更新总附件数：另一个选项可能是在添加/删除了附件后隐藏计数

理想情况下，这些计数应与收集总数保持同步。

考虑是否应该增加默认的40个附件的“每页”

40岁现在感觉还可以。我们可以确保这是可过滤的。如果这是用户可以控制的，比如每页的帖子，或者每页的术语来自屏幕选项选项卡，那就好了。我们需要将其集成到模式本身中，因为它在许多上下文中运行。

[乔多尔森]

更新的修补程序。为上一个集合、设计更新添加焦点处理。

[乔多尔森]

更新的修补程序：

1） 将按钮样式从英雄更改为正常大小
2） 在单行上放置按钮
3） 默认情况下，使“跳转到第一个新项目”可见。

这修复了一个滚动问题，即导航到“跳转”链接可能会导致视图位置发生不必要的更改，并使有视觉的鼠标用户可以使用“跳转“链接。由于第一个添加的项目并不明显，这对所有用户都很有用。

4） 在第一个新添加的项目上进行视觉高亮显示，以便更容易看到新内容。

期望的行为是什么？

在我看来，把注意力转移到“跳到第一个新项目”；在我看来，这更容易预测。

5） 当加载更多按钮在最终屏幕中不存在时，将焦点移动到“跳转到第一个新项目”按钮。

6） 乐观地说，自参考5.7.0以来进行了@更新

更新总附件数。

这需要对获取TotalAttachments进行某种调整。添加或删除项目时，模型似乎不会刷新，但集合会刷新。

40岁现在感觉还可以。

同意。现在可能是一个过滤器，然后探索添加UI增强以支持稍后更改它？

[蚂蚁党]

在最近的媒体组成部分会议上，与会者一致认为，在5.7周期中，改变媒体库体验可能为时已晚。已经确定这更适合作为早期的5.8的候选人，应尽快提交。

正如@joedolson在会议上指出的那样，仍在等待主要未完成项：更新附件数量、审查设计、添加附件编号过滤器。

[润滑碳水化合物]

@乔多尔森你觉得你有时间再看一看这个5.8吗？

[乔多尔森]

刷新的补丁

[乔多尔森]

媒体加载更多初始视图

[乔多尔森]

加载更多后媒体加载更多；突出显示第一个添加的图形。

[乔多尔森]

更新补丁并加载新屏幕截图，显示加载新版本的更多设计和第一个新加载附件的视觉指示器。

[乔多尔森]

更新补丁-语言更改，突出显示新项。

[乔多尔森]

更新的修补程序完成了四件事：

1） 根据媒体会议更改了“跳转”链接的语言。
2） 描述当前显示内容的语言已更改。
3） 当所有显示的项目都已修复时，“跳转到第一个加载的”按钮未对齐。
4） 在添加的所有项目上添加突出显示，以帮助视觉用户感知插入点和更改。

@antpb我没有找到一种方法来解析新添加的项，但我确实发现可以向所有已加载的项添加一个类，并使用：not选择器反转选择以突出显示。

[乔多尔森]

显示新语言的屏幕截图

[乔多尔森]

仍然需要：如果从集合中添加或删除项目，则更新集合总大小。

[弗朗西纳]

已测试

• MacOS 11.3版
• Safari 14.1款
• WordPress 5.8-alpha-50427-src公司
• 二十二点一1.3

应用后，您必须运行npm运行开发为了获得预期的行为，可能值得将其添加到测试说明中。

这个补丁几乎完全适用。在网格模式下上传图像时，总计数器不会更新。只有在刷新页面或再次上传之后。我将附上图像。

也就是说，我有一些反馈：

• UX/UI-网格视图中没有屏幕选项，因此无法一次性更改加载的图像数量
• 复制-“跳转到第一个加载的”指令不清楚。首先什么？当然是图像，但可以更清晰。第一次是什么时候？在最后一批中。如果我想跳转到倒数第二批中加载的第一个图像，该怎么办？
• UX/UI-您考虑过分页吗？如果你有大量的图像，这个补丁解决了无限滚动的问题，但没有解决将图像分成小批量的问题。

谢谢！

[弗朗西纳]

总内容未更新

[乔多尔森]

谢谢，@francina！

在一次媒体会议上，我们讨论了设置图像数量可能是未来发布的一个问题，因为有许多UI问题需要解决。分页也是如此，因为在媒体库中实现分页将是对其工作方式的一个更实质性的改变。

总计数器更新仍然是一个未解决的问题；这是需要解决的主要悬而未决的问题。

“跳转到”的语言是一个难题。它不能是“图像”，因为媒体库包含所有类型的附件。我想可能是“依恋”；我不确定这是否容易理解。但有一个问题是它有多冗长，例如“跳转到这个请求中加载的第一个附件”似乎非常冗长。

我对具体的语言没有特别的偏见；很高兴听到建议！其目的当然是达到易于理解，但最好也是简明扼要的目的。

你能告诉我补丁里有什么不适合你吗？这个补丁应该是干净的；昨天它很新鲜，所以我不知道为什么它不干净。

我不确定是什么npm运行开发做；这不是我用过的命令。

[弗朗西纳]

谢谢@joedolson！
将其作为MVP发布，然后使用分页或任何其他使其可访问、易于理解和可用的解决方案进行迭代是有意义的🙌

跳转到此请求中加载的第一个附件“似乎非常冗长。

任何时候，我都会对不清楚的问题非常详细，但这当然只是我的观点。“请求”是一个技术术语，最终用户可能与之无关。

“跳转到前一个XX中上传的第一个文件”，其中XX可能是“操作、加载、命令”？
我也在想，在意大利语“Vai al primo file caricato dopo l’ultima operazione”中，这听起来会怎么样。
我给自己添加了一个提醒，提醒ping文案和多语种人士想出一个合理的文本：）

---

关于补丁，它应用得很干净。我所说的“几乎”是指总计数器的已知错误。但我不清楚如何传达这一点，我的错。

---

最后是npm命令。此票据没有测试说明，因此需求-测试-信息添加了关键字。
自从WordPress 5.6以来，发布团队就有了一个测试重点负责人：据观察，没有多少门票以所需步骤列表的形式添加了明确的测试说明。
我确实想好了该做什么，并注意到当你使用文字印刷-开发你还需要一个npm运行开发应用修补程序后。但它只适用于在特定环境下进行测试的人。

[蚂蚁党]

在最近的媒体组件会议上讨论过，说“跳转到第一个加载的项目”是一种很好的方法，可以使这一点不再关注图像或任何其他特定媒体类型，而且仍然清晰可见。我同意一些人可能会感到困惑，并错过了这句话的“加载”部分，但这可能是最明确的选择。这也是一个影响很小的按钮，因为它可以让你保持在同一个页面上，只改变你的焦点。人们很快就会知道它的作用。

@hellofromTonya在关于未决计数问题的会议上提到，“我认为如果我们能在5.8测试版之前解决这个问题，对用户来说将是一个更好的体验。也就是说，最好能提交补丁程序，然后继续解决这个特定问题。”

与会者一致认为，尽早登陆并迭代是理想的

[乔多尔森]

更新了补丁语言更改，解决了jshint问题。

[乔多尔森]

已更新提交修补程序。

在提交之前，这将要求您在应用补丁后构建WordPress。

测试：

测试前提条件：媒体中至少需要41个项目才能与这些控件交互。

• 通过浏览器的开发工具调节网络：这有助于观察UI行为，例如微调器可见性
• 测试是否启用了媒体垃圾（通过设置define（'media_trash'，true））；在wp-config.php文件中）
• 测试过滤器addfilter（'media_library_infinite_srolling'，'return_true'）；恢复无限滚动行为
• 测试媒体库
• 使用Classic Editor测试媒体模式（添加媒体，特色图像）
• 用古腾堡测试媒体模式（图像、图库、特色图像……）
• 可选：使用插件测试Media模式，插件实现自定义帧以上传/选择附件
• 检查缓存集合的查询数组

[乔多尔森]

在50829:

媒体：从媒体库和模式中删除无限滚动。

用用户控制的加载更多按钮替换滚动条上的无限自动加载行为。修复媒体库中长期存在的可访问性问题。无限滚动在可访问性、可用性和性能方面带来了广泛的问题。

此更改将库修改为在初始视图中加载40个项目，使用“加载更多”按钮加载接下来的40个项目和一个按钮将焦点从加载更多区域移动到最近添加的第一个项目。

对传递跳转目标的文本进行了广泛讨论，一致认为此处包含的文本将最简洁、最清楚地传达按钮的用途，任何进一步的细节都可以从使用中学习。

Props afercia、adamsilverstein、joedolson、audrasjb、francina
修复#50105。请参阅#40330.

[乔多尔森]

请参见#53171继续讨论附件总数问题。

[谢尔盖·比尤科夫]

在50831:

文件：正确@自标记，用于与媒体库中的无限滚动相关的新属性和功能。

后续行动[50829].

请参见#50105,#40330,#52628.

[spacedmonkey（空格键）]

此更改已在Unsplash插件中中断。这一改变可能会破坏扩展媒体库的其他插件。有没有办法在不破坏集成的情况下做到这一点？

[乔多尔森]

我们需要一些关于破损情况的信息；你能提供任何有关损坏内容的文档吗？

如果需要，插件作者应该能够使用新的过滤器恢复无限滚动：

add_filter（'media_library_infinite_srolling'，'return_true'）

但如果我们能做些什么来确保插件仍然工作，我们很乐意去做。

[spacedmonkey（空格键）]

这里的问题是函数输出的更改wp_ajax查询附件

$result=数组(“附件”=>$posts，“totalAttachments”=>$query->found_posts，);wp_send_json_success（$result）；

输出管理ajax调用现在不同了。我认为更改这里的响应格式可能会破坏其他插件，因为此管理ajax调用用于其他插件，而不仅仅是核心插件。我在客户端使用过几个插件。

不要更改管理ajax调用的输出，而不仅仅是对这些视图使用RESTAPI。如果您将_envelope参数添加到rest-api调用中，那么您将获得相同的数据。请参见​https://make.wordpress.org/core/wp-json/wp/v2/media？_envelope=true

[约十亿美元]

好点子。然而，为这些视图切换到RESTAPI也会破坏覆盖此输出的插件，例如​资产管理器框架.

添加附件总数数据作为HTTP头并保持正文响应不变？

[约十亿美元]

哎哟，我没看到这件事已经发生了：）

[约十亿美元]

重新开放以进一步讨论上述内容。有​覆盖此Ajax响应的几个插件.

如果此更改保持为-is，则需要关于中断更改的开发注释。如果不是这样的话，我认为通过HTTP标头提供总计是有意义的，但这也可能意味着，如果插件覆盖Ajax处理程序，那么核心无法相信标头已经就位。

[spacedmonkey（空格键）]

可能还有其他插件正在使用这些重用下划线视图，但不覆盖管理ajax调用。例如，在Unsplash插件中，我们有一个自定义REST API端点，它复制了管理ajax调用的格式。

当前，视图会执行此操作。

解析：函数（响应，xhr）{if（！_.isArray（response.attachments））{响应=[response.attachments]；}this.totalAttachments=解析Int（response.totalAttachements，10）；

但是，如上所述，属性attachments和totalAttachments可能不存在。代码应该首先检查这些属性是否存在，如果不存在，则不会返回到以前的行为。如果totalAttachments不存在，还应重新启用无限滚动。

这样的做法可能会奏效。

解析：函数（响应，xhr）{if（response.hasOwnProperty（'附件'）&&！_。isArray（响应.附件）{响应=[response.attachments]；}if（response.hasOwnProperty（'totalAttachments'））{this.totalAttachments=解析Int（response.totalAttachements，10）；}

[spacedmonkey（空格键）]

如果我们使用http头，我认为我们应该遵循REST API中的头。

例子

$response->header（'X-WP-Total'，（int）$Total_posts）；$response->header（“X-WP-TotalPages”，（int）$max_pages）；

[乔多尔森]

我认为，检查是否存在特定于此更新的值，并以这些值的存在为条件进行更改是有意义的。如果有任何插件覆盖了数据，那么我认为我们应该假设插件对如何获取和显示数据承担全部责任。

至少，这对我来说很有意义。

Trac票：

[亚当西尔弗斯坦]

我稍微看了一下（公关进展：​https://github.com/WordPress/WordPress-develop/pull/1335)发现xhr对象没有按预期传回以获取，所以我无法从标题中提取帖子计数.也许我们正在其他地方重写fetch？

感谢您在这里帮助获取标题，我同意将此数据移动到标题将最好避免破坏任何内容。另一种选择是为请求引入一个新端点。

[乔多尔森]

在\wp\api.js; 这就是设置和获取X-WP-Nonce标头的地方。

有趣的是，这里记录的XHR对象实际上并不是XHR的对象，而且我看不到任何覆盖模型上的获取的内容。同步正在被扩展，但这不应该改变获取的任何内容。

可能能够按照用于添加nonce的路径来获取此标头。我目前正在考虑这个问题。

[乔多尔森]

好吧。我想这就是我看到的。

媒体库覆盖中的backbone.sync媒体\模型\查询.js.

Backbone使用.fetch来使用.sync获取数据，因此这将覆盖默认的Backbore返回值，使用wp.媒体.ajax而不是。wp.媒体.ajax（的别名wp.ajax.send发送)不返回XHR对象，而是返回一个承诺。

我们可能会给wp.媒体.ajax它获得了XHR对象，但我完全不确定会有什么额外的后果。

在我的快速测试中，我能够获得一个XHR对象，但我没有看到标头出现，但我今天没有时间来寻找原因。

[spacedmonkey（空格键）]

我想知道没有取回xhr对象与jQuery更新和一些在幕后进行的更改有关，但事实似乎并非如此。

一旦我们得到了头，我仍然建议使用与RESTAPI相同的头。

X-WP-总计
X-WP-总页数

不添加新的响应标头，将有助于与那些使用防火墙和保留代理的响应标头兼容。在这些情况下，必须将新标题添加到允许的列表中。重复使用现有的标头可以避免这种情况。

[乔多尔森]

@adamsilverstein的不完整补丁使用了X-WP-TotalPages，所以它包含在@spacedmonkey中。我认为我们没有任何理由改变这一点。

[亚当西尔弗斯坦]

更新的功能修补程序：​https://github.com/WordPress/WordPress-develop/pull/1335，也附在这里。

[亚当西尔弗斯坦]

在50105.10.diff（如果有）​:

• 将附件总计传回标题，恢复原始响应数据形状。
• 使用自定义的获取处理程序捕获返回的X-WP-Total标头。
• 确保加载完成后微调器隐藏。
• 使用剩余端点的方法计算正确的X-WP-Total和X-WP-TolalPages页眉。

[蚂蚁党]

我们将在Beta 1中对此保持开放，因为它是部分合并的，或者与部分合并的功能相关。所需要的只是对上述补丁进行审查，并可能进行改进。如果在Beta 3之前无法合并补丁，我们需要恢复，以便在发布候选版本之前测试恢复。

[亚当西尔弗斯坦]

@spacedmonkey你能在Unsplash插件中确认你的用例的最新补丁吗？

@在johnbillion的最新补丁中，我还原了ajax回调数据返回形状，使其与5.7中的内容相匹配，并按照您的建议将post和page计数信息放在标题中。您能否确认这将恢复Asset Manager Framework的兼容性？

@joedolson你能检查一下最新的补丁并进行测试吗？最大的潜在突破性变化是​将xhr设置为上下文所以我们可以访问标题。最后，我还更改了如何处理微调器隐藏来修复行为。

一旦我们确认这项工作正常，我们应该在beta 2之前提交，以便进行更广泛的测试，尤其是对可能受影响的插件。

[乔多尔森]

这似乎对核心很有效。

对于Unsplash插件，它可以部分工作-然而，Unsplas正在替换返回的对象，因此XHR对象不是我们期望的位置。对于这一个用例，我们可以轻松地解决这个问题，因为它们在_更多:

if（this.hasOwnProperty（'getResponseHeader'））{collection.totalAttachments=parseInt（this.getResponseHeader（'X-WP-Total'），10）；}其他{collection.totalAttachments=parseInt（this._more.getResponseHeader（'X-WP-Total'），10）；}

不过，可能对核心不实用。

如果无法获得总附件数，我还尝试设置默认值。

if（this.hasOwnProperty（'getResponseHeader'））{collection.totalAttachments=parseInt（this.getResponseHeader（'X-WP-Total'），10）；}其他{collection.totalAttachments=0；}

这对Unsplash非常有效，我可以想象这可能是一个很好的通用解决方案，尽管我们只能在实际测试中发现这一点。

我确实发现了在更改选项卡时发生的一个错误-在最初加载媒体库选项卡时，会出现加载更多按钮，但如果您切换到其他选项卡（例如，转到上载文件选项卡或自定义选项卡，如Unsplash），它就会消失。

[乔多尔森]

啊，加载更多按钮没有消失，它只是隐藏起来了。

问题是更新加载更多视图仅在“添加-删除-重置”事件中加载，并且在切换选项卡时不会触发这些事件。

更新加载更多视图也需要在显示视图时激发。

通过在与更新内容，这对我来说很有意义。

即将推出新补丁。

[乔多尔森]

处理非XHR对象的返回数据，并在模式下切换选项卡时创建更多加载视图

[乔多尔森]

新补丁可以处理更多的边缘情况；仍然可以使用@spacedmonkey和@johnbillion的反馈

[spacedmonkey（空格键）]

我目前正在做一些测试。我没有看到加载更多按钮，所以我会推迟。

由于有了一个新的包装div，css也发生了变化，所以unsplash集成将被更新。我认为更改为使用标题，对我们来说更新并不是很容易。我们还必须映射标题。

需要进行一些错误处理以检查标头是否存在。如果它不存在，那就退回到老行为上来。

[乔多尔森]

测试最新的补丁，如果你还没有。

加载更多按钮不应显示在Unsplash中。

[spacedmonkey（空格键）]

在做了更多的测试之后，我让它工作了。我需要做一个小版本来禁用无限滚动并做一些css调整，但最新的补丁工作得很好！

[乔多尔森]

关于新的包装器，这是分组媒体项和加载更多接口所必需的。我认为这是必要的，但它早于我在这个问题上的工作点，所以我不能肯定这是实现这一目标的唯一途径。

但听起来你可以很容易地解决这个问题，这实际上应该是真的。

注意，额外的包装器应该放在开发说明中。

[乔多尔森]

我已经做了一些持续测试，我认为这已经准备好提交了。它清楚地修复了现有实现中的问题，为插件创建了一条更好地适应变化的路径，而且据我所知，没有产生新的问题。

[乔多尔森]

在51145:

媒体：恢复媒体库中的AJAX响应数据形状。

删除无限滚动后，恢复媒体库中AJAX响应数据的原始形状，并在响应标头中传递附件总数X-WP-总计和X-WP-汇总页面.

提高拦截ajax响应的插件的向后兼容性。标头与REST API响应中使用的结构和计数计算相匹配。

修复加载完成后隐藏微调器的问题，并确保在媒体库模式中更改选项卡时创建加载更多视图。

后续行动[50829].

道具亚当·西尔弗斯坦、spacedmonkey、乔多尔森。
修复#50105.

[德斯罗什]

在51147:

编码标准：应用一些对齐修复。

后续行动[51145].
请参见#50105.

[德斯罗什]

在51222:

文档：删除不准确的@自标签。

后续行动[50829,50831,51145,51147].

道具约十亿。
修复#53461。请参阅#50105.

[德斯罗什]

看起来像是[51145] 引入了未定义的变量.$query_args（查询参数）应该是$查询在里面wp_ajax_query_attachments（）.

我和@adamsilverstein核实了一下，只是为了确定，似乎这确实需要纠正。

[audrasjb公司]

的确，$query_args（查询参数）未定义，并且$查询应改为使用。

[乔多尔森]

在51224:

媒体：更正wp_ajax_query_attachments中未定义的变量。

修复中引入的命名错误的变量[51145].变更$query_args（查询参数）到正确定义的变量$查询.

后续行动[51145].

道具desrosj，audrasjb。
修复#50105.

[ricjcs公司]

就我个人而言，我不太喜欢这个功能的实现和可用性。要滚动浏览多个图像，我必须多次按下按钮，速度没有以前快。我认为这应该是一个可选功能。

[野生动植物]

我使用了媒体库定义滚动WordPress 5.8-RC3中的过滤器，但未启用无限滚动。

这是密码。

add_filter（'media_library_infinite_srolling'，'__return_true'）；

我在媒体库页面上发现了一个javascript错误。

[乔多尔森]

嗨，@wildworks

你是如何实现这个过滤器的？我刚刚使用Media Library Enable Infinite Scroll插件进行了测试，它似乎像预期的那样工作。

然而，当启用media_library_infinite_srolling时，我能够重现JS错误，我将对此进行研究。

[乔多尔森]

JS错误非常简单；很快就会修复。已创建新票证：#53672

[野生动植物]

@乔多尔森
感谢您的快速回复。
我将在新版本发布后再次测试它。

[博尼乌91]

嘿！测试完这张票后，我确实有一些问题。

1. 当从后期编辑视图添加图像时，看起来像是RWD效果不佳：

2. 我不确定在新加载的图像下使用灰色背景的目的是什么，这是预期的吗？

3. 我遇到过第一张和第二张新上传的图片没有突出显示的情况（注意第一张图片在虚线框中）

[格斗]

40岁现在感觉还可以。我们可以确保这是可过滤的。如果这是用户可以控制的，比如每页的帖子，或者每页的术语来自屏幕选项选项卡，那就更好了。

@adamsilverstein 5.8？未添加过滤器？如果不是的话，开一张火车票有意义吗？我个人对过滤器最感兴趣，尽管UI对普通用户来说很好。

[乔多尔森]

我认为@antpb会在上周的媒体会议之后这样做（或者至少开张罚单，讨论媒体画廊每页编号的更改），但它还没有开放。5.8中没有添加过滤器，所以我认为您可以继续打开该罚单。

[赫雷格伦]

此票证的修复程序似乎对查询附件ajax操作。

以前不需要发送帖子_每页变量。现在不发送此消息会导致除以0。

请注意，不传递它会导致它回退到全局选项，但这是在查询变量属性，而不是查询属性，该属性仍将包含最初传递的参数。

在此修复中查询属性用于确定可能没有帖子_每页完全设置。

[德斯罗什]

@赫雷格伦，你能开一张新票来解决这个问题吗？