Home
mc-dev
mc-dev@conference.jabber.ru
17.02.2012< ^ >

Тема: http://www.midnight-commander.org | Latest release: 4.8.1 | Latest stable release: 4.7.5.6
[00:34:24] slavazanko/h вышел из конференции
[01:06:57] BanderOS зашёл в конференцию
[01:35:55] BanderOS вышел из конференции
[04:11:15] repolainen зашёл в конференцию
[04:43:05] repolainen вышел из конференции
[05:14:12] andrew_b зашёл в конференцию
[06:52:36] iliamaslakov зашёл в конференцию
[07:17:16] <iliamaslakov> Привет
[07:18:07] <andrew_b> Да.
[07:23:36] <iliamaslakov> andrew_b, Мастер - научи!
[07:24:26] <iliamaslakov> сделал
[07:24:28] <iliamaslakov> +/* */
+typedef enum
+{
+ PANEL_SCROLL_NO_FLAG = -1,
+ PANEL_SCROLL_LEFT_FLAG = 0,
+ PANEL_SCROLL_RIGHT_FLAG = 1
+} scroll_flag_t;
[07:24:39] <iliamaslakov> - if (ret_frm == 1)
+ if (ret_frm == (PANEL_SCROLL_LEFT_FLAG & PANEL_SCROLL_RIGHT_FLAG))
[07:24:46] <andrew_b> Почему PANEL_ ?
[07:25:12] <iliamaslakov> SCROLL_LEFT_FLAG ?
[07:25:23] <andrew_b> FILE_
[07:25:29] <andrew_b> FILENAME
[07:25:30] <iliamaslakov> ок
[07:25:52] <iliamaslakov> меня булевы операции беспокоят
[07:25:58] <iliamaslakov> я понял что не знаю
[07:26:45] <andrew_b> - if (ret_frm == 1)
+ if (ret_frm == PANEL_SCROLL_RIGHT_FLAG)
[07:27:08] <andrew_b> (PANEL_SCROLL_LEFT_FLAG & PANEL_SCROLL_RIGHT_FLAG) == (0 & 1) == 0
[07:45:40] <iliamaslakov> я уже переделал
[07:45:53] <iliamaslakov> теперь вот это
panel_filename_scroll_left_sign= mc_skin_get ("widget-panel", "vert-scroll-left-sign", "{");
panel_filename_scroll_right_sign = mc_skin_get ("widget-panel", "vert-scroll-right-sign", "}");
[07:45:59] <iliamaslakov> устраивает?
[07:46:35] <iliamaslakov> filename-scroll-left-sign
filename-scroll-right-sign

как вариант
[07:47:59] <iliamaslakov> в виджет-пэнел засунул т.к. вроде как это кусок именно панельного виджета
[07:59:27] <andrew_b> s/sign/char/g
[07:59:51] <andrew_b> vert ?
[08:00:31] <andrew_b> iliamaslakov: https://www.midnight-commander.org/ticket/2526#comment:5
[08:02:02] <andrew_b> Надо во всех скинах заменить sign на char.
[08:02:29] <andrew_b> Мне кажется, немного юзеров используют какие-то свои особенные скины.
[08:02:50] <andrew_b> Большинство сидит вообще на дефолтовом.
[08:18:15] iliamaslakov вышел из конференции
[08:22:22] slavazanko/h зашёл в конференцию
[08:48:20] ASM вышел из конференции
[08:48:21] ASM зашёл в конференцию
[09:06:11] iliamaslakov зашёл в конференцию
[10:01:28] <iliamaslakov> я всё сделал
[10:01:35] <iliamaslakov> ща запушу
[10:06:52] <iliamaslakov> andrew_b, мощнятски запушил 2731
[10:27:06] <andrew_b> Не надо так писать.

added bindings 'ScrollLeft?', 'ScrollRight?' for scroll long filenames in panel

Signed-off-by: Ilia Maslakov < il.smind@gmail.com>

some code & typo fixes.

Signed-off-by: Andrew Borodin < aborodin@vmail.ru>
[10:28:29] <andrew_b> В моём понимании, подписывается коммит уеликом, а не часть.
[10:30:46] <andrew_b> + res |= FILENAME_SCROLL_RIGHT;
[10:30:53] <andrew_b> Это неправильно.
[10:31:05] <andrew_b> + int ret_frm;
[10:31:09] <andrew_b> Это тоже.
[10:33:09] <andrew_b> (file_index - panel->top_file) % llines (panel) + 2
Можно один раз посчитать.
[10:34:35] iliamaslakov вышел из конференции: Replaced by new connection
[10:34:36] iliamaslakov зашёл в конференцию
[10:57:17] <iliamaslakov> > andrew_b: я там напаролся на косяк
[10:57:33] <iliamaslakov> что то не пойму как вычислить ширину
[11:08:30] <iliamaslakov> andrew_b, занят?
[11:36:41] slavazanko/h вышел из конференции
[11:41:26] slavazanko/h зашёл в конференцию
[12:53:00] <iliamaslakov> 2731 на голосованияя
[12:53:04] <iliamaslakov> 2731 на голосованиии
[12:53:07] <iliamaslakov> 2731 на голосовании
[12:57:16] <andrew_b> Я же написал замечания.
[12:57:21] <andrew_b> Они не исправлены.
[13:01:08] <Зося Синицкая> RSS: Новости для Midnight Commander
• Ticket #2731 (long filenames scroll) updated
http://www.midnight-commander.org/ticket/2731#comment:4
• Ticket #2731 (long filenames scroll) updated
http://www.midnight-commander.org/ticket/2731#comment:5
[13:06:29] <iliamaslakov> >andrew_b: ?
[13:06:55] <andrew_b> [13:30]<andrew_b>
+ res |= FILENAME_SCROLL_RIGHT;

