Bug 18749 - need improvements for radiobuttons
: need improvements for radiobuttons
Status: CLOSED FIXED
: Sisyphus
(All bugs in Sisyphus/alterator-browser-qt)
: unstable
: all Linux
: P2 normal
Assigned To:
:
:
:
:
:
  Show dependency tree
 
Reported: 2009-02-05 11:16 by
Modified: 2009-04-15 13:43 (History)


Attachments


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2009-02-05 11:16:20
Очень хочется чтобы по аналогии с другими input'ами в radio-кнопках было
событие changed (синоним toggled). Более того:

toggled на группе кнопок будет вызываться дважды (сначала по поводу кнопки с
которой снято выделение, а потом по поводу кнопки которая стала выделенной).
В связи с этим есть предложение включать группы кнопок в группу QButtonGroup и
отслеживать событие buttonClicked (собственно на него и генерировать changed).

И ещё момент: Сейчас в browser-qt есть магия по объединению радио-кнопок в
группы. Можно смело эту магию убирать ибо сейчас все кнопки из одной группы
имеют одинаковое значение атрибута name (как в html). Таким образом по "name"
кнопки можно объединять в разные QButtonGroup'ы.
------- Comment #1 From 2009-02-05 20:12:31 -------
Зачем синоним toggled? Есть же toggled
------- Comment #2 From 2009-02-06 11:47:58 -------
(In reply to comment #1)
> Зачем синоним toggled? Есть же toggled
Чтобы было как в html, единообразное событие change на все инпуты. Ты же делал
changed на select,checkbox, etc.
------- Comment #3 From 2009-02-06 14:23:18 -------
(В ответ на комментарий №2)
> Ты же делал changed на select,checkbox, etc.
Ой, правда. Проверю, работает ли оно. А то, как-минимум, в одном месте оно не
учтено вообще.
------- Comment #4 From 2009-02-07 10:47:16 -------
(In reply to comment #3)
> (В ответ на комментарий №2)
> > Ты же делал changed на select,checkbox, etc.
> Ой, правда. Проверю, работает ли оно. А то, как-минимум, в одном месте оно не
> учтено вообще.
Обрати внимание на мои предложения насчёт QButtonGroup, это могло бы снять
много гимороя и не получать по два сигнала changed(toggled) за раз ;)
------- Comment #5 From 2009-02-09 20:30:35 -------
К слову, магию просто так не уберешь. Придется написать QGroupBox, но без
магии.
------- Comment #6 From 2009-04-14 17:55:49 -------
Серёга, есть подозрение что ты уже исправил это.
------- Comment #7 From 2009-04-15 13:43:36 -------
(В ответ на комментарий №6)
> есть подозрение что ты уже исправил это.
Да