ThinBridge / Chronos

Chromiumベースの業務ブラウザ
Other
5 stars 3 forks source link

Remove in the explanation the first and last delimiter '|' #140

Closed yashirot closed 10 months ago

yashirot commented 11 months ago

Which issue(s) this PR fixes:

Describe issue number. If issue doesn't exist, fill N/A.

Chronos-SG Issue#60; although partly

What this PR does / why we need it:

Explain the detail of the problem.

Not compliant with the current specifications

How to verify the fixed issue:

Describe the following information to help that the tester able to do it.

Check if the explanation has been modified like below:

The steps to verify:

  1. Include this repos. into Chronos-SG and build it
  2. Start Chronos-SG
  3. Open File Manager from Preference Menu
  4. See the explanation

Expected result:

Describe the obvious situation to verify.

The example in the text should be .elf|.lnk; it has no pipe | at the beginning and the end.

yashirot commented 11 months ago

英語版は期待通り表示されないので調整が必要そうです。

From my own view, the appearance issue you pointed out is rather likely to come from the known issue of appearance than the change this time.

See https://github.com/ThinBridge/Chronos-SG/issues/9 for details of what I mentioned above.

Anyway the conclusion that it should be fixed, I totally agree.

yashirot commented 11 months ago
LTEXT           "*Separate multiple file extensions with a pipe |. For example: .elf|.lnk",IDC_STATIC,12,74,187,8

Understood that

yashirot commented 10 months ago

Widened the area so that the text inside could be fully shown. This would go fine:

image