Re: Issue 563 in psi-dev: Доработка патча на тулбары

[psi...@googlecode.com]

Updates:
Summary: Доработка патча на тулбары
Labels: Priority-High Type-Defect

Comment #4 on issue 563 by wadea...@gmail.com: Доработка патча на тулбары
http://code.google.com/p/psi-dev/issues/detail?id=563

Сделать тулбары включенными по-умолчанию

--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

[psi...@googlecode.com]

Comment #5 on issue 563 by li...@mail.ru: Доработка патча на тулбары
http://code.google.com/p/psi-dev/issues/detail?id=563

Fixed:
https://github.com/psi-plus/main/commit/7a6f6f55e2d30085a655ab28079bbd89357f5cd8

[psi...@googlecode.com]

Comment #6 on issue 563 by li...@mail.ru: Доработка патча на тулбары
http://code.google.com/p/psi-dev/issues/detail?id=563

Забыл добавить: пофикшено только то, что указано в #1.

[psi...@googlecode.com]

Comment #7 on issue 563 by dr...@land.ru: Доработка патча на тулбары
http://code.google.com/p/psi-dev/issues/detail?id=563

В новом профиле при создании в оригинальном Psi, тулбары выключены. Поэтому
это не задача для моего патча. Если надо чтоб они были включены изначально
это отдельная задача.

[psi...@googlecode.com]

Comment #8 on issue 563 by li...@mail.ru: Доработка патча на тулбары
http://code.google.com/p/psi-dev/issues/detail?id=563

Тоже сыпется в консоль при запуске:
[15:59:34] <liuch> Dealer_WeARE:
[13:49:18] "Attention Plugin"
[13:49:18] "Chess Plugin"
[13:49:18] "Extended Menu Plugin"
[13:49:18] "GMail Service Plugin"
[13:49:18] "GnuPG Key Manager"
[13:49:18] "Gomoku Game Plugin"
[13:49:18] "Image Plugin"
[13:49:18] "Off-the-Record Messaging Plugin"
[13:49:18] "Watcher Plugin"
[13:49:18] "Conference Logger Plugin"
[13:49:18] "Image Plugin"
...
[16:30:09] <liuch> Dealer_WeARE: Кажется я нашел виноватого. Решил поискать
связь между этими плагинами. И нашел -
http://pix.academ.org/img/2014/03/18/f82c684cadee5cbbc69bfa360816de57.png
[16:30:12] <iSida> Размер .../f82c684cadee5cbbc69bfa360816de57.png - 81.91Kb
[16:30:35] <liuch> taurus: Похоже и тут твой патч виноват. Хотя не проверял.
Есть шанс что патч виноват, но маловероятно.

[psi...@googlecode.com]

Comment #9 on issue 563 by li...@mail.ru: Доработка патча на тулбары

http://code.google.com/p/psi-dev/issues/detail?id=563

Тоже сыпется в консоль при запуске:

[13:49:18] "Attention Plugin"
[13:49:18] "Chess Plugin"
[13:49:18] "Extended Menu Plugin"
[13:49:18] "GMail Service Plugin"
[13:49:18] "GnuPG Key Manager"
[13:49:18] "Gomoku Game Plugin"
[13:49:18] "Image Plugin"
[13:49:18] "Off-the-Record Messaging Plugin"
[13:49:18] "Watcher Plugin"
[13:49:18] "Conference Logger Plugin"
[13:49:18] "Image Plugin"

[16:30:09] <liuch> Dealer_WeARE: Кажется я нашел виноватого. Решил поискать
связь между этими плагинами. И нашел -
http://pix.academ.org/img/2014/03/18/f82c684cadee5cbbc69bfa360816de57.png
[16:30:12] <iSida> Размер .../f82c684cadee5cbbc69bfa360816de57.png - 81.91Kb
[16:30:35] <liuch> taurus: Похоже и тут твой патч виноват. Хотя не проверял.

Есть шанс что патч не виноват, но маловероятно.

зы: когда же тут прикрутят возможность редактировать свои комментарии?

[psi...@googlecode.com]

Comment #10 on issue 563 by dr...@land.ru: Доработка патча на тулбары
http://code.google.com/p/psi-dev/issues/detail?id=563

Это виноват мой патч, там qDebug() ненужный

[psi...@googlecode.com]

Comment #11 on issue 563 by li...@mail.ru: Доработка патча на тулбары
http://code.google.com/p/psi-dev/issues/detail?id=563

#9 fixed:
https://github.com/psi-plus/main/commit/5c90d93870b86307811357a5335a6e2ea88a80b4

[psi...@googlecode.com]

Comment #12 on issue 563 by wadea...@gmail.com: Доработка патча на тулбары
http://code.google.com/p/psi-dev/issues/detail?id=563

#c7 Логично включить исправление именно в этот патч. Иначе придется делать
патч на патч, а это тупо

[psi...@googlecode.com]

Comment #13 on issue 563 by dr...@land.ru: Доработка патча на тулбары
http://code.google.com/p/psi-dev/issues/detail?id=563

можешь тогда закоммитить это в Psi, закоммитишь включу по умолчанию в своем
патче.

Attachments:
psi-enable-central-toolbar.patch 908 bytes

[psi...@googlecode.com]

Comment #14 on issue 563 by wadea...@gmail.com: Доработка патча на тулбары
http://code.google.com/p/psi-dev/issues/detail?id=563

при чем тут пси? исправь 0220 патч.

[psi...@googlecode.com]

Comment #15 on issue 563 by dr...@land.ru: Доработка патча на тулбары
http://code.google.com/p/psi-dev/issues/detail?id=563

почему кстати в Psi+ и Psi разные настройки по умолчанию?

[psi...@googlecode.com]

Comment #16 on issue 563 by vladimir.shelukhin: Доработка патча на тулбары
http://code.google.com/p/psi-dev/issues/detail?id=563

drizt: тут кое-что есть http://code.google.com/p/psi-dev/issues/detail?id=31

[psi...@googlecode.com]

Comment #17 on issue 563 by dr...@land.ru: Доработка патча на тулбары
http://code.google.com/p/psi-dev/issues/detail?id=563

Ага весьма ценно. Из переписки видно что задача на включенный тулбар по
умолчанию была, http://code.google.com/p/psi-dev/issues/detail?id=31#c13 ,
потом задача была закрыта, и по всей видимости это изменение сделано не
было. Закрыто кстати дилером. Посмотрел svn репозиторий, там это сделано,
вообщем. Это потом rion убрал здесь
https://github.com/psi-plus/main/commit/b9976af28835dbfe42eeeced9f6bdc07262c3224#diff-92cb10aaee9e8bf5cd7b184d0464db65L30 .

[psi...@googlecode.com]

Comment #18 on issue 563 by dr...@land.ru: Доработка патча на тулбары
http://code.google.com/p/psi-dev/issues/detail?id=563

Вообщем хоть это и не моя ошибка, и вообще не ошибка, я это исправил.
https://github.com/psi-plus/main/commit/2c0b220a9a99c517c1814b381ff7b95006407149

[psi...@googlecode.com]

Comment #19 on issue 563 by andreyma...@gmail.com: Доработка патча на
тулбары
https://code.google.com/p/psi-dev/issues/detail?id=563

Почему-то попросили написать это сюда; в общем, если переключить настройки
режима вкладок в <<всё в одном окне>> и обратно в <<для чатов и конференций в
одном окне>> без перезапуска, то тулбары съезжают.

Attachments:
psi-toolbar.mp4 897 KB