Bug 18749 - need improvements for radiobuttons
Summary: need improvements for radiobuttons
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: alterator-browser-qt (show other bugs)
Version: unstable
Hardware: all Linux
: P2 normal
Assignee: Sergey V Turchin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-02-05 11:16 MSK by inger@altlinux.org
Modified: 2009-04-15 13:43 MSD (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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)
> есть подозрение что ты уже исправил это.
Да