leewoody / psi-dev

Automatically exported from code.google.com/p/psi-dev
0 stars 0 forks source link

Доработка патча на многострочные таббары #595

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
1. мидлклик и лефтклик работают только на 
первой строке
2. сделать возможность отключения 
многострочности
3. если на последней строке одна вкладка - 
растягивается на всю строку. некрасиво.

Original issue reported on code.google.com by wadea...@gmail.com on 3 Oct 2014 at 1:11

GoogleCodeExporter commented 8 years ago
А пример со скриншотом можно привести? Как 
оно выглядит/должно выглядеть?

Original comment by vladimir.shelukhin on 3 Oct 2014 at 2:06

GoogleCodeExporter commented 8 years ago
Я бы еще добавил:

4. по-умолчанию должен быть включен именно 
многострочный режим, потому что он многим 
пользователям будет интереснее (либо 
удобнее), а новые пользователи долго будут 
искать как же его включить, либо вообще им в 
голову не придет факт его существования. а 
еще в многострочном режиме вкладки не 
залазят под кнопочки, что справа вверху 
(закрыть текущую вкладку и ту со списком 
вкладок и стрелкой вниз), что есть 
нереально очень очень хорошо.

Original comment by W.prox...@gmail.com on 3 Oct 2014 at 2:20

GoogleCodeExporter commented 8 years ago
>> #1 А пример со скриншотом можно привести? 
Как оно выглядит/должно выглядеть?

сейчас оно выглядит так, в зависимости от 
ширины окна:
http://postimg.org/image/x7gnqlchb/
http://postimg.org/image/7t4023nmv/
http://postimg.org/image/a5u0ag9lf/

но в этом виде наблюдается например:
>> 3. если на последней строке одна вкладка - 
растягивается на всю строку. некрасиво. 

Original comment by W.prox...@gmail.com on 3 Oct 2014 at 2:47

GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
>> #1 А пример со скриншотом можно привести? 
Как оно выглядит/должно выглядеть?

Должно выглядеть примерно так (гимп):
http://postimg.org/image/t73csqwdb/

Хотя лучше если в каждой строчке будет 
одинаковое число табов по возможности 
(близкое к этому значение) с условием что 
использовано минимально возможное число 
строк при ширине каждого таба не меньше 
минимально допустимой. Как-то так.

Original comment by W.prox...@gmail.com on 3 Oct 2014 at 4:59

GoogleCodeExporter commented 8 years ago
5. Когда во неактивной вкладке с 
конференцией (возможно и для приватного 
чата, надо проверять) появляется новое 
сообщение, адресованное Вам (возможно и 
просто любое новое сообщение), то переход 
на эту вкладку не сбрасывает 
значка-облачка (о новых сообщениях) вкладки 
при уловии, что у Вас установлен 
какой-нибудь стаут типо Away (возможно и при 
статусе Online то же самое). Если все-таки 
связано с хайлайтом, то надо проверять 
совместимость с патчем liuch для поддержки 
событий-уведомлений о хайлайтах в 
конференциях. Лично у меня включено, чтобы 
эти события гененрировались (то есть не так 
как по-умлочанию options.ui.muc.allow-highlight-events ). 
Чтобы значок все-таки сбросить на 
правильный сейчас нужно перещелкнуть 
временно на другую вкладку и обратно (?надо 
ли обязательно обратно?).

Original comment by W.prox...@gmail.com on 6 Oct 2014 at 5:53

GoogleCodeExporter commented 8 years ago
Вкладки фиксированной ширины удобнее и 
выглядят лучше. Хороший пример -- 
дополнение Tab Mix Plus в Firefox. В нем можно 
указать минимальную и максимальную ширину 
вкладок, и она будет меняться только в этих 
пределах в зависимости от доступного 
пространства.

Original comment by tehnic...@yandex.ru on 6 Oct 2014 at 12:12

GoogleCodeExporter commented 8 years ago

Original comment by drizt@land.ru on 8 Oct 2014 at 8:09

Attachments:

GoogleCodeExporter commented 8 years ago
Добавил опцию options.ui.tabs.multi-rows, по умолчанию 
true.
Так же пофикшены баги с отображением табов.

Original comment by drizt@land.ru on 10 Oct 2014 at 9:38

GoogleCodeExporter commented 8 years ago
Я обновил Psi+ до последней версии с гитахаба 
дабы увидеть последние улучшения в этом 
патче. Пункт options.ui.tabs.multi-rows в настройках не 
появился, но это наверно нормально. Я ввел 
его вручную со значением false и перезапустил 
Psi+. Увидел обычный режим табов в одну 
строчку ввреху как и ожидалось. Далее я 
установил options.ui.tabs.multi-rows в true и 
перезапустил, и тут я увидел нечто 
неожиданное: замечательные многострочные 
табы уползли вних окна:
http://postimg.org/image/outnzpjul/ 

