Bug 18749

Summary: need improvements for radiobuttons
Product: Sisyphus Reporter: inger <inger>
Component: alterator-browser-qtAssignee: Sergey V Turchin <zerg>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: normal    
Priority: P2 CC: inger
Version: unstable   
Hardware: all   
OS: Linux   

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

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

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