ionic-team / ionic-team/ionic-framework

bug: (ion-col) offset, push, and pull props are not RTL-aware

未关闭
#26,088 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
package: core type: bug
主要语言
TypeScript
星标
52.7k
派生
13.3k
平均合并
1 天 14 小时
30 天内合并 PR
50

描述

### Prerequisites

- [X] I have read the [Contributing Guidelines](https://github.com/ionic-team/ionic-framework/blob/main/.github/CONTRIBUTING.md#creating-an-issue).
- [X] I agree to follow the [Code of Conduct](https://ionicframework.com/code-of-conduct).
- [X] I have searched for [existing issues](https://github.com/ionic-team/ionic-framework/issues) that already report this problem, without success.

### Ionic Framework Version

- [ ] v4.x
- [ ] v5.x
- [X] v6.x
- [ ] Nightly

### Current Behavior

The `offset`, `push`, and `pull` props for `ion-col` always offset/push/pull in the same direction (for example, always to the right for `offset`), regardless of whether the page is in LTR or RTL.

### Expected Behavior

The display of the columns should be a mirror image between LTR and RTL. For example, `offset` should push to the right in LTR and to the left in RTL.

### Steps to Reproduce

Offset: https://stackblitz.com/edit/rnuv7g?file=index.html
Push/pull: https://stackblitz.com/edit/jmyavm?file=index.html

Change the `dir` on the `body` tag from `ltr` to `rtl`. Notice that the layout does not get mirrored properly.

### Code Reproduction URL

https://stackblitz.com/edit/rnuv7g?file=index.html + https://stackblitz.com/edit/jmyavm?file=index.html

### Ionic Info

See Stackblitzes

### Additional Information

_No response_

贡献指南

打开贡献指南

调研方向

使用报告中链接的两个 StackBlitz 示例重现该问题,将 body 的方向从 ltr 改为 rtl。从 ion-col 组件开始,跟踪 offset、push 和 pull 的应用方式;当列布局在两个方向上都能正确镜像时,即表示完成。

由索引模型根据 Issue 内容生成。

评估

技术栈
typescript
领域
frontend, internationalization
Issue 类型
缺陷
难度
3/5
预计耗时
1-2 天
活跃度
停滞
描述清晰度
基本清楚
新手友好度
45/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。