nkzxw / psi-dev

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

Исправление бага с невозможностью завершения (по Enter) переименования (F2) контакта из ростера #593

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Шаги по воспроизведению проблемы:
What steps will reproduce the problem?
1. Запускаем Psi+ в режиме все-в-одном-окне 
(отправка сообщений в чате должна быть 
настроена на значение по умолчанию: Enter).
2. Нажимаем на контакт который 
дозволительно переименовывать мышью и 
жмем F2.
3. Переименовываем контакт и пытаемся 
сохранить результат нажатием Enter.

Каков ожидаемый результат?
What is the expected output?
Ожидается что редактирование остановится 
и переименование будет применено.

Что Вы видите вместо этого?
What do you see instead?
Редактирование таким способом завершить 
невозможно (но чтобы завершить можно 
кликнуть мышью на другой контакт в ростере) 
в режиме чата-конференции, вместо этого 
будет отправляться сообщение в конфу если 
оно набрано. В режиме приватного чата 
редактирование может быть завершено, если 
набрано сообщение, тогда оно будет 
завершено и одновременно будет отправлено 
сообщение. 

Какую версию Psi+ / ОС Вы используете?
What version of the Psi+/OS are you using?
Psi+ 0.16.383 (2014-09-18) / Ubuntu 12.04. Qt 4.8.1.

Дополнительная информация по проблеме:
Additional information:
Пачт решающий проблему в приложении в ишью. 
Надо бы его проверить во всех режимах, не 
возникает ли проблем других.

Original issue reported on code.google.com by W.prox...@gmail.com on 22 Sep 2014 at 2:23

Attachments:

GoogleCodeExporter commented 9 years ago
исправленный патч

Original comment by W.prox...@gmail.com on 22 Sep 2014 at 3:05

Attachments:

GoogleCodeExporter commented 9 years ago
этот патч лучше применять где-то в самом 
конце.

Original comment by W.prox...@gmail.com on 23 Sep 2014 at 9:59

GoogleCodeExporter commented 9 years ago

Original comment by wadea...@gmail.com on 3 Oct 2014 at 1:26

GoogleCodeExporter commented 9 years ago
залейте пожалуйста улучшенную версию 
патча этого 2110-psi-fix-send-action.diff в dev. здесь во 
вложении под другим именем. проблему с 
багом падения при изменении настроек не 
решает, но код капельку лучше. и ишью 
вероятно надо переоткрыть.

Original comment by W.prox...@gmail.com on 25 Oct 2014 at 2:35

Attachments:

GoogleCodeExporter commented 9 years ago
я вроде как пофиксил багу патча, из-за 
которой если в настроках 
включить/выключить например опцию 
Chat->Autoresize input field, то Psi+ вылетала на вызове 
функции qt. проверял с qt4 и без webkit. 
исправленный патч во вложении в этому 
сообщению. его надо переименовать в 
2110-psi-fix-send-action.diff, который в dev лежит.

Original comment by W.prox...@gmail.com on 29 Oct 2014 at 1:22

Attachments:

GoogleCodeExporter commented 9 years ago
Последний патч ломает отправку сообщений. 
Ни Enter, ни Shift+Enter не работают. Без патча всё 
работет

Original comment by the...@gmail.com on 30 Oct 2014 at 10:19

GoogleCodeExporter commented 9 years ago
новый враиант патча во вложении.
он же: 
https://github.com/proxy-m/main/blob/5f317d394ec17c4af42fd6294eb4368a822ebcf1/pa
tches/dev/2110-psi-fix-send-action.diff
и он же: https://github.com/psi-plus/main/pull/16

Original comment by W.prox...@gmail.com on 30 Oct 2014 at 6:13

Attachments:

GoogleCodeExporter commented 9 years ago
исправил отступы:
он: 
https://github.com/proxy-m/main/blob/f350c874d896dfbb1bea06d2d4cbf35f52bc81fc/pa
tches/dev/2110-psi-fix-send-action.diff
и он же: https://github.com/psi-plus/main/pull/17

Original comment by W.prox...@gmail.com on 30 Oct 2014 at 6:44