登录
注册
开源
企业版
高校版
搜索
帮助中心
使用条款
关于我们
开源
企业版
高校版
私有云
模力方舟
登录
注册
代码拉取完成,页面将自动刷新
开源项目
>
建站系统
>
BBS论坛/问答
&&
捐赠
捐赠前请先登录
取消
前往登录
扫描微信二维码支付
取消
支付完成
支付提示
将跳转至支付宝完成支付
确定
取消
Watch
不关注
关注所有动态
仅关注版本发行动态
关注但不提醒动态
896
Star
3.8K
Fork
1.5K
Discuz
/
DiscuzX
代码
Issues
105
Pull Requests
0
Wiki
统计
流水线
服务
PHPDoc
质量分析
Jenkins for Gitee
腾讯云托管
腾讯云 Serverless
悬镜安全
阿里云 SAE
Codeblitz
SBOM
我知道了,不再自动展开
474
【轻量级 PR】:修复 帖子列表页点击长条形下一页按钮时加载出的帖子后方图标样式异常的问题
已合并
N/A
Discuz:master
popcorner
创建于 2020-05-04 02:52
克隆/下载
HTTPS
SSH
复制
下载 Email Patch
下载 Diff 文件
可能是我提交过的内容最简单的PR :joy: (但是确实是有必要的) 在处理模板问题的时候,发现在点击页面下方最长的下一页按钮,通过ajax加载更多列表内容的时候,加载进来的每一行帖子标题后方的图标之间的间距全都是错的。准确的说,是全挤一起了,非常难看。 经仔细观察,这个长条形ajax加载列表的功能的实现方式非常的简单粗暴,加载的就是正常的下一页的帖子列表页,并用正则表达式从中匹配出每一行的帖子相关信息,然后填充到现有页面的帖子列表后面。 问题就出在这个js上,按理说加载完全相同的内容,样式是不应该有区别的,但事实上这个js在做正则匹配之前,把HTML里所有的换行符全给删了(我猜可能是当初写正则的人因为正则匹配不上换行符而去掉的)。虽说换行符本身在HTML里没什么意义也显示不出换行,但实际上在inline、inline-block等状态下,换行符在渲染时会被视为空白字符,并会在页面上渲染出一个空格,删除换行将会导致渲染出来的行为不再符合原本设计的预期。 虽说依赖这一特性做样式并不是一个好主意,但改变预期行为就更不好了。 因此本PR对此进行了修改,将原有的删除换行符的行为,更改为把换行符替换为普通空格。这样的替换可以在不改变原有正则的前提下,保证浏览器的渲染行为不变(都是在页面渲染出最多一个的空格,因此即便使用CRLF的discuz的换行会被替换为2个空格,行为也不会有区别)。
此 Pull Request 需要通过一些审核项
类型
指派人员
状态
审查
monkeye
湖中沉
DiscuzX
LooTan
Discuz!
oldhuhu
已审查通过
comsenz-service
已完成
(1/0人)
怎样手动合并此 Pull Request
git checkout master
git pull https://gitee.com/Discuz/DiscuzX.git N/A
git push origin master
评论
0
提交
1
文件
1
检查
代码问题
0
批量操作
展开设置
折叠设置
审查
Code Owner
审查人员
oldhuhu
oldhuhu
comsenz-service
comsenz-service
奶罩
naizhao
老周部落
laozhoubuluo
popcorner
popcorner
Discuz
dismall
北北″
moxue
Hyman
guohui1301
云诺
yunnuowangluo
科站网
dzlab
monkeye
Discuzx
DiscuzX
3dming
LooTan
lootan
湖中沉
zoewho
Discuz!
ComsenzDiscuz
未设置
最少人数
0
测试
oldhuhu
oldhuhu
comsenz-service
comsenz-service
奶罩
naizhao
老周部落
laozhoubuluo
popcorner
popcorner
Discuz
dismall
北北″
moxue
Hyman
guohui1301
云诺
yunnuowangluo
科站网
dzlab
monkeye
Discuzx
DiscuzX
3dming
LooTan
lootan
湖中沉
zoewho
未设置
最少人数
0
优先级
不指定
严重
主要
次要
不重要
标签
标签管理
未设置
关联 Issue
未关联
Pull Request 合并后将关闭上述关联 Issue
里程碑
未关联里程碑
参与者
(1)
Cherry-pick 提交
Cherry-pick 可以将
特定提交(Commit)
从某个分支挑选并应用到另一个分支,实现快速集成特定更改,而无需合并整个分支。
请选择应用 Cherry-pick 提交 (Commit) 的目标分支
新建分支
当前账号不存在 Fork 仓库,建议 cherry-pick 到 Fork 仓库。
Fork 仓库
提交列表
Commit SHA
Commit Message
基于 Cherry-pick 后的分支发起 Pull Request
取消
Cherry-pick
PHP
1
https://gitee.com/Discuz/DiscuzX.git
git@gitee.com:Discuz/DiscuzX.git
Discuz
DiscuzX
DiscuzX
点此查找更多帮助
搜索帮助
Git 命令在线学习
如何在 Gitee 导入 GitHub 仓库
Git 仓库基础操作
企业版和社区版功能对比
SSH 公钥设置
如何处理代码冲突
仓库体积过大,如何减小?
如何找回被删除的仓库数据
Gitee 产品配额说明
GitHub仓库快速导入Gitee及同步更新
什么是 Release(发行版)
将 PHP 项目自动发布到 packagist.org
评论
仓库举报
回到顶部
登录提示
该操作需登录 Gitee 帐号,请先登录后再操作。
立即登录
没有帐号,去注册