serdarcevher
commented
8 years ago Merhaba, öncelikle emeğiniz için çok teşekkürler. Ancak şöyle bir problem var; birincisi, proje temel olarak bir Chrome eklentisi olduğu için temel kodu birden fazla tarayıcı için aynı anda çalıştırmak gibi bir yöntem benimseyerek conditional kısımlar eklemek yerine, Firefox eklentisinin firefox isminde ayrı bir klasör altına alınması daha uygun olur diye düşünüyorum. Duruma göre ana klasördeki CSS'i include edebilir veya Firefox'a özgü ayrı bir CSS dosyası kullanabilirsiniz.
İkinci olarak da, Chrome Store sadece ID'yle erişim imkanı tanısa bile bu aşamada Chrome için olan eklentinin isminin sabit kalmasının daha doğru olacağına inanıyorum. Bu yüzden Firefox eklentisinin kendi manifest.json dosyasına sahip olmasının ve yazar bölümünde de sizin isminizi taşımasının mantıklı olacağı görüşündeyim.
Bu şekilde bir commit gönderirseniz en kısa sürede firefox klasörü altında ilgili dosyaları repo'ya eklerim efendim.
dcelasun
Merhaba,
5 nolu issue'da konuştuğumuz üzere eklentiye Firefox desteği ekledim. Yalnız önceden kestiremediğim birkaç nokta var, sorun yaratmıyor ama bilginiz olsun:
manifest.jsoniçinde uygulamanın adını "Eksi Sozluk Normallestirici" olarak değiştirmek zorunda kaldım, zira başka birisi aynı adı kullanmış (linkteki eklenti sadece css değiştiriyor ve eski Jetpack SDK'yı kullanıyor). Chrome Web Store için sorun olmaması lazım, çünkü orada ID kullanılıyor, isim değiştirmek sorun olmuyor.script.jsiçinde storage API'larına erişilemiyor (bug #1197346). Firefox tarafında düzeltilince ben de düzelteceğim.Görüşleriniz, sorularınız varsa cevaplayabilirim. addons.mozilla.org'a gönderdim, 1-2 hafta içinde onaylanacaktır tahminen.