search

ThinBridge / Chronos

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

Disable to edit default Upload/Download field #145

Closed kenhys closed 10 months ago

kenhys commented 10 months ago

Which issue(s) this PR fixes:

CSG#5

What this PR does / why we need it:

Chronos doesn't expect the default Upload/Download path is changed from B: drive. In other words, if changed from B: to something, it doesn't work expected.

This situation is not use-friendly, so disable it by default.

How to verify the fixed issue:

  1. Show Tool > Preferences > File Manager

Expected result:

Text edit field about drive information is readonly.

HashidaTKS commented 10 months ago

Memo for me:

実装を見る限りでは、必ずしもB:\直下でなくても、ドライブがB:\でありさえすれば、予めB:\testfolderのようなものを用意しておいて、既定のフォルダーをB:\testfolderとすれば、現状でも動作しそうではある。 ※予めB:\testfolderが存在していないと、ファイルマネージャーからそのフォルダを作成しようにも、ファイルマネージャーを開いたタイミングでB:\testfolderがなくてエラーになってフォルダも作れない。

結局、（管理者も含めたChronosの）ユーザーが画面から気軽に変更できる値ではないので、画面から変更できなくても問題なさそう。 つまり、現在の対応内容で良さそう。

HashidaTKS commented 10 months ago

Also we should update documents in another task. RootPath should be moved from GUI項目名 to 非GUI項目名 in documents.

HashidaTKS commented 10 months ago

LGTM.

yashirot commented 10 months ago

To be sure: Is it right that the only way to change RootPath if needed, is to edit ChronosDefault.conf directly?

念のため：RootPathを変更するときは、ChronosDefault.confを直接編集するという認識で合ってますか？

kenhys commented 10 months ago

念のため：RootPathを変更するときは、ChronosDefault.confを直接編集するという認識で合ってますか？

はい、CSGのChronosDefault.confで設定されるものという認識です。