Bug 23770

Summary: при прокрутке изображение не перерисовывается, если окно частично закрыто (+патч)
Product: Sisyphus Reporter: Vladislav Zavjalov <slazav>
Component: gqviewAssignee: Yuri N. Sedunov <aris>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: normal    
Priority: P3 CC: aris, kurakin
Version: unstable   
Hardware: all   
OS: Linux   
Attachments:
Description Flags
patch1 - using gdk_window_scroll instead of "manual" scrolling
none
patch2 - using event->region instead of event->area none

Description Vladislav Zavjalov 2010-07-14 14:51:58 MSD
Открыть достаточно большое изображение. Расположить окно так, чтобы его часть была не видна за краем экрана или за другими окнами. Попробовать двигать изображение (стрелками или левой кнопкой мыши). Части изображения, выдвигающиеся из спятанной области, остаются серыми.

Повесил два патча на эту тему в upstream. Первый лечит указанную проблему, второй - оптимизация, не слишком существенная, но рекомендованая документацией gtk (там есть ссылка).

http://sourceforge.net/tracker/?func=detail&aid=3029153&group_id=4050&atid=104050
http://sourceforge.net/tracker/?func=detail&aid=3029180&group_id=4050&atid=104050

На всякий случай пишу и сюда, вдруг кому-то пригодится, или будут какие-то замечания, дополнения, исправления...
Comment 1 Vladislav Zavjalov 2010-07-14 14:54:25 MSD
Created attachment 4458 [details]
patch1 - using gdk_window_scroll instead of "manual" scrolling
Comment 2 Vladislav Zavjalov 2010-07-14 14:56:09 MSD
Created attachment 4459 [details]
patch2 - using event->region instead of event->area
Comment 3 Andrey Rahmatullin 2010-07-14 15:26:02 MSD
Вроде ж апстрим gqview сдох, вместо него geekie.
Comment 4 Vladislav Zavjalov 2010-07-14 15:34:39 MSD
Не знал. bugzilla там и впрямь какая-то засохшая.

Тогда буду надеяться, что тут эти патчи пригодятся - gqview мне очень нравится, а эта ошибка довольно сильно раздражала.
Comment 5 Andrey Rahmatullin 2010-07-14 15:42:54 MSD
Ну я предлагаю apt-get install gqview- geeqie, затем проверить и при необходимости повесить патчи уже в их апстрим.
Comment 6 Victor Forsyuk 2010-07-14 15:56:31 MSD
Спасибо за патчи. А может сами и соберете? А то у меня сейчас со временtм туго, а в ACL у gqview указан @everybody.
Comment 7 Vladislav Zavjalov 2010-07-14 16:12:06 MSD
2wrar: В geequi все правильно работает. Хотя scroll сделан по-прежнему вручную.

Можно, конечно, тоже попробовать переделать все на gdk_window_scroll - но смысла меньше, да и там теперь еще какие-то overlay'и поверх картинки, про которые наверняка тоже придется думать.

В общем, не буду копаться в geeqie, пока не припрет :)

2force: Хорошо, gqview соберу.
Comment 8 Vladislav Zavjalov 2010-07-14 16:16:44 MSD
собрать из srpm или это уже не важно и можно все перенести в git?
Comment 9 Victor Forsyuk 2010-07-14 17:07:23 MSD
(В ответ на комментарий №8)
> собрать из srpm или это уже не важно и можно все перенести в git?

Уже не имею какого-либо интереса в gqview, поэтому ничего против git не имею.
Comment 10 Repository Robot 2010-07-15 16:44:59 MSD
gqview-2.1.5-alt7 -> sisyphus:

* Thu Jul 15 2010 Vladislav Zavjalov <slazav@altlinux> 2.1.5-alt7
- enhance image scrolling (closes #23770)