[13:31]<andrew_b> Это неправильно.
[13:31]<andrew_b>
+ int ret_frm;

[13:31]<andrew_b> Это тоже.
[13:33]<andrew_b> (file_index - panel->top_file) % llines (panel) + 2
Можно один раз посчитать.
[13:07:00] <iliamaslakov> то что ты сказал я исправил
[13:07:32] <iliamaslakov> ну вообще всё поправлено кроме
>+ res |= FILENAME_SCROLL_RIGHT;
[13:07:48] <iliamaslakov> если не сложно скажи на что заменить
[13:07:58] <andrew_b> Тебе виднее.
[13:08:09] <iliamaslakov> если мне то именно так правильно
[13:08:11] <andrew_b> Я не в курсе, как это должно работать.
[13:08:21] <iliamaslakov> работает правильно
[13:08:26] <andrew_b> Это неправильно.
[13:09:07] <iliamaslakov> поясни пожалуйста почему не правильно
[13:09:27] <andrew_b> + FILENAME_NOSCROLL = -1,
+ FILENAME_SCROLL_LEFT = 0,
+ FILENAME_SCROLL_RIGHT = 1

FILENAME_NOSCROLL | FILENAME_SCROLL_RIGHT == FILENAME_NOSCROLL
[13:09:45] <andrew_b> (FILENAME_NOSCROLL | FILENAME_SCROLL_RIGHT) == FILENAME_NOSCROLL
[13:09:57] <iliamaslakov> я понял
[13:13:50] <andrew_b> panel_content_scroll_left (WPanel * panel)
{
+ show_dir (panel);
if (panel->content_shift > -1)
{
[13:14:41] <andrew_b> А обязательно рамку перерисовывать при каждом нажатии на ScrollLeft, если сдвига не было?
[13:17:04] <iliamaslakov> if (ret_frm == FILENAME_SCROLL_RIGHT)
[13:17:22] <andrew_b> ?
[13:17:25] <iliamaslakov> на что заменить чтобы проверить наличие флага?
[13:17:33] <iliamaslakov> & ?
[13:17:50] <andrew_b> Ничем.
[13:18:05] <andrew_b> У тебя не битовые флаги.
[13:18:11] <andrew_b> Маску не наложишь.
[13:21:30] <iliamaslakov> я поправил
[13:21:34] <iliamaslakov> запушил
[13:22:20] <andrew_b> if ((ret_frm & FILENAME_SCROLL_RIGHT) == 0)
[13:23:25] <iliamaslakov> а if (ret_frm & FILENAME_SCROLL_RIGHT) не корректно?
[13:23:42] <andrew_b> Корректно, но у нас так не принято.
[13:24:19] <iliamaslakov> исправил
[13:24:20] <andrew_b> 922 if (ret_frm != FILENAME_NOSCROLL)
923 {
924 if (mv)
925 {

Можно соединить в один if.
[13:24:29] <iliamaslakov> ок
[13:25:06] <andrew_b> Всё, я домой.
[13:25:36] <iliamaslakov> поправил
[13:25:44] <iliamaslakov> спасибо за
[13:25:48] <andrew_b> Дома досмотрю.
[13:25:51] andrew_b вышел из конференции
[13:25:53] <iliamaslakov> ок
[13:57:33] <iliamaslakov> >А обязательно рамку перерисовывать при каждом нажатии на ScrollLeft, если сдвига не было?
поправил
[14:01:42] <Зося Синицкая> RSS: Новости для Midnight Commander
• Ticket #2530 (Don't reset selection on mouse clicks, but only on mouse drags) reopened
http://www.midnight-commander.org/ticket/2530#comment:6
• mc-4.8.0-reset-after-store.diff attached to Ticket #2530
http://www.midnight-commander.org/attachment/ticket/2530/mc-4.8.0-reset-after-store.diff
[15:30:59] andrew_b зашёл в конференцию
[15:32:09] <andrew_b> iliamaslakov: описочка вышла

- if ((ret_frm & FILENAME_SCROLL_RIGHT) == 0)
+if ((ret_frm & FILENAME_SCROLL_RIGHT) != 0)
[16:08:29] <iliamaslakov> >andrew_b: iliamaslakov: описочка вышла
нет нет
[16:08:37] <iliamaslakov> всё верно написано
[16:10:59] <iliamaslakov> andrew_b, тут?
[16:37:34] <iliamaslakov> не тут видать...
[17:04:00] sfionov вышел из конференции
[17:05:22] sfionov зашёл в конференцию
[17:15:45] slavazanko/h вышел из конференции
[17:19:34] slavazanko/h зашёл в конференцию
[17:33:32] slavazanko/h вышел из конференции
[17:39:46] slavazanko/h зашёл в конференцию
[17:43:38] <andrew_b> Тут.
[17:54:40] andrew_b вышел из конференции
[20:42:38] repolainen зашёл в конференцию
[21:46:11] repolainen вышел из конференции
Powered by freQ Powered by Python Valid XHTML 1.0 Transitional Valid CSS!