search

Okuyun / Kuran

https://okuyun.github.io/Kuran/
6 stars 3 forks source link

Resizable panes #12

Closed maeyler closed 3 years ago

maeyler commented 3 years ago

This request is from Prof Oğuz Borat:

[geniş ekranda] Sağdaki Kuran ve mealini bir kademe büyütünce meal soldaki "kelime bul/mucem" ekran parçası altına giriyor. Her satırı okurken meali kısmını yatay hareket ettirme ihtiyacı ortaya çıkıyor. Bunun için en soldaki ekranı dilimini biraz daraltmak isterim,

Implemented in Commit 2124e

will be closed after testing

maeyler commented 3 years ago

Note to implementors:

maeyler commented 3 years ago

Explanation added to the help page:

Resizable panes Panel genişliğini değiştirmek için çift okun olduğu noktayı kaydırın

maeyler commented 3 years ago

Reported by @EbrarSena: Paneli büyütmek için çift yönlü oka tıkladığımızda aynı yere tekrar mouse’la basmazsak şöyle bir problem çıkıyor: Slider (Kuran sayfasını değiştirmek için kullandığımız çubuk) işlev değiştirip panel genişliğini ayarlamaya başlıyor.

a0m0rajab commented 3 years ago

Sorunu pek anlamadım 😅 bir az açıklayabilir misiniz hocam?

From: M Akif Eyler notifications@github.com Sent: Sunday, March 7, 2021 6:34:46 PM To: Okuyun/Kuran Kuran@noreply.github.com Cc: Abdurrahman Rajab rajab-2@windowslive.com; Comment comment@noreply.github.com Subject: Re: [Okuyun/Kuran] Resizable panes (#12)

Reported by @EbrarSenahttps://github.com/EbrarSena: Paneli büyütmek için çift yönlü oka tıkladığımızda aynı yere tekrar mouse’la basmazsak şöyle bir problem çıkıyor: Slider (Kuran sayfasını değiştirmek için kullandığımız çubuk) işlev değiştirip panel genişliğini ayarlamaya başlıyor.

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/Okuyun/Kuran/issues/12#issuecomment-792297608, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AELNMKKGSGU5Y7JGTC2LTG3TCOMJNANCNFSM4YQY22CQ.

maeyler commented 3 years ago

Reported by @gulcu:

Bir kere panel genişliği ile oynandıktan sonra, aşağıdaki alana her sol kliklendiğinde panel tekrar resize oluyor. Aynı şey her iki sağ kliğin birinde oluyor. İlk sağ klikte context-menu beliriyor, ikincisinde resize ediyor.

This looks like bug in css resize It happens even without the observer:

Partial solution by @EbrarSena is in the original report above:

aynı yere tekrar mouse’la basmazsak...

Click on the resize icon and the misbehavior is gone But it is not possible to force users to remain on the icon

gulcu commented 3 years ago

The resized panel widths should persist from one session to another / Iqra tekrar tekrar kullanıldığında genişliği elle ayarlanmış paneller en son ayarlandığı şekilde kullanıcının karşısına çıkmalı çünkü: Yüksek çözünürlüklü ekranlarda yazılar küçücük kalıyor. Fontları AA ikonu ile büyütünce ise yazılar Mushaf paneline sığmaz oluyor. Persistence olmayınca (Iqra önceki ayarları hatırlamayanca) kullanıcı her defasında panelleri resize edecek, sonra CSS resize bug yüzünden uygulama her klikte yeni bir resize işlemi yapmaya kalkacak! Geçici bir çözüm önerisi şu: Kullanıcı bir kez resize etsin, sonra kapatsın, tekrar açsın, bir daha da resize ikonunu ellemek zorunda kalmasın. NeedForResize

maeyler commented 3 years ago

This looks like a bug in css resize, it happens even without the observer:

  • Click on the resize icon, drag and release outside the icon
  • Dragging the mouse again causes an unexpected resize
  • When the observer is called, iframe is already resized

I have identified the problem behavior on a test page: https://maeyler.github.io/JS/simple/resize-css.html

maeyler commented 3 years ago

This problem was reported as a chromium bug: https://bugs.chromium.org/p/chromium/issues/detail?id=1188063 Thank you Abdurrahman...

The partial solution implemented above seems to solve our problem

maeyler commented 3 years ago

Yazılımcının en büyük korkusu "bir özellik eklerken başka bir yerini bozmak" bugün başıma geldi. Yukarıda anlatılan Chrome bug çözülmeyince, zor olduğunu bildiğim bir yöntem denedim. Resize çalıştı, swipe bozuldu.

Bir pembe buton daha mı? Evet. Bunlar panelleri kontrol ettiği için daha yukarı seviyede:

image

Beş adet kontrol dar bir bölgeye mi sıkışmış? Evet ama soldakiler Mucem tablosuna ait, bir bütünlüğü var. Sağdakiler ise Mushaf'ın altında ayrı bir bölge. Resize buton bunların arasında bağımsız duruyor., iki panelin boyunu ayarlıyor.

Drag konusunun alt yapısını iki yıl önce hazırdı, derste de anlatıyordum: https://maeyler.github.io/JS/hard/Dragging.html Burada çıkan zorluk, resize işleminin uzun sürmesi nedeniyle mouse, butonun dışına çıkıyor, örnek sayfada bu sorun yoktu. Çözümünü bilmiyorum.

maeyler commented 3 years ago

Bugün sorun çözüldü: resize ve swipe ikisi de çalışıyor

maeyler commented 2 years ago

This problem was reported as a chromium bug: https://bugs.chromium.org/p/chromium/issues/detail?id=1188063

After exactly one year, we get this automated message: image