Terminal: ColorTool: perbaikan bug parser dan refactor kode kecil

Dibuat pada 21 Apr 2019  ·  3Komentar  ·  Sumber: microsoft/terminal

Saya menggunakan ColorTool sebagai bagian dari mengelola tema untuk cmd / PowerShell, dan saya mengalami beberapa masalah saat menggunakannya:

  1. Pesan kesalahan dilempar dari parser yang tidak digunakan, misalnya parser INI mencoba untuk menjalankan meskipun Anda menggunakan itermcolors , jadi kesalahan akan muncul karena XML bukan file INI yang valid.
  2. Indeks Latar Depan / Latar Belakang tidak disimpan saat menggunakan --defaults (atau --both ).

Ini adalah perbaikan yang cukup mudah dan saya akan dengan senang hati memperbaikinya. Saya berpikir untuk membersihkan kode sedikit saat saya di sana, yaitu:

  1. Mengizinkan parser berjalan secara bersyarat, jadi tidak semua parser berjalan untuk setiap file (harus memadamkan beberapa kesalahan)
  2. Menambahkan antarmuka umum untuk "menyetel properti konsol saat ini" dan "default konsol", sehingga lebih sulit untuk melupakan fungsionalitas penerapan keduanya.

Namun, sebelum saya mulai mengerjakan ini, saya ingin memberi tahu Anda sebelumnya, dan memastikan saya tidak bekerja untuk tujuan yang berlawanan dengan seseorang dalam proyek ini. Apakah Anda tertarik dengan PR yang memperbaiki 2 poin di atas?

Terima kasih!

Area-Settings Issue-Bug Product-Colortool Resolution-Fix-Available

Semua 3 komentar

@waf , itu akan menjadi perubahan yang sangat disambut. Tidak ada orang di pihak kami yang saat ini mengerjakan salah satu dari itu.

@ DHowett-MSFT terima kasih, saya telah membuka # 418 untuk ini.

Saya yakin ini sekarang tersedia dalam rilis yang saya buat beberapa minggu lalu.

Apakah halaman ini membantu?
0 / 5 - 0 peringkat