已合并
fix: include attachments in nested text touch targets #1879
fix: include attachments in nested text touch targets #1879
已合并
Patryk Fryda创建于 2025年12月8日
Patryk Fryda
Patryk Fryda
2025年12月8日

Description

Attachments were excluded from touch target list and fragment rects calculation, creating unresponsive areas. Now all fragments (text and attachments) are included.

Two issues are solved:

  • one character mismatch in fragment's rect calculation when nested text contains attachment,
  • pressing area above attachment did had no effect

Changes

  • fixed nested text attachment touch behaviour

Test Plan

How to test this PR. For example:

  1. Open the Tester project
  2. Execute the npm run start command
  3. Go to Text section
  4. Test pass after pressing area above the attachment and pass after pressing the last character

Checklist

likedislike
Pull Request已成功合入, 合并人@huangyouhua
(感谢 Patryk Fryda 的贡献)
openharmony_ci
openharmony_ci成员
2025年12月8日 评论:

感谢提交 Pull Requests !此PR未通过DCO校验。
校验失败可能原因:

1. 未签署“DCO协议”(开发者原创声明协议),在线签署、查看签署状态。

2. Commits 中未包含 Signed-off-by信息,参考FAQ处理。

修复上述问题后,在PR的评论框输入“check dco” ,单击”评论”,系统将再次进行DCO校验。

当前检测到如下Commits 未包含Signed-off-by信息:


Thanks for submitting a pull request. This pull request has not passed the DCO check.
Possible causes:

1. You have not signed the Developer Certificate of Origin (DCO). Sign the DCO and check DCO status.

2. The commits do not contain the Signed-off-by information. To resolve this issue, see FAQs.

After resolving the preceding issues, enter check dco in the comment box of this pull request and click Comment. The system will check DCO status again.

The following commits do not contain the Signed-off-by information:

likedislike
openharmony_ciopenharmony_ci成员
2025年12月8日 添加了label:dco检查失败
Patryk FrydaPatryk Fryda
2025年12月8日 修改了pull request 的描述
Patryk FrydaPatryk Fryda
2025年12月8日 修改了pull request 的描述
Patryk FrydaPatryk Fryda
2025年12月8日 强制推送  1 个提交:44a36b39-fix: include attachments in nested text touch targets
Patryk FrydaPatryk Fryda
2025年12月8日 关联了issue:Nesting a View inside Text offsets the touch area by about one character
openharmony_ci
openharmony_ci成员
2025年12月8日 评论:

感谢提交 Pull Requests!
Thanks for submitting a pull request.

likedislike
openharmony_ciopenharmony_ci成员
2025年12月8日 删除了label:dco检查失败
openharmony_ciopenharmony_ci成员
2025年12月8日 添加了label:dco检查成功
gyange成员
2025年12月9日 评论:

start build

likedislike
openharmony_ci
openharmony_ci成员
2025年12月9日 评论:

非PR提交人或管理员不允许触发门禁

likedislike
zichenyang21zichenyang21
2025年12月9日 将zichenyang21设为评审人
zichenyang21zichenyang21
2025年12月9日 通过了评审
Patryk Fryda
Patryk Fryda
2025年12月9日 评论:

start build

likedislike
openharmony_ci
openharmony_ci成员
2025年12月9日 评论:

首次触发
门禁构建开始,包含静态检查、代码编译【ohos_react_native_master编译】,预计在60分钟内完成,门禁结果会同步发送到注册邮箱。您可以通过如下链接跟踪门禁进展:http://dcp.openharmony.cn/workbench/cicd/detail/6937c8a664650f998bfbbbb0/runlist

likedislike
openharmony_ciopenharmony_ci成员
2025年12月9日 添加了label:编译成功
openharmony_ciopenharmony_ci成员
2025年12月9日 添加了label:静态检查成功
openharmony_ciopenharmony_ci成员
2025年12月9日 通过测试
openharmony_ci
openharmony_ci成员
2025年12月9日 评论:

代码门禁通过
您可以通过如下链接查看门禁报告:http://dcp.openharmony.cn/workbench/cicd/detail/6937c8a664650f998bfbbbb0/runlist

静态检查:

# check type result report
1 codeCheck pass >>>

编译测试:
# Device build result package
1 ohos_react_native_master success >>>

likedislike
gyange成员
2025年12月9日 评论:

@pafry please resolve the conflict, and then retry.

likedislike
Patryk FrydaPatryk Fryda
2025年12月9日 评审状态已重置,评审人: zichenyang21
Patryk FrydaPatryk Fryda
2025年12月9日 测试状态已重置,测试人: openharmony_ci
Patryk FrydaPatryk Fryda
2025年12月9日 强制推送  3 个提交:07606e83-2 commits from branch master,aca79d98-fix: include attachments in nested text touch targets
openharmony_ciopenharmony_ci成员
2025年12月9日 删除了label:静态检查成功
openharmony_ciopenharmony_ci成员
2025年12月9日 删除了label:编译成功
openharmony_ci
openharmony_ci成员
2025年12月9日 评论:

代码有更新,重置PR验证状态

likedislike
Patryk Fryda
Patryk Fryda
2025年12月9日 评论:

start build

likedislike
openharmony_ci
openharmony_ci成员
2025年12月9日 评论:

感谢提交 Pull Requests!
Thanks for submitting a pull request.

likedislike
openharmony_ci
openharmony_ci成员
2025年12月9日 评论:

本地或库上代码有更新,全量重新构建,重置所有关联PR的验证状态
门禁构建开始,包含静态检查、代码编译【ohos_react_native_master编译】,预计在60分钟内完成,门禁结果会同步发送到注册邮箱。您可以通过如下链接跟踪门禁进展:http://dcp.openharmony.cn/workbench/cicd/detail/69380cf264650f998b6565af/runlist

likedislike
openharmony_ciopenharmony_ci成员
2025年12月9日 添加了label:编译成功
openharmony_ciopenharmony_ci成员
2025年12月9日 添加了label:静态检查成功
openharmony_ciopenharmony_ci成员
2025年12月9日 通过测试
openharmony_ci
openharmony_ci成员
2025年12月9日 评论:

代码门禁通过
您可以通过如下链接查看门禁报告:http://dcp.openharmony.cn/workbench/cicd/detail/69380cf264650f998b6565af/runlist

静态检查:

# check type result report
1 codeCheck pass >>>

编译测试:
# Device build result package
1 ohos_react_native_master success >>>

likedislike
Patryk Fryda
Patryk Fryda
2025年12月9日 评论:

@gyange it is ready

likedislike
zichenyang21zichenyang21
2025年12月10日 通过了评审
huangyouhuahuangyouhua
2025年12月10日 通过审查
huangyouhuahuangyouhua
2025年12月10日 关闭了关联的issue
huangyouhuahuangyouhua
2025年12月10日 合入了pull request