'Sıcaklık Değeri Elle de Girilebilecek' #21 is solved.

[asiminnesli]

Açıklama

Temperature can be entered from the input field.

#21

Değişiklik Tipi

• (feature) Temperature bar can be changed from the input field

Ekler

Nasıl Test Edilir?

• Note that the temperature value is fixed at 7000 for any temperature measurement type (celcius, kelvin, etc.).

Kontrol Listesi:

• Kodum bu projenin kod formatı kurallarına uygun.
• Kodumu inceledim.
• Özellikle anlaşılması güç olabilen alanlar için yorumlar bıraktım.
• Gerekliyse, dökümanlari güncelledim.
• Değişikliklerim yeni hatalara sebep olmuyor. ( Please check the below )
• Değişiklerim doğrultusunda testler ekledim veya testleri güncelledim.
• Varsa, değişikliklerime bağlı, diğer değişikliklerin tümü hali hazırda ana branch'te mevcut.

Known issue : when Temperature is 7000 C, user changed to C -> K ( new Temperature is equals 7273 ) but bar max Width 7000 so bar getting error. ( fixed )

Test Video

Screen.Recording.2022-01-21.at.17.57.44.mov

[cangencler]

Yaptigin degisikliklerin testlerini ekleyebilir misin?

[nesimtunc]

Known issue : when Temperature is 7000 C, user changed to C -> K ( new Temperature is equals 7273 ) but bar max Width 7000 so bar getting error.

This error is not caused by this commit, it is already present.

Bunu çözmeniz bu PR'da çözebilir misiniz rica etsek? Ya da farklı bir issue da oluşturarak yapabilirsiniz.

@asiminnesli

[asiminnesli]

@cangencler en kısa zamanda ekleyeceğim sanıırım testleri video olarak ekliyoruz.

@nesimtunc bunun için projede daha yetkili birinin yönlendirmesini tercih ederim. Çünkü projenin akışıyla ilgili değişikliklere yol açacaktır. Birşeyler deneyebilirim ama

[HKskn]

@asiminnesli koda biraz göz attım, inputtan değer alacaksa alacağı değeri max 7000 °C olacak şekilde düzenleyebilir miyiz?
Aynı şekilde Farklı sıcaklık birimlerinin max değerini alabilmeli; K için 7273, F için 12632.

Sliderdaki problem için; slider componenti içinde vue-range-slider componentine max değeri static verilmiş(7000)

<vue-range-slider ref="slider" :value="viewTemperature" :tooltip="'none'" :min="minDegree" :max="7000" :drag-on-click="true" @change="onTemperatureChange" @dragging="onTemperatureChange" />

min de yapıldığı gibi seçilen ölçüm birimine göre bir güncelleme muhtemelen çözecektir (minDegree fonksiyonundan referans alabilirsin). Hazır buralara dokunmuşken bunu da çözmen epey yardımcı olacak

[cangencler]

@cangencler en kısa zamanda ekleyeceğim sanıırım testleri video olarak ekliyoruz.

Hayır testleri dosya olarak ekliyoruz.