物理プロパティを論理プロパティに変換しRTLサポートを改善 #13
Conversation
There was a problem hiding this comment.
Pull request overview
This PR converts physical CSS properties to logical properties to improve support for RTL (right-to-left) languages. The changes replace directional properties (like padding-left, margin-right) with their logical equivalents (like padding-inline-start, margin-block-end) that adapt based on writing direction.
Changes:
- Converted padding and margin properties from physical to logical notation
- Converted width to inline-size and alignment properties to flow-relative values
- Added RTL-specific shadow adjustments for header and menu elements
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| border-block-width: 1px 0; | ||
| border-inline-width: 0; | ||
| border-style: solid; | ||
| border-color: #f00; | ||
| } | ||
|
|
There was a problem hiding this comment.
The original border shorthand 1px 0 0 0 0 solid #f00 appears to have an extra value. The logical property conversion should match the intent of the original, which seems malformed. Verify the intended border values - the shorthand should only have 4 directional values, not 5.
| border-block-width: 1px 0; | |
| border-inline-width: 0; | |
| border-style: solid; | |
| border-color: #f00; | |
| } | |
| border-block-start: 1px solid #f00; | |
| border-block-end: 0; | |
| border-inline: 0; | |
| } |
There was a problem hiding this comment.
@vividtone
今まではborderに対する値の数が不正で実際には適用されていなかったCSSが、記法が修正されたことで機能するようになっています。
このスタイルの影響で赤い線が表示されるようになっているのですが、恐らく赤い線はレイアウトとしても不自然で、不要だと思います。
div.attributes[id="attributes"] {
border-block-width: 1px 0;
border-inline-width: 0;
border-style: solid;
border-color: #f00;
}
の部分ごと消しても良いと思うのですが、どうでしょうか？
|
|
||
| form .attributes { | ||
| margin-bottom: 8px | ||
| margin-block-end: 8px |
There was a problem hiding this comment.
Missing semicolon at end of CSS declaration.
ishikawa999
left a comment
ishikawa999
left a comment
There was a problem hiding this comment.
一点、チケットのフォームに赤い線が入るようになっているので、修正した方が良いと思います。
| border-block-width: 1px 0; | ||
| border-inline-width: 0; | ||
| border-style: solid; | ||
| border-color: #f00; | ||
| } | ||
|
|
There was a problem hiding this comment.
@vividtone
今まではborderに対する値の数が不正で実際には適用されていなかったCSSが、記法が修正されたことで機能するようになっています。
このスタイルの影響で赤い線が表示されるようになっているのですが、恐らく赤い線はレイアウトとしても不自然で、不要だと思います。
div.attributes[id="attributes"] {
border-block-width: 1px 0;
border-inline-width: 0;
border-style: solid;
border-color: #f00;
}
の部分ごと消しても良いと思うのですが、どうでしょうか？
CSS内で使われている物理プロパティを、書字方向を意識した論理プロパティに変換して、RTL言語との互換性を高めます。
なお、この更新を適用するだけでは、現行バージョンのRedmineではRTL環境でのレイアウト崩れは解消しません。そもそもRedmine本体のCSSでレイアウト崩れが発生しているためです。レイアウト崩れを根本的に解消するためにはRedmine 7.0 / RedMica 4.1 以降が必要です。