Original comment by W.prox...@gmail.com on 10 Oct 2014 at 3:24

GoogleCodeExporter commented 8 years ago
проверь настройку options.ui.tabs.put-tabs-at-bottom, true - 
табы внизу, false - вверху.

Original comment by drizt@land.ru on 10 Oct 2014 at 5:08

GoogleCodeExporter commented 8 years ago
спасибо. ну я даю: похоже я случайно сделал 
двойной клик по options.ui.tabs.put-tabs-at-bottom и она 
перещелкулась с false на true. дело в том, что 
она там прямо под options.ui.tabs.multi-rows находится. 
в общем теперь все в порядке.

а поводу этого патча на поддержку 
многострочных вкладок:

6. В режиме многострочных вкладок не 
работает перетаскивание вкладки мышью. 
Хотя оно и не нужно, пусть себе не работает.

Original comment by W.prox...@gmail.com on 10 Oct 2014 at 5:47

GoogleCodeExporter commented 8 years ago
1) У активной вкладки крестик (закрыть) был  
красного цвета, что выделяло активную 
вкладку от остальных, что прошу заметить 
очень полезно (скриншот 
http://pix.academ.org/img/2014/10/26/932d13b75babd2b6429d1623c77a2b45.png ) 
прошу вернуть
2) ширина одной вкладки не оправдано мала 
(скриншот 
http://pix.academ.org/img/2014/10/26/f6fe0ca59f32f94ad9c5c2203a6ebdc0.png ) 
что нас возвращает к комментарию #10 что на  
самом деле очень логично и удобно, как 
пример можно привести ещё и tkabber.
3) как сказано в #15 перетаскиванию мышью не 
работает, может оно кому-то и не нужно, а мне 
довольно необходимо, так что надо починить.

Original comment by korobcoff on 26 Oct 2014 at 1:15

GoogleCodeExporter commented 8 years ago
Re:#16
Могу снова предложить (насчет "2)") сделать 
как я описал в концовке комментария #8 :

"""
Хотя лучше если в каждой строчке будет 
одинаковое число табов по возможности 
(близкое к этому значение) с условием что 
использовано минимально возможное число 
строк при ширине каждого таба не меньше 
минимально допустимой. Как-то так.
"""

При этом как раз табы будут максимально 
возможной ширины при минимально возможном 
количестве строк, и в каждой строчке будет 
примерно одинаковое число вкладок. Я бы 
сказал даже, что будет очень красиво.

Original comment by W.prox...@gmail.com on 26 Oct 2014 at 1:25

GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
Пункт №1 из #16 исправлен
+ 
4) добавить гуевую настройку можно в 
основных настройках, можно в libextendedoptionsplugin 
где не принципиально, но гуевая настройка 
быть на эту опцию должна 
5) при открытии первой вкладки её ширина 
сперва больше, но через долю секунды (~0.3 
секунды) становиться меньше, как минимум  
воспроизводиться если первой вкладкой 
открывать конференцию. предположительная 
причина:
>>[17:41:09] <nsof_> taurus: а с размером, такое чувство 
что таб сперва на всё доступное 
пространство, а потом через долу секунды 
срабатывает правило что оно должно быть 
фиксированой ширины, и оно уменьшается, там 
размер не так прописан? В смысл что сначала 
ставиться на всю ширину, а потом 
фиксируется?

Original comment by korobcoff on 26 Oct 2014 at 3:04

GoogleCodeExporter commented 8 years ago
Хочу добавить, что уже несколько жалоб в 
конфе было, что табы не таскаются. Значит, 
это востребовано и нужно исправить.

Original comment by wadea...@gmail.com on 14 Nov 2014 at 8:28

GoogleCodeExporter commented 8 years ago
да, подтверждаю. драг-н-дроп нужен. а зачем 
это сломали?

Original comment by vladimir.shelukhin on 14 Nov 2014 at 8:41

GoogleCodeExporter commented 8 years ago
Окей. Как время будет.

Original comment by drizt@land.ru on 14 Nov 2014 at 9:36

GoogleCodeExporter commented 8 years ago
[20:49:38] <O01eg> Периодически при запуске 
случается F:ASSERT: "index < d->closeButtons.size()" in file 
widgets/tabbar.cpp, line 358
[20:50:15] <O01eg> происходит во время открытия 
конференций из закладок

Original comment by wadea...@gmail.com on 28 Nov 2014 at 7:20